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=226407

Paul Howarth <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|needinfo?([email protected] |
                   |)                           |

--- Comment #37 from Paul Howarth <[email protected]> 2010-03-05 09:38:42 EST 
---
A few minor nits I spotted this morning:

 * Can we make the -cf and -docs packages noarch?
 * Can we change %define to %global throughout?
 * The perl script interpreter fixup for contrib stuff could get done in %prep
rather than %install
 * %description doc still says that "papers are provided in PostScript(TM) and
troff formats" even though the only format packaged is PDF

I'll have a last look next week when I have a bit more time.

-- 
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
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to