[Bug 1170374] Review Request: yum-plugin-needs-restarting - YUM plugin for listing processes using old files after upgrade

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1170374

Miroslav Suchý <msuchy@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |msuchy@xxxxxxxxxx



--- Comment #1 from Miroslav Suchý <msuchy@xxxxxxxxxx> ---
1) I'm not sure if it is worth the work to continue adding something for yum. I
would focus work on some plugin for DNF instead.

2) There is much better plugin (for DNF) called tracer. See bug 1158629


And if you want to learn something:

* %defattr(-, root, root)
This is default and not needed since rpm 4.2

* %{__python} -c "import compileall;
compileall.compile_dir('%{buildroot}/%pluginhome', 1)"
rpm do that automatically, no need to do that.

* rm -rf %{buildroot}
not needed: http://fedoraproject.org/wiki/Packaging:Guidelines#.25clean

* you should preserve timestamps and use "cp -a"

* license should be marked using %license macro
  %license COPYING

* do not enumerate all extension and just use:
  %{pluginhome}/needs-restarting.py*

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]