Red Hat Bugzilla – Bug 436114
Review Request: xdvipdfmx - An extended version of DVIPDFMx with support for XeTeX output
Last modified: 2008-03-07 19:22:52 EST
Spec URL: http://nbecker.dyndns.org:8080/RPM/xdvipdfmx.spec
SRPM URL: http://nbecker.dyndns.org:8080/RPM/xdvipdfmx-0.4-1.src.rpm
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.
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
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.
mock build (f9) passed
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
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:
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:
New Package CVS Request
Package Name: xdvipdfmx
Short Description: An extended version of DVIPDFMx with support for XeTeX
Owners: nbecker, pertusus
Branches: F-7 F-8
Cvsextras Commits: yes
Not sure how to setup co-maintainer, did I do this correctly?
Sorry, I totally missed this review BZ. Anyway, I'll request co-maintainership
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.