Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: pylibacl - POSIX.1e ACLs library wrapper for python https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235527 kevin@xxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- OtherBugsDependingO|163776 | nThis| | Flag|fedora-review? |fedora-review+ ------- Additional Comments From kevin@xxxxxxxxx 2007-04-18 23:10 EST ------- 1. ok. 2. ok. 3. ok. 4. I thought we explicitly wanted (-, root, root, -), but looking at the guilelines it just says that there must be a defattr section for each files section, so I guess thats ok. Note that the fedora package is always going to be built under a controlled setup and the umask shouldn't ever be a problem. I don't see the need for the: #Packager tag removed due to FE requirements #libacl is forced by RPM comments. Thats no blocker, but unless you need them perhaps remove them before import? I don't see any further blockers here, so this package is APPROVED. Don't forget to close this review request once this package has been imported and built. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review