Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: google-droid-fonts - The Droid font set
https://bugzilla.redhat.com/show_bug.cgi?id=475661
Summary: Review Request: google-droid-fonts - The Droid font set Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: nicolas.mailhot@laposte.net QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
Spec URL: http://nim.fedorapeople.org/fontpackages/google-droid-fonts.spec SRPM URL: http://nim.fedorapeople.org/fontpackages/google-droid-fonts-1.0.112-1.fc11.s... Description: The Droid typeface family was designed in the fall of 2006 by Ascender’s Steve Matteson, as a commission from Google to create a set of system fonts for its Android platform. The goal was to provide optimal quality and comfort on a mobile handset when rendered in application menus, web browsers and for other screen text.
http://fedoraproject.org/wiki/Google_Droid_fonts
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=475661
Nicolas Mailhot nicolas.mailhot@laposte.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |fedora-fonts-bugs-list@redh | |at.com Blocks| |446451 Depends on| |472635, 475593
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=475661
Bug 475661 depends on bug 472635, which changed state.
Bug 472635 Summary: Google Droid fonts licensing https://bugzilla.redhat.com/show_bug.cgi?id=472635
What |Old Value |New Value ---------------------------------------------------------------------------- Status|NEW |VERIFIED Resolution| |UPSTREAM Status|VERIFIED |CLOSED
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=475661
Nicolas Mailhot nicolas.mailhot@laposte.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends on|475593 |
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=475661
Nicolas Mailhot nicolas.mailhot@laposte.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends on| |475593
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=475661
Orcan 'oget' Ogetbil oget.fedora@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |oget.fedora@gmail.com AssignedTo|nobody@fedoraproject.org |oget.fedora@gmail.com Flag| |fedora-review+
Bug 475661 depends on bug 475593, which changed state.
Bug 475593 Summary: Review Request: fontpackages - Common directory and macro definitions used by font packages https://bugzilla.redhat.com/show_bug.cgi?id=475593
What |Old Value |New Value ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
--- Comment #1 from Orcan 'oget' Ogetbil oget.fedora@gmail.com 2009-01-14 14:30:21 EDT --- I poked around this package for 2 hours now. I couldn't find any blockers. There are two minor things I think I should say:
* Not a blocker but, as I say in all my reviews, I prefer using %defattr(-,root,root,-)
* All fonts (including sans-serif ones) say that they are "Serif" as (OS/2) PFM-Family in fontforge. I think upstream needs notified.
Other than these two, you might want to wait for the subpackage naming decision of FESCO for http://fedoraproject.org/wiki/PackagingDrafts/Font_package_naming_(2009-01-1...)
Otherwise, everything is clean
----------------------------------------------------- This package (google-droid-fonts) is APPROVED by oget -----------------------------------------------------
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=475661
Nicolas Mailhot nicolas.mailhot@laposte.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Summary|Review Request: |Review Request: |google-droid-fonts - The |google-droid-fonts - |Droid font set |General-purpose fonts | |released by Google as part | |of Android Flag| |fedora-cvs?
--- Comment #2 from Nicolas Mailhot nicolas.mailhot@laposte.net 2009-01-14 16:53:36 EDT --- (In reply to comment #1)
I poked around this package for 2 hours now. I couldn't find any blockers. There are two minor things I think I should say:
- Not a blocker but, as I say in all my reviews, I prefer using
%defattr(-,root,root,-)
For fonts an explicit defattr is part of the guidelines since there is no reason to have fonts with strange permissions installed :p
- All fonts (including sans-serif ones) say that they are "Serif" as (OS/2)
PFM-Family in fontforge. I think upstream needs notified.
Missed this, nice to know, will do
Other than these two, you might want to wait for the subpackage naming decision of FESCO for http://fedoraproject.org/wiki/PackagingDrafts/Font_package_naming_(2009-01-1...)
Since the changes FPC requested didn't affect srpm naming, I'll do the cvs request now.
This package (google-droid-fonts) is APPROVED by oget
Thank you!
New Package CVS Request ======================= Package Name: google-droid-fonts Short Description: General-purpose fonts released by Google as part of Android Owners: nim Branches: F-9 F-10 InitialCC: fonts-sig
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=475661
Kevin Fenzi kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
--- Comment #3 from Kevin Fenzi kevin@tummy.com 2009-01-15 15:26:50 EDT --- cvs done.
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=475661
Nicolas Mailhot nicolas.mailhot@laposte.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |RAWHIDE
--- Comment #4 from Nicolas Mailhot nicolas.mailhot@laposte.net 2009-01-16 17:09:32 EDT --- Will let it steam in rawhide a bit before importing in stable releases as it includes CJK fonts and we always seem to get them wrong at first somehow
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=475661
Rex Dieter rdieter@math.unl.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |feng.shaun@gmail.com
--- Comment #5 from Rex Dieter rdieter@math.unl.edu 2009-04-01 10:17:37 EDT --- *** Bug 493240 has been marked as a duplicate of this bug. ***
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=475661
--- Comment #6 from Fedora Update System updates@fedoraproject.org 2009-04-15 14:02:41 EDT --- google-droid-fonts-1.0.112-3.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report.
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=475661
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version| |1.0.112-3.fc10 Resolution|RAWHIDE |NEXTRELEASE
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=475661
Mark Chappell tremble@tremble.org.uk changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |tremble@tremble.org.uk Flag|fedora-cvs+ |fedora-cvs?
--- Comment #7 from Mark Chappell tremble@tremble.org.uk 2010-06-22 09:38:53 EDT --- Package Change Request ====================== Package Name: google-droid-fonts New Branches: EL-6 Owners: tremble
nim listed as welcoming EPEL maintainers.
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=475661
--- Comment #8 from Nicolas Mailhot nicolas.mailhot@laposte.net 2010-06-22 14:38:58 EDT --- BTW if you have time to spend on the Fedora package, co-maintainers are welcome there too
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=475661
Kevin Fenzi kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
--- Comment #9 from Kevin Fenzi kevin@tummy.com 2010-06-22 21:41:18 EDT --- cvs done.
package-review@lists.fedoraproject.org