[Bug 225302] Merge Review: automake

[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.


https://bugzilla.redhat.com/show_bug.cgi?id=225302

--- Comment #15 from Paul Howarth <paul@xxxxxxxxxxxx> 2010-03-29 05:49:59 EDT ---
A few minor nits:

* Redundant text about installing Autoconf still present in %description
(package dependency enforces installation of autoconf so there's no need to
mention this)

* Creation of empty directory ${RPM_BUILD_ROOT}%{_datadir}/aclocal in %install
is no longer necessary as the package no longer owns this directory

* Old man pages still in CVS and can be removed since they're no longer
referenced from the spec

I'll approve this package tomorrow unless anyone has any further objections.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review

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