Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/show_bug.cgi?id=436114
Summary: Review Request: xdvipdfmx - An extended version of DVIPDFMx with support for XeTeX output Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: ndbecker2@gmail.com QAContact: extras-qa@fedoraproject.org CC: fedora-package-review@redhat.com,notting@redhat.com
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?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: xdvipdfmx - An extended version of DVIPDFMx with support for XeTeX output
https://bugzilla.redhat.com/show_bug.cgi?id=436114
pertusus@free.fr changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |pertusus@free.fr
------- Additional Comments From pertusus@free.fr 2008-03-05 09:43 EST ------- 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.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: xdvipdfmx - An extended version of DVIPDFMx with support for XeTeX output
https://bugzilla.redhat.com/show_bug.cgi?id=436114
------- Additional Comments From ndbecker2@gmail.com 2008-03-05 09:52 EST ------- mock build F9 OK
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: xdvipdfmx - An extended version of DVIPDFMx with support for XeTeX output
https://bugzilla.redhat.com/show_bug.cgi?id=436114
------- Additional Comments From pertusus@free.fr 2008-03-06 04:58 EST ------- 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.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: xdvipdfmx - An extended version of DVIPDFMx with support for XeTeX output
https://bugzilla.redhat.com/show_bug.cgi?id=436114
------- Additional Comments From ndbecker2@gmail.com 2008-03-06 08:00 EST ------- 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
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: xdvipdfmx - An extended version of DVIPDFMx with support for XeTeX output
https://bugzilla.redhat.com/show_bug.cgi?id=436114
pertusus@free.fr changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@fedoraproject.org |pertusus@free.fr Status|NEW |ASSIGNED Flag| |fedora-review+
------- Additional Comments From pertusus@free.fr 2008-03-06 14:55 EST ------- 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).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: xdvipdfmx - An extended version of DVIPDFMx with support for XeTeX output
https://bugzilla.redhat.com/show_bug.cgi?id=436114
ndbecker2@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
------- Additional Comments From ndbecker2@gmail.com 2008-03-06 18:33 EST ------- All above addressed: http://nbecker.dyndns.org:8080/RPM/xdvipdfmx-0.4-3.src.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: xdvipdfmx - An extended version of DVIPDFMx with support for XeTeX output
https://bugzilla.redhat.com/show_bug.cgi?id=436114
------- Additional Comments From ndbecker2@gmail.com 2008-03-06 18:39 EST ------- 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
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: xdvipdfmx - An extended version of DVIPDFMx with support for XeTeX output
https://bugzilla.redhat.com/show_bug.cgi?id=436114
------- Additional Comments From ndbecker2@gmail.com 2008-03-06 18:40 EST ------- Not sure how to setup co-maintainer, did I do this correctly?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: xdvipdfmx - An extended version of DVIPDFMx with support for XeTeX output
https://bugzilla.redhat.com/show_bug.cgi?id=436114
kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
------- Additional Comments From kevin@tummy.com 2008-03-06 19:25 EST ------- yes.
cvs done.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: xdvipdfmx - An extended version of DVIPDFMx with support for XeTeX output
https://bugzilla.redhat.com/show_bug.cgi?id=436114
------- Additional Comments From jonathan.underwood@gmail.com 2008-03-07 19:12 EST ------- Sorry, I totally missed this review BZ. Anyway, I'll request co-maintainership in pkgdb.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: xdvipdfmx - An extended version of DVIPDFMx with support for XeTeX output
https://bugzilla.redhat.com/show_bug.cgi?id=436114
------- Additional Comments From jonathan.underwood@gmail.com 2008-03-07 19:21 EST ------- Neal - since you've imported and built this package for Rawhide you should close this bug as NEXTRELEASE.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: xdvipdfmx - An extended version of DVIPDFMx with support for XeTeX output
https://bugzilla.redhat.com/show_bug.cgi?id=436114
ndbecker2@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
------- Additional Comments From ndbecker2@gmail.com 2008-03-07 19:22 EST ------- I don't plan to release for F7 and F8 for now, since requirements (texlive) are not in place.
package-review@lists.fedoraproject.org