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=542461 --- Comment #5 from Nicolas Mailhot <nicolas.mailhot@xxxxxxxxxxx> 2009-12-09 16:56:43 EDT --- (In reply to comment #4) > (In reply to comment #2) > > Sorry for taking so long to review, I monitor fedora-fonts-bugs-list but only > > check for "wild" font package submissions when I have some free time. > > No problem. I'm not a font packager and I missed the suggestion to CC the > fedora-fonts-bugs-list. > > > Anyway, review : > > > > 1. (comment) The font could probably be classified as cursive, but then the > > boundary between "fantasy handwriting" and "realistic handwriting" is a bit > > fuzzy > > See /usr/share/fontconfig/templates/fontconfig-generics.txt > > I choose "fantasy" because I browsed Fedora CVS for other handwriting fonts and > this is what I found: > > brettfont-fonts: There is no fontconfig.conf > http://cvs.fedoraproject.org/viewvc/rpms/brettfont-fonts/devel/ > > sj-fonts: "fantasy" > http://cvs.fedoraproject.org/viewvc/rpms/sj-fonts/devel/ > > dustin-domestic-manners-fonts: "fantasy" > http://cvs.fedoraproject.org/viewvc/rpms/dustin-domestic-manners-fonts/devel/ If you want to say "handwriting" the correct generic is "cursive" in fontconfig and in css. "fantasy" is just where we dump fonts we can not categorize otherwise. > > 4. (blocking) Since this font is distributed via the Open Font LiBrary project, > > the package should be named oflb-dignas-handwriting-fonts > > I renamed the package, but I found really a lot of fonts in Fedora CVS that do > not comply with this guideline. For example the aforementioned brettfont, > icelandic-fonts, asana-math-fonts, roadstencil-fonts, sportrop, etc that are > also distributed via the Open Font Library project. If you knew how I wished their packagers fixed those to comply with our naming guidelines (grumbles)… This is all stuff that antedates the formal font package naming guideline that was adopted last year. New packages can't use old problems as excuse to introduce new ones. > http://musuruan.fedorapeople.org/oflb-dignas-handwriting-fonts.spec > http://musuruan.fedorapeople.org/oflb-dignas-handwriting-fonts-20031109-2.fc12.src.rpm Will check those ASAP -- 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review