Skip to content

Conversation

@AA-Turner
Copy link
Member

@AA-Turner AA-Turner requested a review from hugovk October 21, 2025 11:31
@hugovk
Copy link
Member

hugovk commented Oct 21, 2025

.github/workflows/build-release.yml also runs make dist, and there's a bunch of latex dependencies we could skip.

And run_release.py has a couple of lines where it waits for PDF files.

(Not important: tests/README.rst mentions "PDF". This is a copy of https://github.com/python/cpython/blob/main/README.rst for testing version bumps, but we can replace the upstream "PDF" with EPUB" and might as well here too.)

@AA-Turner
Copy link
Member Author

AA-Turner commented Oct 21, 2025

Forgot to look in .github/, thanks!

A

@hugovk
Copy link
Member

hugovk commented Oct 21, 2025

And can we skip some/all of these in .github/workflows/build-release.yml?

          sudo apt-get install --yes --no-install-recommends \
            fonts-freefont-otf \
            latexmk \
            texinfo \
            texlive-fonts-extra \
            texlive-fonts-recommended \
            texlive-latex-base \
            texlive-latex-extra \
            texlive-latex-recommended \
            texlive-xetex \
            xindy

Copy link
Member

@hugovk hugovk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

build-docs is now down to 5m 25s.

Previously it took 17m 31s, 19m 26s, or even 42m 12s when the now-removed apt-get install took half an hour :)

@AA-Turner AA-Turner merged commit a8981b1 into main Oct 21, 2025
25 checks passed
@AA-Turner AA-Turner deleted the docs-no-pdf branch October 21, 2025 14:13
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.

3 participants