Skip to content

Docs: EPUB builds are broken #136155

Closed
Closed

Description

@m-aciek

Documentation

#132220 has broken EPUB builds of documentation. This has been only partially fixed in #133720, because yet configuration of sphinxext-opengraph adds not closed tags to every page.

Image

sphinx-doc/sphinxext-opengraph#136 and sphinx-doc/sphinxext-opengraph#137 have been opened upstream, but the PR is not merged.

Status update [2025-07-14]: now (after skipping OpenGraph custom tags in Docs conf.py) only 8 chapters fail to render in EPUB format. It can be addressed by any of sphinx-doc/sphinxext-opengraph#120 and sphinx-doc/sphinxext-opengraph#137, as far as I'm concerned. Adding a CI check definitely should help with the stability of EPUB build. Last year Inada made an effort to fix the EPUB (#118689).

Linked PRs

Activity

  1. added
    triagedThe issue has been accepted as valid by a triager.
    on Jun 30, 2025
  2. hugovk commented on Jul 1, 2025

    @hugovk
    Member

    Would you like to open a PR with your tags suggestion from #134071 (comment)?

    Otherwise, let's do #134071.

  3. added a commit that references this issue on Jul 1, 2025
  4. m-aciek commented on Jul 1, 2025

    @m-aciek
    ContributorAuthor

    I've figured out the tags approach and confirmed it works locally: #136187.

  5. added a commit that references this issue on Jul 7, 2025
  6. added 2 commits that reference this issue on Jul 7, 2025
  7. hugovk commented on Jul 8, 2025

    @hugovk
    Member

    #136187 has improved it but not fixed them all: #136187 (comment)

    They are:

    FATAL - RSC-016 - Python.epub/howto/descriptor.xhtml:11:78 - Fatal Error while parsing file: The value of attribute "content" associated with an element type "meta" must not contain the '<' character.
    FATAL - RSC-016 - Python.epub/howto/logging-cookbook.xhtml:11:62 - Fatal Error while parsing file: The value of attribute "content" associated with an element type "meta" must not contain the '<' character.
    FATAL - RSC-016 - Python.epub/howto/logging.xhtml:11:62 - Fatal Error while parsing file: The value of attribute "content" associated with an element type "meta" must not contain the '<' character.
    FATAL - RSC-016 - Python.epub/howto/regex.xhtml:11:64 - Fatal Error while parsing file: The value of attribute "content" associated with an element type "meta" must not contain the '<' character.
    FATAL - RSC-016 - Python.epub/whatsnew/3.4.xhtml:11:66 - Fatal Error while parsing file: The value of attribute "content" associated with an element type "meta" must not contain the '<' character.
    FATAL - RSC-016 - Python.epub/whatsnew/3.5.xhtml:11:71 - Fatal Error while parsing file: The value of attribute "content" associated with an element type "meta" must not contain the '<' character.
    FATAL - RSC-016 - Python.epub/whatsnew/3.6.xhtml:11:71 - Fatal Error while parsing file: The value of attribute "content" associated with an element type "meta" must not contain the '<' character.
    FATAL - RSC-016 - Python.epub/whatsnew/3.7.xhtml:11:70 - Fatal Error while parsing file: The value of attribute "content" associated with an element type "meta" must not contain the '<' character.
    

    Here's the first couple, the rest are also all similar email address tags:

    <meta property="og:description" content="Author, Raymond Hettinger,, Contact,<python at rcn dot com>,. Contents: Descriptor Guide- Primer- Simple example: A descriptor that returns a constant, Dynamic lookups, Managed attributes, Customiz..." />
    <meta property="og:description" content="Author, Vinay Sajip <vinay_sajip at red-dove dot com>,. This page contains a number of recipes related to logging, which have been found useful in the past. For links to tutorial and reference info..." />

    (macOS tip: opening the EPUB in BBEdit gives a folder-based UI and you can see the plaintext xhtml files inside.)

  8. added 3 commits that reference this issue on Jul 8, 2025
  9. added a commit that references this issue on Jul 11, 2025
  10. 39 remaining items

  11. added a commit that references this issue on Sep 10, 2025
  12. hugovk commented on Sep 10, 2025

    @hugovk
    Member

    There's still a problem with the 3.12 EPUB: #136406 (comment)

    Here's an extra backport to fix it: #138741

  13. added a commit that references this issue on Sep 24, 2025
  14. m-aciek commented on Sep 25, 2025

    @m-aciek
    ContributorAuthor

    With 3.12 fixed, let's close

  15. hugovk commented on Sep 30, 2025

    @hugovk
    Member

    Do we also need to fix the other security branches: 3.9-3.11?

  16. AA-Turner commented on Sep 30, 2025

    @AA-Turner
    Member

    #132220 wasn't backported that far, so there shouldn't've been any changes on our side, it would only be if a bad opengraph version is still there.

  17. hugovk commented on Sep 30, 2025

    @hugovk
    Member
  18. hugovk commented on Oct 23, 2025

    @hugovk
    Member

    Looking good now, I checked 3.9, 3.11 and 3.14 🚀

  19. added a commit that references this issue on Mar 3, 2026
  20. added a commit that references this issue on Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    3.11only security fixesdocsDocumentation in the Doc dirtriagedThe issue has been accepted as valid by a triager.

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions