-
-
Notifications
You must be signed in to change notification settings - Fork 14.8k
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
Sphinx: 5.3.0 -> 7.1.2 #249230
Sphinx: 5.3.0 -> 7.1.2 #249230
Conversation
@ofborg build sphinx |
Why is this breaking? |
Oh no, a patch fixing that |
I have been busy with another project, so I hope some WIP I had can help you get this across the finish line. There is one pretty non-obvious challenge with it. (Or, I'd love to collaborate to figure out the best way forward): #244625 |
I can see a path forward here. First, upstream in sphinx-doc/sphinx#11495 updated and released many extensions that now use implicit namespaces, and we can try to pull those in. For the ones that are remaining, #250830 opened my eyes to #244625 has a record of the packages that I found were involved. I'm currently busy trying to fix all the breakages I introduced in #248866, so it will be a couple of days before I can get to this again. If in the meantime you'd like to take it on, please be my guest! |
closing in favor of the linked PR from tjni |
Description of changes
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)