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(a)redhat.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |MODIFIED
------- Additional Comments From jorton(a)redhat.com 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.