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=484591 --- Comment #3 from Hans de Goede <hdegoede@xxxxxxxxxx> 2009-02-10 03:14:23 EDT --- With that said, and hopefully made very clear! I would like to review this package as I think it is great that someone is working on moving packages from planetccrma into Fedora proper. The Epoch is fine, but with the other 2 issues I have to side with Ralf, using "AutoProv: no" is not acceptable, your package will miss essential requires and adding them manually is error prone and will break when the soname's of libraries you use change (and you do a rebuild). The correct solution here is to use filtering of the generated provides as explained in the previous review request. Also the warnings are serious and need fixing, Ralf has already explained how (replace "%d" with "%zd"), let me know if there are other warnings which you need help fixing. Can you please do a new revision with these 2 issues fixed? then I'll do a full review. -- 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