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=518539 José Matos <jamatos@xxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Flag| |fedora-review+ --- Comment #5 from José Matos <jamatos@xxxxxxxx> 2009-08-20 18:02:14 EDT --- Review: The source package is the same as upstream. The license is correct (BSD) and appropriate for Fedora. This is even more appropriate in the case since the copyright comes from Berkeley. :-) The package build in the supported Fedora architectures. (see #2) Build on F11 and rawhide. The package follows the packaging guidelines. The package is _approved_. PS: One small detail, I noticed that you have placed README.devel in the devel sub-package, after reading it I am not sure how useful the file is. I think that the content is useless for any developer, but I leave to you the final decision. The README file on the other hand is really helpful and in a sense it belongs more to devel than to the main package. If you think that I am being pedantic in the last argument (README) then I agree with you and you are free to ignore me. -- 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