Thanks to visit codestin.com
Credit goes to github.com

Skip to content

3.13 build comments #3300

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.

Sign up for GitHub

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

Merged
merged 3 commits into from
Nov 22, 2024
Merged

3.13 build comments #3300

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
9a478e3
Clarify `make build -j` double red herring
rtobar Nov 22, 2024
526beb4
Use sphinx~=8.0.0 as intended
rtobar Nov 22, 2024
e55d799
Remove unnecessary conf.py overrides
rtobar Nov 22, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Use sphinx~=8.0.0 as intended 
8.1 introduces the built-in cvw/cwe roles. During the migration the
requirement was bumped to 8.1 while trying to sort out a problem with
the pypi role, which turned out to be the cpython submodule not being
committed/pushed properly earlier, and hence the update wasn't necessary
(and broke things).

Signed-off-by: Rodrigo Tobar <[email protected]>
  • Loading branch information
@rtobar
rtobar committed Nov 22, 2024
commit 526beb47c1b543738c84c1ba379250c4f8aa6a1a
9 changes: 0 additions & 9 deletions conf.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,15 +29,6 @@
sys.path.insert(0, os.path.abspath('cpython/Doc'))
from conf import *

# FIXME
# For fixing the warnings that state that 'cwe', 'cve' roles are already
# defined.
#
# WARNING: role 'cve' is already registered, it will be overridden [app.add_role]
# WARNING: role 'cwe' is already registered, it will be overridden [app.add_role]
del extlinks["cwe"]
del extlinks["cve"]

# Call patchlevel with the proper path to get the version from
# instead of hardcoding it
from patchlevel import get_version_info
Expand Down
2 changes: 1 addition & 1 deletion requirements.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ sphinx-lint==0.7.0
tabulate

# from cpython/Doc/requirements.txt
sphinx~=8.1.0
sphinx~=8.0.0

blurb

Expand Down