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.aplpy: init at 2.0.3 #55711
Conversation
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.
Please correct the commit messages and fix the evaluation errors.
@GrahamcOfBorg build python2.pkgs.aplpy python3.pkgs.aplpy |
|
Where does this come from? |
You will find it in the build log (see the checks): |
Please reorder the commits and squash the last 3 ones into your earlier ones. |
@GrahamcOfBorg build python2.pkgs.aplpy python3.pkgs.aplpy |
@GrahamcOfBorg build python3.pkgs.aplpy |
I still get the same error locally. |
I'm trying to reproduce this error on macOS with sandboxing, but I get a different error:
Is this a bug in the |
@smaret I have no experience whatsoever with Darwin. Maybe @matthewbauer can help out. |
I've just on opened an issue for the gnutar sandbox issue on Darwin (#56591). |
@GrahamcOfBorg build python3.pkgs.aplpy |
@GrahamcOfBorg build python3.pkgs.aplpy |
I finally managed to get the sandbox working on Darwin 🎉 The download error was caused by an automatic update of the |
@dotlambda Can you please have a look at this when you get a chance? I think that the PR is now ready for merging. |
@GrahamcOfBorg build python3.pkgs.aplpy |
During the build of pyavm I get
Please add an appropriate |
Done. |
@GrahamcOfBorg build python3.pkgs.aplpy |
Thanks for your detailed review of this PR @dotlambda ! |
Motivation for this change
Add a Nix expression for the Aplpy Python package and its dependencies.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)