Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

gh-139588: Docs: PDF/LaTeX: bump max list depth#140709

Merged
AA-Turner merged 3 commits intopython:mainfrom
m-aciek:docs/pdf/fix-too-deeply-nested/maxlistdepth
Oct 28, 2025
Merged

gh-139588: Docs: PDF/LaTeX: bump max list depth#140709
AA-Turner merged 3 commits intopython:mainfrom
m-aciek:docs/pdf/fix-too-deeply-nested/maxlistdepth

Conversation

@m-aciek
Copy link
Contributor

@m-aciekm-aciek commentedOct 28, 2025
edited
Loading

Prevents LaTeX PDF build errorToo deeply nested. (Regression from#135945.)

Thanks@jfbu for thefix suggestion.

Zrzut ekranu 2025-10-28 o 15 45 51

Better alternative to#140445.

Could it be labeled for backport on 3.13-3.15 please?


📚 Documentation preview 📚:https://cpython-previews--140709.org.readthedocs.build/

Prevents LaTeX PDF build error `Too deeply nested`. (Regression frompython#139509.)
Prevents LaTeX PDF build error `Too deeply nested`. (Regression frompython#139509.)
@AA-Turner
Copy link
Member

Thanks, I've also increased it to 8 per Jean-François' suggestion.

A

m-aciek reacted with thumbs up emoji

@AA-TurnerAA-Turner merged commitce4b0ed intopython:mainOct 28, 2025
32 checks passed
@miss-islington-app
Copy link

Thanks@m-aciek for the PR, and@AA-Turner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull requestOct 28, 2025
(cherry picked from commitce4b0ed)Co-authored-by: Maciej Olko <maciej.olko@affirm.com>Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull requestOct 28, 2025
(cherry picked from commitce4b0ed)Co-authored-by: Maciej Olko <maciej.olko@affirm.com>Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
@bedevere-app
Copy link

GH-140724 is a backport of this pull request to the3.14 branch.

@bedevere-appbedevere-appbot removed the needs backport to 3.14bugs and security fixes labelOct 28, 2025
@bedevere-app
Copy link

GH-140725 is a backport of this pull request to the3.13 branch.

@bedevere-appbedevere-appbot removed the needs backport to 3.13bugs and security fixes labelOct 28, 2025
AA-Turner added a commit that referenced this pull requestOct 28, 2025
)Co-authored-by: Maciej Olko <maciej.olko@affirm.com>Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
AA-Turner added a commit that referenced this pull requestOct 28, 2025
)Co-authored-by: Maciej Olko <maciej.olko@affirm.com>Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
@m-aciekm-aciek deleted the docs/pdf/fix-too-deeply-nested/maxlistdepth branchOctober 28, 2025 20:20
StanFromIreland pushed a commit to StanFromIreland/cpython that referenced this pull requestDec 6, 2025
Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
@m-aciek
Copy link
ContributorAuthor

FYI I discovered yesterday that this PR didn't fully fixed the PDF build issue. Nowmake all-pdf command in latex build directory still returns non-zero exit code and prevents next PDF file (whatsnew.pdf) to be build when there's no flag--keep-going used, despite the fact that the PDF is rendered correctly.

The error frommake all-pdf can be silenced with:

Index: Doc/conf.pyIDEA additional info:Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP<+>UTF-8===================================================================diff --git a/Doc/conf.py b/Doc/conf.py--- a/Doc/conf.py(revision 8255901127193a0b5e82034e730f7749b68c460f)+++ b/Doc/conf.py(revision a93073936f4ea1cd3fa62e0a478b1b3aa1f40a98)@@ -355,6 +355,7 @@ \let\Verbatim=\OriginalVerbatim \let\endVerbatim=\endOriginalVerbatim \setcounter{tocdepth}{2}+\makeatletter\let\@toodeep\relax\makeatother ''',     # The paper size ('letterpaper' or 'a4paper').     'papersize': 'a4paper',

I plan to prepare a follow-up PR soon.

@jfbu
Copy link
Contributor

jfbu commentedMar 4, 2026 via email

Hi, I guess setting `'maxlistdepth'` to a higher value `'10'` or `'12'` did not help? It would be nice to fetch the actual reported LaTeX error for analysis, if at all possible.

@m-aciek
Copy link
ContributorAuthor

Hi, I guess setting'maxlistdepth' to a higher value'10' or'12' did not help?

I will try it later.

It would be nice to fetch the actual reported LaTeX error for analysis, if at all possible.

The logs are available here:https://github.com/m-aciek/python-docs-offline/actions/runs/22630330823. There ispdf-logs.zip artifact with.tex and.log files.

@jfbu
Copy link
Contributor

jfbu commentedMar 4, 2026

Inreference.log:

[73]! LaTeX Error: Too deeply nested.See the LaTeX manual or LaTeX Companion for explanation.Type  H <return>  for immediate help. ...                                                                                                l.8582 \begin{DUlineblock}{\DUlineblockindent}                                              You're in trouble here.  Try typing  <return>  to proceed.If that doesn't work, type  X <return>  to quit.[74] [75] [76

Try increasing the setting of'maxlistdepth'.

@m-aciek
Copy link
ContributorAuthor

I am sorry for confusion, I was wrong! I was working on an old checkout from repository 🤦 and didn't have above changes locally. Sorry for the noise, and thank you for the help@jfbu!

@jfbu
Copy link
Contributor

jfbu commentedMar 4, 2026 via email

no problem, glad you could figure it out!

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@AA-TurnerAA-TurnerAA-Turner approved these changes

@hugovkhugovkAwaiting requested review from hugovkhugovk is a code owner

+1 more reviewer

@jfbujfbujfbu left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@m-aciek@AA-Turner@jfbu@StanFromIreland

[8]ページ先頭

©2009-2026 Movatter.jp