Skip to content

Unvendor packages to fix CI#1187

Merged
mikeshulman merged 1 commit intoHoTT:masterfrom
danielzgtg:fix/ubuntu2504
Jul 23, 2025
Merged

Unvendor packages to fix CI#1187
mikeshulman merged 1 commit intoHoTT:masterfrom
danielzgtg:fix/ubuntu2504

Conversation

@danielzgtg
Copy link
Contributor

@danielzgtg danielzgtg commented Jul 21, 2025

Something related to "fixcleverref" is incompatible with texlive-full=2024.20250309-1 on CI or Ubuntu 25.04. The CLI had "Latex failed to resolve 216 citation(s)". The end of the introduction had "??" instead of links, and the word "section" in unstyled text before headings.

Unvendoring all the (now-) unnecessarily vendored local packages fixes the problem. I enabled -g for latexmk to fix the .pdf not being rebuilt when I delete it.

Before

?? and other breakage everywhere

After

links work

Fixes: #1172
While the workflow awaits approval, a passing run can be seen at https://github.com/danielzgtg/hott/actions/runs/16428214132/job/46423784525

Something related to "fixcleverref" is incompatible with
texlive-full=2024.20250309-1 on CI or Ubuntu 25.04. The CLI had
"Latex failed to resolve 216 citation(s)". The end of the introduction
had "??" instead of links, and the word "section" in unstyled text
before headings.

Unvendoring all the (now-) unnecessarily vendored local
packages fixes the problem. I enabled `-g` for `latexmk` to fix the .pdf
not being rebuilt when I delete it.

Fixes: HoTT#1172
@mikeshulman
Copy link
Contributor

I don't remember why those packages were vendored in the first place.

@danielzgtg
Copy link
Contributor Author

a9eb3bc added some because "The IAS machines and several other people seem to have old geometry packages installed."

Well, that was 12 years ago. The solutions are either to simply unvendor the packages whereby the breakages on CI and modern machines are fixed, or to have someone spend a lot of time finding the correct version of peer dependencies to also vendor.

@mikeshulman
Copy link
Contributor

Thanks for the sleuthing!

@mikeshulman mikeshulman merged commit c4e57d5 into HoTT:master Jul 23, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI workflow broken

2 participants