Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: apr-util https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225254 jorton@xxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |MODIFIED ------- Additional Comments From jorton@xxxxxxxxxx 2007-04-05 12:11 EST ------- Thanks for the review! Please see 1.2.8-7 in CVS/Raw Hide soon. > 1. Build root is not one of the recommended options The buildroot used does meet the mandatory requirements. > 2. It is recommended to use %{?dist} in Release: This is not mandatory. > 3. There is an outstanding bug for apr-util, but it may not apply to > Fedora. Not relevant to review process. > 4. Conflicts: should not be used. Perhaps Requires: subversion >= > 0.20.1-2 instead? Removed, as suggested. > 5 rpmlint output > > W: apr-util invalid-license Apache Software License 2.0 No standard for License tags exists yet. > E: apr-util use-of-RPM_SOURCE_DIR This is not prohibited by packaging guidelines, see f-m@ flamewar. > W: apr-util-devel no-documentation All expected. > 6. Build Requires > doxygen is in BuildRequires but it does not seem to be used in the Removed. > 7. .la files exist and must be removed The .la file is part of the defined build interface for this package. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review