Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adapt to upcoming Sphinx==4.0 release #71

Merged
merged 1 commit into from
May 31, 2021
Merged

Conversation

hoefling
Copy link
Contributor

@hoefling hoefling commented May 4, 2021

Close #72.

sphinx.util.osutil.ENOENT is deprecated since Sphinx==2.0 and is removed in the upcoming 4.0 release, breaking compatibility. Since Python 2 is not supported by Sphinx anymore anyway, FileNotFoundError is used instead. Other than that, I have adjusted tests to work with 4.0.

@mgaitan if you approve the PR, can you publish the next release early enough so we are prepared for the upcoming 4.0 release on 8th May?

Signed-off-by: oleg.hoefling oleg.hoefling@gmail.com

Signed-off-by: oleg.hoefling <oleg.hoefling@gmail.com>
@AnthonyDiGirolamo
Copy link

I'm hitting this as well, any chance this can be merged?

copybara-service bot pushed a commit to google/pigweed that referenced this pull request May 10, 2021
Version >=4.0.0 of Sphinx removes some APIs, one of which is used by
sphinxcontrib-mermaid. Error:

Running Sphinx v4.0.0
Extension error:
Could not import extension sphinxcontrib.mermaid (exception: cannot import name
'ENOENT' from 'sphinx.util.osutil'

Failed roll:
https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/44223

This workaround can be removed after
mgaitan/sphinxcontrib-mermaid#71 is merged.

No-Docs-Update-Reason: Sphinx 4.0 workaround.
Change-Id: I63e192428dda4efa228dea892ef857c782ec5674
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/44320
Commit-Queue: Anthony DiGirolamo <tonymd@google.com>
Pigweed-Auto-Submit: Anthony DiGirolamo <tonymd@google.com>
Reviewed-by: Rob Mohr <mohrr@google.com>
@thepabloaguilar
Copy link

Hey, I have the same issue too!

@mgaitan Can you help us, please?!

@mgaitan mgaitan merged commit cb44181 into mgaitan:master May 31, 2021
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.

some APIs are removed in Sphinx 4.0 version
4 participants