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

Failure when building with Sphinx 7.2 #1150

Closed
s3v- opened this issue Nov 22, 2023 · 4 comments
Closed

Failure when building with Sphinx 7.2 #1150

s3v- opened this issue Nov 22, 2023 · 4 comments

Comments

@s3v-
Copy link

s3v- commented Nov 22, 2023

Hi,
behave fails to build against Sphinx 7.2 as reported in Debian [1].

$ sphinx-build -b html behave-1.2.6/docs/ .
Running Sphinx v7.2.6
WARNING: html_static_path entry '_static' is placed inside outdir
loading intersphinx inventory from https://docs.python.org/3/objects.inv...
building [mo]: targets for 0 po files that are out of date
writing output... 
building [html]: targets for 26 source files that are out of date
updating environment: [new config] 26 added, 0 changed, 0 removed
reading sources... [  8%] api
Exception occurred:
  File "/usr/lib/python3/dist-packages/sphinx/ext/extlinks.py", line 108, in role
    title = caption % part
            ~~~~~~~~^~~~~~
TypeError: not all arguments converted during string formatting

Kind Regards

[1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1042610

@bittner
Copy link
Member

bittner commented Nov 22, 2023

Have to tried building the docs using Tox?

tox run -e docs

@s3v-
Copy link
Author

s3v- commented Nov 23, 2023

Have to tried building the docs using Tox?

tox run -e docs

Not yet, but I don't expect failures with that.
I see a constraint has been introduced in ddcaca4

# BEHAVE: PYTHON PACKAGE REQUIREMENTS: For documentation generation
# ============================================================================
# REQUIRES: pip >= 8.0
+ # AVOID: shponx v4.4.0 and newer -- Problems w/ new link check suggestion warnings

- sphinx >= 1.6
+ sphinx >=1.6,<4.4
sphinx-autobuild
sphinx_bootstrap_theme >= 0.6.0

Root problem is that a single "%s" is mandatory (since Sphinx 6) whether the caption is a string in "extlinks" config value.
Related commit

@bittner
Copy link
Member

bittner commented Jan 25, 2024

I think we can close this issue, can't we?

@bittner bittner closed this as completed Jan 25, 2024
@jenisys
Copy link
Member

jenisys commented May 12, 2024

sphinx == 7.3.7 (latest release) is now supported in behave.

INTEGRATED-INTO: behave v1.2.7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants