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.notebook: cleanup #37615
pythonPackages.notebook: cleanup #37615
Conversation
@GrahamcOfBorg build python2.pkgs.notebook python3.pkgs.notebook |
@@ -44,8 +44,8 @@ buildPythonPackage rec { | |||
# disable warning_filters | |||
preCheck = lib.optionalString (!isPy3k) '' | |||
echo "" > setup.cfg |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This could be replaced by touch
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, updated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It just occurred to me that this was probably intentional to truncate the file. Should we reinstate it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup, it seems to fail when building with Python 2. I've opened #37620.
As discussed in NixOS#37607 (comment).
69d6f76
to
01f29bb
Compare
Success on x86_64-linux (full log) Attempted: python2.pkgs.notebook, python3.pkgs.notebook Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: python2.pkgs.notebook, python3.pkgs.notebook Partial log (click to expand)
|
Thanks! |
These were accidentally broken by NixOS#37615.
As discussed in #37607 (comment).
Motivation for this change
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)