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@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |MODIFIED
------- Additional Comments From jorton@redhat.com 2007-04-05 12:11 EST ------- Thanks for the review! Please see 1.2.8-7 in CVS/Raw Hide soon.
- Build root is not one of the recommended options
The buildroot used does meet the mandatory requirements.
- It is recommended to use %{?dist} in Release:
This is not mandatory.
- There is an outstanding bug for apr-util, but it may not apply to Fedora.
Not relevant to review process.
- 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.
- Build Requires doxygen is in BuildRequires but it does not seem to be used in the
Removed.
- .la files exist and must be removed
The .la file is part of the defined build interface for this package.