Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=459855 --- Comment #16 from Jarod Wilson <jarod@xxxxxxxxxx> 2009-09-29 11:33:21 EDT --- At a quick glance: - %preun client uses an explicit list of services to stop, but %postun client uses a somewhat odd method of looking at /usr/share/ncid/ncid-* to figure out what to condrestart. I'd make %postun client use the same list as %preun client. - The %files lists still use /etc, that should be replaced with %{_sysconfdir} -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review