[Bug 225254] Merge Review: apr-util
bugzilla at redhat.com
bugzilla at redhat.com
Thu Apr 5 16:11:18 UTC 2007
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 at redhat.com changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |MODIFIED
------- Additional Comments From jorton at 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.
More information about the package-review
mailing list