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

v0.5.0: Could not import extension sphinxarg.ext (exception: cannot import name 'get_default_settings' from 'docutils.frontend' #57

Closed
fmigneault opened this issue Jul 15, 2024 · 2 comments

Comments

@fmigneault
Copy link

https://github.com/crim-ca/weaver/actions/runs/9948054780/job/27481903248

Extension error:
Could not import extension sphinxarg.ext (exception: cannot import name 'get_default_settings' from 'docutils.frontend' (/opt/hostedtoolcache/Python/3.10.14/x64/lib/python3.10/site-packages/docutils/frontend.py))

with

docutils==0.18.1
Sphinx==7.3.7
sphinx-argparse==0.5.0
sphinx-autoapi==3.1.2
sphinx-autodoc-typehints==2.2.2
sphinx-paramlinks==0.6.0
sphinx-rtd-theme==1.3.0
sphinxcontrib-applehelp==1.0.8
sphinxcontrib-devhelp==1.0.6
sphinxcontrib-htmlhelp==2.0.5
sphinxcontrib-jquery==4.1
sphinxcontrib-jsmath==1.0.1
sphinxcontrib-qthelp==1.0.7
sphinxcontrib-redoc==1.6.0
sphinxcontrib-serializinghtml==1.1.10

The docutils<0.19 is enforced by an older version from sphinx-rtd-theme==1.3.0.
Since sphinx-rtd-theme==2.0.0 was just released, I expect some flaky compatibility with many of those cross-dependencies. I must pin older versions until everything is resolved everywhere.
Therefore, sphinx-argparse<0.5.0 must also be set in the meantime, unless older docutils get handled accordingly.

@AA-Turner
Copy link
Member

Ahh -- the function was added in Docutils 0.19, not 0.18 as they document. Fixed in a1621bc.

A

@AA-Turner
Copy link
Member

I have released sphinx-argparse v0.5.1 with a fix.

A

tomkinsc added a commit to broadinstitute/viral-pipelines that referenced this issue Jul 30, 2024
bump `sphinx-rtd-theme` in requirements for docs build, pin min version of `sphinx-argparse` in attempt to resolve build failurea similar to the one described in this issue: sphinx-doc/sphinx-argparse#57
tomkinsc added a commit to broadinstitute/viral-pipelines that referenced this issue Jul 31, 2024
* add build environment spec to .readthedocs.yml config

a build.os value is now required by readthedocs; see: https://blog.readthedocs.com/use-build-os-config/

* Update requirements.txt

bump `sphinx-rtd-theme` in requirements for docs build, pin min version of `sphinx-argparse` in attempt to resolve build failurea similar to the one described in this issue: sphinx-doc/sphinx-argparse#57

* allow newer sphinx versions <v8

* adjust doc build requirements to pin to particular versions; specify `--jobs auto` for parallel sphinx build jobs

* remove miniwdl from docs build reqs
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

No branches or pull requests

2 participants