Bug 2397826 - Review Request: texlive-mathtime-ltx - LaTeX macros for using MathTime and MathTime Plus
Summary: Review Request: texlive-mathtime-ltx - LaTeX macros for using MathTime and Ma...
Keywords:
Status: NEW
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Miroslav Suchý
QA Contact: Fedora Extras Quality Assurance
URL: http://tug.org/texlive/
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2025-09-24 14:52 UTC by Tom "spot" Callaway
Modified: 2025-10-01 18:51 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed:
Type: ---
Embargoed:
msuchy: fedora-review+


Attachments (Terms of Use)
The .spec file difference from Copr build 9597677 to 9636887 (2.14 KB, patch)
2025-10-01 15:33 UTC, Fedora Review Service
no flags Details | Diff

Description Tom "spot" Callaway 2025-09-24 14:52:27 UTC
Spec URL: https://spot.fedorapeople.org/tl2025/texlive-mathtime-ltx.spec
SRPM URL: https://spot.fedorapeople.org/tl2025/texlive-mathtime-ltx-svn362-1.fc44.src.rpm
Description: 
LaTeX macros for using MathTime and MathTime Plus.
Fedora Account System Username: spot

This package is part of an effort to modularize and more completely package TeXLive in Fedora, starting with TL2025. A copr with F43 and Rawhide builds for this package (and the rest of the package set) can be found here:

https://copr.fedorainfracloud.org/coprs/spot/texlive-2025/

The use of a high epoch is intentional, this is the legacy of TeXLive, and it ensures consistency in dependencies and forcing upgrades.

The "mathtime-ltx" component is not part of "proper" TeXLive, it lives in the tlcontrib space, but components within "proper" TeXLive depend on it, so I've packaged it as a standalone package (along with "acrotex" and "eq-save").

Comment 1 Fedora Review Service 2025-09-24 14:56:01 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/9597677
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2397826-texlive-mathtime-ltx/fedora-rawhide-x86_64/09597677-texlive-mathtime-ltx/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.

Comment 2 Miroslav Suchý 2025-09-24 21:29:40 UTC
You may want to fix:
texlive-mathtime-ltx.spec:18: W: mixed-use-of-spaces-and-tabs (spaces: line 8, tab: line 18)

You may document in a comment how you created `Source2:        texlive-licenses.tar.xz`

The above is not a blocker. But the below is a blocker:

> %install
> tar -xf %{SOURCE0} -C %{buildroot}%{_texmf_main}
> tar -xf %{SOURCE1} -C %{buildroot}%{_texmf_main}

Please move the extraction to %prep and %install do just `cp -a` of these files.

Comment 3 Tom "spot" Callaway 2025-10-01 15:19:03 UTC
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.

New SPEC: https://spot.fedorapeople.org/tl2025/texlive-mathtime-ltx.spec
New SRPM: https://spot.fedorapeople.org/tl2025/texlive-mathtime-ltx-svn362-2.fc44.src.rpm

Comment 4 Fedora Review Service 2025-10-01 15:33:42 UTC
Created attachment 2108237 [details]
The .spec file difference from Copr build 9597677 to 9636887

Comment 5 Fedora Review Service 2025-10-01 15:33:44 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/9636887
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2397826-texlive-mathtime-ltx/fedora-rawhide-x86_64/09636887-texlive-mathtime-ltx/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.

Comment 6 Miroslav Suchý 2025-10-01 18:51:03 UTC
APPROVED


Note You need to log in before you can comment on or make changes to this bug.