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@uos.de changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |martin.gieseking@uos.de
--- Comment #4 from Martin Gieseking martin.gieseking@uos.de 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.