-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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
pythonPackages.sphinx: 1.8.3 -> 2.2.0 #71601
Conversation
Thank you for combining all the commits into a single PR :) |
Do you mind rebasing this on top of the staging branch, then changing the base branch to staging. For large rebuilds we do this to avoid having people build against master get hit by large rebuilds |
7bfeb9a
to
f0bedec
Compare
f0bedec
to
8a320a9
Compare
Ok, picked up the change requests from the individual PRs, rebased onto staging cleaned up the history. Feel free to close the individual PRs. |
Thanks.
Could you do that yourself? |
Could some please take the duty of running nix-shell -p nix-review --run 'nix-review pr 71601 This includes building pretty much the whole Haskell ecosystem due to the extensive usage of Sphinx for documentation. I expect quite a few builds to break as Sphinx has changed some defaults, e.g. the name of the documentation index page (contents.rst vs. index.rst) and not all packages may have that configured properly in their |
I will push this to a special branch soon where it will build on Hydra. It won't build everything but a lot. |
Thx. |
8a320a9
to
65b9c59
Compare
rebased on current staging |
ahh, I forgot to include sphinx itself when I cherry-picked the range. Sorry for that. |
Yep. But having sphinx in its own update isn't that bad, as it is the sphinx package itself which will trigger the mass rebuilds and probably break building of documentation for some packages. |
Triggered a job at https://hydra.nixos.org/jobset/nixpkgs/python-unstable |
This has landed in master so closing. |
Motivation for this change
http://www.sphinx-doc.org/en/master/changes.html#release-2-2-0-released-aug-19-2019
depends on #71591 #71593 #71594 #71597 #71598 #71599
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @