Spec URL: http://www.cs.technion.ac.il/~danken/texlive-fonts-hebrew.spec SRPM URL: http://www.cs.technion.ac.il/~danken/texlive-fonts-hebrew-0.1-9.fc9.src.rpm Description: Support using the Culmus Hebrew fonts in TeXLive! (minor conversion of tetex-fonts-hebrew)
The naming of tex packages is now tex-fonts-hebrew
Thanks, but why do http://download.fedora.redhat.com/pub/fedora/linux/development/x86_64/os/Packages/texlive-2007-31.fc10.x86_64.rpm and its likes suggest otherwise?
Well, texlive is itself a distribution and has many subpackages, but this is not a texlive subpackage. Previously Fedora used tetex, and thus you will see many tetex-* package still in the distribution, but due to the change we prefer to have a tex-distribution-neutral naming scheme; hence, tex-fonts-hebrew. See http://fedoraproject.org/wiki/Packaging/Minutes20071002 for the packaging committee decision. Unfortunately I think this fell through the cracks and wasn't ever written into the guidelines anywhere; if you'd prefer to to wait until that is done and I can refer you directly to the guidelines, I'd understand. Anyway, this package builds OK; rpmlint has the following complaints: W: file-not-utf8 /usr/share/doc/texlive-fonts-hebrew-0.1/hebhello.tex Normally we fix these up, but I'm not sure what to do with tex files. W: dangling-relative-symlink /usr/share/texmf/fonts/type1/public/culmus ../../../../fonts/culmus W: dangling-relative-symlink /usr/share/texmf/fonts/afm/public/culmus ../../../../fonts/culmus It's OK to have links to files in other packages as long as they're pulled in via dependencies, which is the case here since fonts-hebrew, a dependency, will pull in culmus-fonts. W: invalid-license GPL Please use an appropriate License: tag from http://fedoraproject.org/wiki/Licensing. W: obsolete-not-provided tetex-fonts-hebrew If you want to replace the existing tetex-fonts-hebrew package, you need to provide a proper upgrade path. See http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Renaming.2Freplacing_existing_packages for more information.
Thanks for the quick review. I think I answer your comments in http://www.cs.technion.ac.il/~danken/tex-fonts-hebrew-0.1-9.fc9.src.rpm and http://www.cs.technion.ac.il/~danken/tex-fonts-hebrew.spec Please tell me if you find somthing to the contrary.
Sorry for taking so long; I was distracted by another project. At least I remember why this seems so familiar; I reviewed tetex-fonts-hebrew a couple of years ago. We have a few new guidelines since then, and I might have missed a couple of things, so there are a few issues even though this package is mostly the same. rpmlint is down to the file-not-utf8 and two dangling-relative-symlink complaints. I think these are all OK. I know you're maintaining the tarball yourself; we have a guideline about that now. Please see http://fedoraproject.org/wiki/Packaging/SourceURL (the "We are Upstream" section). I'm not sure the License: is correct. There is a GPL notice, but as far as I can tell, nothing in the package specifies a version, so by the language in the GPL notice any GPL version is acceptable (and hence the License: tag should contain GPL+). Also, the file "hebrew.ldf" seems to be under the LPPL. If that's correct then it seems to me that you should have: License: GPL+ and LPPL with a comment before that indicating which parts are under which license. But I could be missing something. The Summary isn't actually correct on F9 at least, because teTeX isn't in use there. The /usr/share/texmf/fonts/enc/dvips/culmus directory seems to be unowned. X source files can't be compared with upstream sources. * package meets naming and versioning guidelines. * specfile is properly named, is cleanly written and uses macros consistently. X summary shouldn't mention tetex. * description is OK. * dist tag is present. * build root is OK. X license field doesn't seem to match the actual license. * license is open source-compatible. * license text included in package. * BuildRequires are proper. * %clean is present. * package builds in mock (rawhide, x86_64). * package installs properly. * rpmlint has acceptable complaints. * final provides and requires are sane: tetex-fonts-hebrew = 0.1-9.fc10 tex-fonts-hebrew = 0.1-9.fc10 = /bin/sh /usr/bin/texhash /usr/bin/updmap-sys fonts-hebrew texlive X fails to own /usr/share/texmf/fonts/enc/dvips/culmus * doesn't own any directories it shouldn't. * no duplicates in %files. * file permissions are appropriate. * scriptlets are OK. * code, not content. * documentation is small, so no -doc subpackage is necessary. * %docs are not necessary for the proper functioning of the package.
Don't worry about small delays; now that I have this package working on my own machine I'm in no hurry. I hope that I answered all your comments. (mainly added comments to the spec...) Please review the same URLs again.
Things look good to me: /usr/share/texmf/fonts/enc/dvips/culmus is owned properly. The License: looks good. Summary looks good. Lack of upstream sources is commented properly. APPROVED
New Package CVS Request ======================= Package Name: tex-fonts-hebrew Short Description: Support using (Culmus) Hebrew fonts in TeXLive Owners: danken Branches: F-9 InitialCC: danken Cvsextras Commits: yes
CVS done.
tex-fonts-hebrew-0.1-9.fc9 has been submitted as an update for Fedora 9
tex-fonts-hebrew-0.1-9.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report.
Please don't leave review tickets open after they're done — thanks