[Bug 1000949] Review Request: ttfautohint - Automated hinting utility for TrueType fonts

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

 



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

Zbigniew Jędrzejewski-Szmek <zbyszek@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|fedora-review?              |fedora-review+



--- Comment #4 from Zbigniew Jędrzejewski-Szmek <zbyszek@xxxxxxxxx> ---
(In reply to Christopher Meng from comment #3)
> The identical docs are needed as -gui is a dependent package which doesn't
> need to require main package:
> 
> %package        gui
> Summary:        GUI for %{name} based on Qt4
OK. But I think that the docs should not be *installed* twice. ttfautohint-gui
can contain stuff in /usr/share/docs/ttfautohint, so that the user doesn't get
a 1.5 MB hit on installing both. Just a suggestion.

> For the file-not-utf8 issue, I think it can be ignored.
OK.

> -----------------
> 
> Spec URL: http://cicku.me/ttfautohint.spec
> SRPM URL: http://cicku.me/ttfautohint-0.97-1.fc21.src.rpm

Package is APPROVED.

-- 
You are receiving this mail because:
You are always notified about changes to this product and component
_______________________________________________
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]