Spec URL: http://nbecker.dyndns.org:8080/RPM/xdvipdfmx.spec SRPM URL: http://nbecker.dyndns.org:8080/RPM/xdvipdfmx-0.4-1.src.rpm Description: xdvipdfmx is an output driver for the XeTeX typesetting system. It is an extended version of DVIPDFMx by Jin-Hwan Cho and Shunsaku Hirata, which is itself an extended version of dvipdfm by Mark A. Wicks. This driver converts XDV (extended DVI) output from the xetex program into standard PDF that can be viewed or printed. rpmlint OK mock: building on f9 now. For f7/f8, what is used to BR kpathsea-devel?
kpathsea static lib is in tetex-fonts in F7/F8. But I am not sure that it is worth shipping xdvipdfmx for F7/F8 since there is no xetex/xelatex in tetex. It is already in the texlive for F8 from Jindrich. So in my opinion it should only be in devel.
mock build F9 OK
The Vendor tag should be dropped. The requires tetex should better be tex(tex). The Requires fontconfig should be dropped, it is autodetected. A BuildRequires libpaper-devel is missing. Missing something along: %doc README AUTHORS BUGS COPYING TODO doc/tug2003.pdf index.html *.css The data files and configuration is taken from dvipdfmx, therefore a Requires: dvipdfmx is missing.
All above fixed. rpmlint silent mock build (f9) passed http://nbecker.dyndns.org:8080/RPM/xdvipdfmx.spec http://nbecker.dyndns.org:8080/RPM/xdvipdfmx-0.4-2.src.rpm
There is a typo, the Requires is tex(tex) not te(tex). I think that it would be nice to have a comment for the dvipdfmx requires. Also I think that there should be a %dist tag. The source archive timestamp cannot be kept looks like it changes at each download. The source match: 2bb9d65f2406a112fa53dbd0697d93ad xdvipdfmx-0.4.tar.gz If the typo is fixed, and the %dist tag is added, it is APPROVED. I can be co-maintainer and I think that Jindrich and Jonathan should also be co-maintainer (if they agree to).
All above addressed: http://nbecker.dyndns.org:8080/RPM/xdvipdfmx-0.4-3.src.rpm
New Package CVS Request ======================= Package Name: xdvipdfmx Short Description: An extended version of DVIPDFMx with support for XeTeX output Owners: nbecker, pertusus Branches: F-7 F-8 InitialCC: Cvsextras Commits: yes
Not sure how to setup co-maintainer, did I do this correctly?
yes. cvs done.
Sorry, I totally missed this review BZ. Anyway, I'll request co-maintainership in pkgdb.
Neal - since you've imported and built this package for Rawhide you should close this bug as NEXTRELEASE.
I don't plan to release for F7 and F8 for now, since requirements (texlive) are not in place.