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=544016 --- Comment #5 from Chris St. Pierre <chris.a.st.pierre@xxxxxxxxx> 2010-03-04 11:38:30 EST --- Thanks! I've kept the %codename, but changed %define to %global; I got rid of the others %defines. The naming of this project is pretty muddled. It's officially called "policyd"; but v.2 was codenamed "cluebringer"; and lots of files are named with "cbpolicyd". A spec file is distributed with Policyd (which doesn't conform to the Fedora packaging guidelines) that creates an RPM called 'cluebringer'. People with policyd v1 installed will have an RPM installed called 'policyd'. Although these have never been in Fedora as far as I know, I added the Provides and Obsoletes to ensure backwards compat with these older packages since I couldn't find any sort of guidelines on this. %preun fixed. Given that /etc/cluebringer is the only thing specified in more than one package, I don't know that a -common package makes a lot of sense here. The errors you're getting from rpmlint about permissions are actually due to those config files not being world-readable, but they both contain passwords so it's important that they not be. They'll have to be added to the exception list for that check once I get this package into Fedora. Until then, those errors from rpmlint can be ignored. %attr(0640,...) is appropriate here. %config added where necessary. New spec and SRPM: http://www.nebrwesleyan.edu/people/stpierre/cbpolicyd.spec http://www.nebrwesleyan.edu/people/stpierre/cbpolicyd-2.0.10-5.fc12.src.rpm -- 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. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review