[Bug 226517] Merge Review: urw-fonts

[bugzilla at redhat.com]

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: urw-fonts


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


roozbeh at farsiweb.info changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|nobody at fedoraproject.org    |than at redhat.com
                 CC|                            |roozbeh at farsiweb.info
               Flag|                            |fedora-review-




------- Additional Comments From roozbeh at farsiweb.info  2007-02-03 09:12 EST -------
Random notes:
* As upstream version is 1.0.7pre40, I believe EVR should be changed to
1:1.0.7-0.1.pre40
* Summary ends with a period. It shouldn't.
* Remove the comma after "Free" in the description field.
* Should not mention the copyright holder in the License field.
* Change BuildRoot to %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
* Perhaps the individual fonts that are listed as Source1 to Source8 could be
made into a tarball, which also makes the SPEC file (%install section) more
readable.


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