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
python3Packages.ipykernel: fix build on darwin #80940
Conversation
@GrahamcOfBorg build python37Packages.ipykernel |
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.
Seems to build fine before for me however.
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.
the 2 commits should probably be squashed
nixpkgs doesn't have a python3.4 interpreter, so the disable change is largely a "noop"
7059e1b
to
165cee5
Compare
Also python34 is not supported: ipython/ipykernel@979e6fd nixpkgs doesn't provide python34 anymore, so pythonOlder "3.5" is always true and can be removed.
165cee5
to
d35009e
Compare
Done!
Yes, I thought of it like more of a declaration. I updated the commit to remove the condition instead of updating it. |
@GrahamcOfBorg build python3Packages.ipykernel |
Also python34 is not supported: ipython/ipykernel@979e6fd nixpkgs doesn't provide python34 anymore, so pythonOlder "3.5" is always true and can be removed. (cherry picked from commit d35009e) cc #80940
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)