Bug 484057 (epigrafica-fonts)
Summary: | Review Request: epigrafica-fonts - The Epigrafica font family. | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Stephen Carter <scarter4> |
Component: | Package Review | Assignee: | Nobody's working on this, feel free to take it <nobody> |
Status: | CLOSED INSUFFICIENT_DATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | low | ||
Version: | rawhide | CC: | ctyler.fedora, fedora-package-review, notting, paskalis, paul, tcallawa |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2010-02-20 16:23:58 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 201449 |
Description
Stephen Carter
2009-02-04 15:30:04 UTC
Some feedback (not a real complete review yet, you have some rework to do first) 1. your packaging is based on the templates we used 2/3 months ago. Since then a major new version has been approved. You need to read the current version of http://fedoraproject.org/wiki/Packaging:FontsPolicy and adapt to it (in particular http://fedoraproject.org/wiki/Packaging:FontsPolicy#Technical_implementation but do read the rest, it has changed and been clarified too) The policy changes have been made to simplify packaging, trying to minimize the work needed to adapt current packages, so it should not be too hard for you. 2. your summary is not very informative 3. URL needs to be the font project homepage, your chosen URL is not really useful in a browser 4. since this entity creates other fonts, a foundry prefix would be a good idea in the naming (for example aegean) 5. since this you know this font is a cosmetica fork, which is itself an optima fork, you need to tell this to fontconfig via some substitution rules. fontpackages-devel has some templates you can follow to do this easily 6. since we've considerably simplified fonts packaging lately, we require at least *two* clean font package submissions before sponsoring someone. So you'll need another submission for this one to succeed. My second font submission can be found here: https://bugzilla.redhat.com/show_bug.cgi?id=485542 . I will be cleaning up this one and uploading fresh packages very soon. Whoops! I didn't notice that someone else had already started working on packaging this font, and I opened my own review request (492900). If you still want to maintain this, I will close mine out, please let me know. (In reply to comment #3) > Whoops! I didn't notice that someone else had already started working on > packaging this font, and I opened my own review request (492900). If you still > want to maintain this, I will close mine out, please let me know. Yes, I'm still going to maintain this one. :) *** Bug 492900 has been marked as a duplicate of this bug. *** I have been in touch with Stephen Carter by email and he has confirmed that he will not be completing this package. I'm going to try to close this bug in accordance with the Package maintainer policy on the wiki. Closing |