[Bug 225617] Merge Review: bitmap-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.


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


Nicolas Mailhot <nicolas.mailhot at laposte.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|nicolas.mailhot at laposte.net |
         AssignedTo|nobody at fedoraproject.org    |nicolas.mailhot at laposte.net
               Flag|                            |fedora-review?,
                   |                            |needinfo?(nobody at fedoraproj
                   |                            |ect.org)




--- Comment #34 from Nicolas Mailhot <nicolas.mailhot at laposte.net>  2009-11-28 12:32:26 EDT ---
Ok, looking at this srpm, most of the warnings and errors are gone. Good!

However:

1. (not blocking) fontlint does not like any of the resulting font files, they
all need fixing but I guess it's not worse than before

2. (not blocking) the lucida typewriter files declare broken style naming that
really needs fixing or apps won't be happy

3. (not blocking) all the files have almost-complete (but not complete)
coverage of one or more unicode blocks (probably reflects unicode changes since
they were last updated)

4. (blocking) License: Lucida is probably wrong for the base package, it should
be GPLv2 and MIT

5. (not blocking) Requires: fontpackages-filesystem is not needed for the base
package

6. (blocking) since the bitmap-fonts package does not exist anymore, it needs
to be garbage-collected. Please add an
Obsoletes: bitmap-fonts < 0.3-14
to the bitmap-fixed-fonts subpackage (let's not be fancy, it's not worth it)

It seems I was a little optimistic before, this package needs some tweaking
yet. But it has progressed a lot

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