Bug 2397821
| Summary: | Review Request: texlive-acrotex - The AcroTeX education bundle | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Tom "spot" Callaway <spotrh> | ||||
| Component: | Package Review | Assignee: | Miroslav Suchý <msuchy> | ||||
| Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||
| Severity: | medium | Docs Contact: | |||||
| Priority: | medium | ||||||
| Version: | rawhide | CC: | msuchy, package-review | ||||
| Target Milestone: | --- | Keywords: | AutomationTriaged | ||||
| Target Release: | --- | Flags: | msuchy:
fedora-review+
|
||||
| Hardware: | All | ||||||
| OS: | Linux | ||||||
| URL: | http://tug.org/texlive/ | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | Doc Type: | --- | |||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2026-02-07 13:55:16 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: | |||||||
| Attachments: |
|
||||||
|
Description
Tom "spot" Callaway
2025-09-24 14:35:04 UTC
Copr build: https://copr.fedorainfracloud.org/coprs/build/9597640 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2397821-texlive-acrotex/fedora-rawhide-x86_64/09597640-texlive-acrotex/fedora-review/review.txt Please take a look if any issues were found. --- This comment was created by the fedora-review-service https://github.com/FrostyX/fedora-review-service If you want to trigger a new Copr build, add a comment containing new Spec and SRPM URLs or [fedora-review-service-build] string. You may want to fix
texlive-acrotex.spec:18: W: mixed-use-of-spaces-and-tabs (spaces: line 8, tab: line 18)
You can document how you get:
Source2: texlive-licenses.tar.xz
Ad:
tar -xf %{SOURCE0} -C %{buildroot}%{_texmf_main}
tar -xf %{SOURCE1} -C %{buildroot}%{_texmf_main}
move the extraction to %prep and in %install do just `cp -a`.
Big %docs:
texlive-acrotex.noarch: W: package-with-huge-docs 87%
you may consider doc subpackage as the package has almost 3MB.
I will move the extraction for these smaller packages, but there is a significant buildtime cost (and added spec complexity) to copying the tex components twice (once in %build, again in %install) when building the larger collection packages, and I do not think it is worth changing them there. To your point on the docs... in the past, there were -doc subpackages for most of the texlive component packages and it scaled poorly. rpm has mechanisms to install without files marked as %doc and that is a far better solution for most texlive components (and it makes the packaging so much cleaner). Only in egregious cases where the docs are massive (texlive-context-doc) do I force a -doc subpackage. New SPEC: https://spot.fedorapeople.org/tl2025/texlive-acrotex.spec New SRPM: https://spot.fedorapeople.org/tl2025/texlive-acrotex-svn330-2.fc44.src.rpm Created attachment 2108239 [details]
The .spec file difference from Copr build 9597640 to 9636927
Copr build: https://copr.fedorainfracloud.org/coprs/build/9636927 (succeeded) Review template: https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2397821-texlive-acrotex/fedora-rawhide-x86_64/09636927-texlive-acrotex/fedora-review/review.txt Please take a look if any issues were found. --- This comment was created by the fedora-review-service https://github.com/FrostyX/fedora-review-service If you want to trigger a new Copr build, add a comment containing new Spec and SRPM URLs or [fedora-review-service-build] string. APPROVED Miroslav, can you flip the review flag on and off here, the tooling says "Could not execute request_repo: The Bugzilla bug's review was approved over 60 days ago", which I did not know was a thing. :/ updated The Pagure repository was created at https://src.fedoraproject.org/rpms/texlive-acrotex |