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

pythonPackages.sphinx: 1.8.3 -> 2.2.0 #71601

Closed
wants to merge 1 commit into from

Conversation

wamserma
Copy link
Member

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
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.
Notify maintainers

cc @

@jonringer
Copy link
Contributor

Thank you for combining all the commits into a single PR :)

@jonringer
Copy link
Contributor

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

@FRidh FRidh self-assigned this Oct 22, 2019
@wamserma wamserma changed the base branch from master to staging October 22, 2019 07:22
@wamserma
Copy link
Member Author

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

Ok, picked up the change requests from the individual PRs, rebased onto staging cleaned up the history. Feel free to close the individual PRs.

@FRidh
Copy link
Member

FRidh commented Oct 22, 2019

Thanks.

Feel free to close the individual PRs.

Could you do that yourself?

@wamserma
Copy link
Member Author

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 conf.py.

@FRidh
Copy link
Member

FRidh commented Oct 23, 2019

I will push this to a special branch soon where it will build on Hydra. It won't build everything but a lot.

@wamserma
Copy link
Member Author

I will push this to a special branch soon where it will build on Hydra. It won't build everything but a lot.

Thx.

@FRidh FRidh added this to New in Staging Oct 24, 2019
@FRidh FRidh moved this from New to Not ready in Staging Oct 24, 2019
@wamserma
Copy link
Member Author

rebased on current staging

@FRidh
Copy link
Member

FRidh commented Oct 28, 2019

ahh, I forgot to include sphinx itself when I cherry-picked the range. Sorry for that.

@wamserma
Copy link
Member Author

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.

@FRidh
Copy link
Member

FRidh commented Oct 28, 2019

@FRidh
Copy link
Member

FRidh commented Nov 12, 2019

This has landed in master so closing.

@FRidh FRidh closed this Nov 12, 2019
Staging automation moved this from WIP to Done Nov 12, 2019
@wamserma wamserma deleted the sphinx-update branch November 12, 2019 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Staging
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants