Bug 436114
Summary: | Review Request: xdvipdfmx - An extended version of DVIPDFMx with support for XeTeX output | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Neal Becker <ndbecker2> |
Component: | Package Review | Assignee: | Patrice Dumas <pertusus> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-package-review, jonathan.underwood, notting, pertusus |
Target Milestone: | --- | Flags: | pertusus:
fedora-review+
kevin: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2008-03-08 00:22:52 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Neal Becker
2008-03-05 14:22:53 UTC
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. |