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.bleach: add implicit setuptools dependency #70006

Merged
merged 1 commit into from Oct 3, 2019

Conversation

elohmeier
Copy link
Contributor

@elohmeier elohmeier commented Sep 29, 2019

Motivation for this change

I've noticed the bleach package uses modules from the setuptools package, which is not explicitly referenced in the setup.py.

Related to #69411.

Things done

Added setuptools to the propagatedBuildInputs.

  • 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 @prikhi

Copy link
Contributor

@jonringer jonringer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nix-review passes on NixOS (4 broken on master)
diff LGTM
fixes original issue

[37 built (4 failed), 196 copied (1955.2 MiB), 494.0 MiB DL]
error: build of '/nix/store/42yn16xxfdjn6dhyrkx7qvqa55vw7dq0-env.drv' failed
https://github.com/NixOS/nixpkgs/pull/70006
2 package are marked as broken and were skipped:
ihaskell python27Packages.spyder

4 package failed to build:
python27Packages.cufflinks python37Packages.cufflinks python37Packages.datashader python37Packages.starfish

64 package were build:
isso jupyter matrix-synapse python27Packages.ansible-kernel python27Packages.bleach python27Packages.colorcet python27Packages.holoviews python27Packages.hvplot python27Packages.ipywidgets python27Packages.jupyter python27Packages.kmapper python27Packages.mezzanine python27Packages.modeled python27Packages.moretools python27Packages.nbconvert python27Packages.nbsmoke python27Packages.nbsphinx python27Packages.notebook python27Packages.readme python27Packages.readme_renderer python27Packages.restview python27Packages.robotframework-tools python27Packages.runway-python python27Packages.twine python27Packages.widgetsnbextension python27Packages.zetup python37Packages.Nikola python37Packages.Pweave python37Packages.ansible-kernel python37Packages.bleach python37Packages.colorcet python37Packages.fints python37Packages.holoviews python37Packages.hvplot python37Packages.intake python37Packages.ipywidgets python37Packages.jupyter python37Packages.jupyterhub python37Packages.jupyterhub-ldapauthenticator python37Packages.jupyterlab python37Packages.jupyterlab_launcher python37Packages.jupyterlab_server python37Packages.kmapper python37Packages.mezzanine python37Packages.modeled python37Packages.moretools python37Packages.nbconvert python37Packages.nbsmoke python37Packages.nbsphinx python37Packages.notebook python37Packages.oauthenticator python37Packages.readme python37Packages.readme_renderer python37Packages.restview python37Packages.robotframework-tools python37Packages.runway-python python37Packages.scikit-tda spyder python37Packages.toggl-cli python37Packages.twine python37Packages.widgetsnbextension python37Packages.zetup sage sageWithDoc

trackpy PR #70223
cufflinks PR #70220

@jonringer
Copy link
Contributor

@GrahamcOfBorg build python3Packages.bleach pythonPackages.bleach

@jonringer jonringer merged commit 31c4f79 into NixOS:master Oct 3, 2019
@jonringer jonringer added the 9.needs: port to stable A PR needs a backport to the stable release. label Oct 3, 2019
@jonringer
Copy link
Contributor

backported 014afee

@jonringer jonringer added 8.has: port to stable A PR already has a backport to the stable release. and removed 9.needs: port to stable A PR needs a backport to the stable release. labels Oct 3, 2019
@elohmeier elohmeier deleted the bleach branch October 4, 2019 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants