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

Daiki Ueno <du...@redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |du...@redhat.com
         AssignedTo|nob...@fedoraproject.org    |du...@redhat.com
               Flag|                            |fedora-review?

--- Comment #2 from Daiki Ueno <du...@redhat.com> 2010-05-28 02:53:59 EDT ---
Mostly ok.  How about including FONTLOG to doc?

Cosmetic things:

- the "Version" is 003.000 (which came from the font, right?), but the upstream
uses simpler naming for the archive: ofl-goudy-3.0.zip.  I think we could use
3.0.

- rpmlint spews a couple of warnings (harmless, I think):

 ofl-sorts-mill-goudy-fonts.src: W: invalid-url Source0:
http://sortsmill.googlecode.com/files/ofl-goudy-3.0.zip HTTP Error 404: Not
Found
 ofl-sorts-mill-goudy-fonts.src: W: uncompressed-zip ofl-goudy-3.0.zip

I have no idea how to workaround them :)

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

Reply via email to