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: courier-authlib - Courier authentication library https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=208064 mfleming+rpm@xxxxxxxxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@xxxxxxxxxxxxxxxxx |mfleming+rpm@xxxxxxxxxxxxxxx | |m OtherBugsDependingO|163776 |163778 nThis| | ------- Additional Comments From mfleming+rpm@xxxxxxxxxxxxxxxx 2006-09-26 10:01 EST ------- As I maintain a local version (and have done for a while so I'll probably compare along the way) I'll pick this one up for review. The specfile does deviate from what FE expects, so cleaning it up completely is indeed a challenge. :-) I have noticed a couple of things: * Unless you're patching the init file, it defaults to enabled on boot. * There's a few BR's there that are redundant (gcc-c++, libtool) * The Epoch references in subpackage Requires: are probably not needed, * Have you considered for the userdb databases? (configure --with-db=db) - I just don't trust the default gdbm... -- 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