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

Replace intersphinx_mapping format removed in Sphinx 8.0.0 #1365

Closed
wants to merge 1 commit into from

Conversation

d7415
Copy link
Collaborator

@d7415 d7415 commented Oct 31, 2024

Fixes #1356

@d7415
Copy link
Collaborator Author

d7415 commented Oct 31, 2024

The new format has been supported since v1.0 in 2010.

@d7415
Copy link
Collaborator Author

d7415 commented Oct 31, 2024

Branch protection means I can't merge this until we fix the tests, possibly with #1364.

@mathstuf
Copy link

mathstuf commented Dec 2, 2024

Fedora Rawhide is looking to update to Sphinx 8…any progress here? I see concerns about icalendar 6 being "too new"…but we're getting to the point that distros are going to start requiring the non-deprecated behavior too…

@mathstuf
Copy link

This has now become an FTBFS in Fedora 42: https://bugzilla.redhat.com/show_bug.cgi?id=2340695

@antonmosich
Copy link

Is intersphinx even used anywhere? Is there a reason to not outright remove that line, as well as the import of the extension?

@WhyNotHugo
Copy link
Member

Nearly missed this, thanks.

@WhyNotHugo
Copy link
Member

The new format has been supported since v1.0 in 2010.

I'm glad that someone else has confirmed this; I though I was going crazy XD

@WhyNotHugo
Copy link
Member

Merged.

@WhyNotHugo WhyNotHugo closed this Feb 12, 2025
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.

Build problem with sphinx 8
4 participants