Red Hat Bugzilla – Bug 173897
Review Request: fonts-hebrew-fancy
Last modified: 2009-03-02 18:43:20 EST
Spec Name or Url: http://ivrix.org.il/redhat/fonts-hebrew-fancy.spec
SRPM Name or Url: http://ivrix.org.il/redhat/fonts-hebrew-fancy-0.101-1.src.rpm
Description: The fonts-hebrew-fancy package contains fancy (non-standard) Hebrew fonts from the Culmus project by Maxim Iorsh.
The core package fonts-hebrew include only the bare minimum fonts. For fancier fonts, such as Ktav-Yad (cursive), install fonts-hebrew-fancy.
As is bug 173683, this is an attempt to make Fedora more useful for Hebrew speakers.
Thank you for packaging these - I've been looking for good Hebrew fonts (I'm
learning it) - found a few, and found some expensive nice commercial ones.
Anyway, these look nice.
Back on topic - some suggestions:
1) Please use a full url path to the tarballs - IE
2) Please install them into their own directory, rather than a directory owned
by a core font package.
That will also remove the need for explicit dependency on hebrew-fonts
3) FC-5 uses fonts.cache-2 - so its easiest to touch and ghost both
Then in the %files section - ghost both of them.
I *think* that is incorrect - may work in fc4, but I believe it doesn't in fc5.
I believe in fc5 you have to specify the directory. IE
The %postun is I believe correct.
Please change that to
A Hebrew summary and description would be nice, but not required.
Good luck with your Hebrew studies; you might find my verb conjugator
I believe that Maxim Iorsh, who drew the fonts, deserves all your thanks.
I did what you asked, and hope it's ok (no idea what is this %ghost...
should browse MaximumRPM).
Do note that I copied the %post script from FC4 fonts-hebrew package.
One should make sure it is correct in FC5.
.spec and SRPM are updated
Formal review coming (including fc5 test)
What %ghost does is mark a file such that it is not actually installed by the
rpm but is owned by the rpm if it exists.
This allows the proper program (in this case) fc-cache to be responsible for
creating the file, but it is owned by the rpm - so that when the rpm is removed,
the ghosted file is removed as if the rpm had been who put it there in the
I don't know if Maximum rpm talks about it or not. I think the Red Hat RPM Guide
(which I believe either is or shortly will be available for free) does.
Reviewed per http://fedoraproject.org/wiki/PackageReviewGuidelines
* rpmlint clean
* Package name appropriate
- upstream distributes as a series of tarballs, but the source of these
- fonts are grouped together as fancy
- These fonts are from the same project as the Core package fonts-hebrew
* Spec file name matches src.rpm name
* Package meets packaging guidelines
* Specified License (GPL) matches project page and license provided with source
* License in %doc
* Spec file written in American English, Legible
* Sources match upstream md5sum
* spec file works
* No BuildRequires (thus none from exceptions or redundant)
* No locale files
* no shared libraries
* No use of Prefix:
* No duplicates
* Appropriate permissions
* Proper %clean
* consistent use of macros
* contains permisable content
* No need for separate docs package
* files in %doc not necessary for runtime
* No devel/library/etc to worry about
* Contains summary/description for Hebrew
* Sane scriptlets
* Works in FC4 and FC5
Package Change Request
Package Name: culmus-fancy-fonts
according to bug 477387 I need (among other things) to rename fonts-hebrew-fancy to culmus-fancy-fonts. Is this the place to ask for that (and inclusion of the new name in F11)?
No. You need to submit a new review request for the newly named package. ;)
Once it's approved and added, you can go back and mark the current name dead.