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: bouml-doc - Documentation for the BOUML tool https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=249892 mtasaka@xxxxxxxxxxxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+ ------- Additional Comments From mtasaka@xxxxxxxxxxxxxxxxxxx 2007-08-11 11:25 EST ------- Actually almost nothing to be reviewed, however: * rpmlint complaints: -------------------------------------------------- W: bouml-doc invalid-license GPL W: bouml-doc no-%build-section -------------------------------------------------- - Change the license to GPLv2+ - Add %build section, even if it is empty -------------------------------------------------- $ rpmlint -I no-%build-section no-%build-section : The spec file does not contain a %build section. Even if some packages don't directly need it, section markers may be overridden in rpm's configuration to provide additional "under the hood" functionality, such as injection of automatic -debuginfo subpackages. Add the section, even if empty. -------------------------------------------------- Other thing okay. Please fix the issue above before committing to CVS. -------------------------------------------------- This package (bouml-doc) is APPROVED by me -------------------------------------------------- Note: the template of CVSAdminProcedure changed so please re-read http://fedoraproject.org/wiki/PackageMaintainers/CVSAdminProcedure -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review