[Bug 225254] Merge Review: apr-util

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]