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=744977 Martin Gieseking <martin.gieseking@xxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |martin.gieseking@xxxxxx --- Comment #4 from Martin Gieseking <martin.gieseking@xxxxxx> 2011-11-27 14:04:53 EST --- There are some things that should be addressed before the package is checked in: - the devel package should require the base package this way: http://fedoraproject.org/wiki/PackagingGuidelines#Requiring_Base_Package - Don't add the %doc files several times. Drop AUTHORS, LICENSE, and COPYING from the devel package. Since it requires the base package, these files are installed anyway. - add README and NOTICE to the base package (with %doc) and doc/QUICK_START to the devel package - I suggest to build the doxygen API documentation (cd into docs/ and run doxygen doxygen.conf) the devel package. - Either add a Group field to the base package (System Environment/Libraries), or remove it from the devel package. Currently, the Group field is used inconsistently. -- 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