Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.

Bug 1362130

Summary: Review Request: olpcau-abc123-fonts - A nice font for kids/readability
Product: [Fedora] Fedora Reporter: Sam P. <sam>
Component: Package ReviewAssignee: Zbigniew Jędrzejewski-Szmek <zbyszek>
Status: ASSIGNED --- QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: unspecified    
Version: rawhideCC: package-review, sam, zbyszek
Target Milestone: ---Flags: zbyszek: fedora-review?
Target Release: ---   
Hardware: All   
OS: Linux   
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Sam P. 2016-08-01 10:49:44 UTC
Spec URL:

abc123 is a typeface developed for literacy John Greatorex, Sridhar
Dhanapalan and Ruben Rodriguez for One Laptop per Child Australia.
abc123 includes letters in the same shapes that children are taught to
write them and avoids shapes hard to read for small children.

Fedora Account System Username:  samtoday

Comment 1 Zbigniew Jędrzejewski-Szmek 2016-08-12 14:43:13 UTC
The Spec URL doesn't seem to work for me, I'm using

Summary should not repeat the package name. It also doesn't need the leading article (it looks better in listing if there's no article, since otherwise almost everything would start with "A").

Group: User Interface/X
→ not used []

I don't think you have to copy sources in %prep. This breaks 'fedpkg local' builds too.

appstream-util validate-relax --nonet should be added in %check:

You should also run full validation locally with "validate" and fix issues:
olpcau-abc123.metainfo.xml: FAILED:
• style-invalid         : <summary> requires sentence case [abc123, a typeface developed for literacy]
Validation of files failed

Package looks good.


I can sponsor you into the packager group. Apart from fixing the few issues pointed out above, I would like you to do two or three reviews of other packages ( is a good start) and to be familiar with mock, fedora-review, etc. There's a bunch of fonts packages waiting for review, so you might want to take those (or not, whatever you prefer). Until you're a packager, you cannot formally approve a package, so please don't assign the review to yourself, just paste whatever comments you have. If nobody beats you to it, you will be able to formally approve those packages after you become packager. If you have any questions, feel free to mail me, or ping on IRC (I'm "zbyszek" everywhere).

Comment 2 Parag AN(पराग) 2016-08-13 05:42:10 UTC

Just my thought here that removal of 177841 at this stage looks a proactive action here. Generally we (Sponsor) make sure that package submitter is having a understanding of rpm packaging by asking him/her to review others packages waiting for their reviews and complete this package review and then remove the 177841 blocker.

Comment 3 Zbigniew Jędrzejewski-Szmek 2016-08-13 16:57:22 UTC
I'm not sure ;) I unset the flag when I make the offer to sponsor someone because I assume that other sponsors are looking at bugs with that flag for people to sponsor. Please correct me if that's not the case.

Comment 4 Parag AN(पराग) 2016-08-14 17:15:13 UTC
I am using this as general practice that let the review be on completion along with sponsorship process requirements and then remove 177841 and sponsor. 

If you are ready to sponsor then first step is to assign the review to yourself and change the flag to fedora-review? This will make sure other sponsors that you already picked this for review and then will be knowing you will complete the sponsorship.

Comment 5 Zbigniew Jędrzejewski-Szmek 2016-11-12 15:35:12 UTC
Any progress?

Comment 6 Sam P. 2016-11-14 11:44:29 UTC
Thanks for the review Zbigniew!

> I don't think you have to copy sources in %prep. This breaks 'fedpkg local' builds too.

I tried removing the copy, and it continued to work via mock and rpmbuild locally on my computer.  However, it caused it to fail when building on COPR [1].  So I kept it in the end.

I've updated the package:

Spec URL:  

[1] and

Comment 7 Zbigniew Jędrzejewski-Szmek 2016-11-14 18:01:58 UTC
developed for literacy John Greatorex → missing "by"

The package looks good. I'll approve the package once you are sponsored into the packagers group. Any chance you could review some other packages?

Comment 9 Zbigniew Jędrzejewski-Szmek 2017-03-29 02:37:29 UTC
Hi, sorry for the long silence. I went away on vacation and then completely forgot about this. For the future, in such cases, it often helps to send a ping after a week or two, just to get things going again. Let's get this ball rolling again!

I looked at your comments on those reviews, but they are a bit scant. Before I asked you for a fedora-review run on #1388945. I'd like to know that you have set up mock and can do test builds there. Can you please do a review of one more package, this time using fedora-review as the starting point?