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
coconut: init at 1.4.3 #102675
coconut: init at 1.4.3 #102675
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.
And you are the module in the top level file python-packages.nix
and the application in all-packages.nix
.
efaa65b
to
9d093a1
Compare
451d34f
to
d275c2f
Compare
Thanks for your review @jonringer, I have applied all the corrections you pointed out, rebuilt locally, and retested the binary in NixOS. |
d275c2f
to
dd2d932
Compare
Ok, changed to fetch from github instead of PyPI, though I am having an issue with the coconut test-suite where it doesn't find the 'coconut' binary it just built in some tests and some tests attempt to fetch external resources via git E AssertionError: Return code not as expected (128 != 0) in: ['git', 'clone', 'https://github.com/evhub/pyston.git'] |
9328162
to
920c05b
Compare
Enabled passing tests only |
yea, tests which do impure things like doing git commands, should be deselected. Really, we just want to make sure that the package is usable at runtime and there's not any assumptions broken. E.g. shell commands not working |
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.
otherwise LGTM
920c05b
to
e593c14
Compare
e593c14
to
4a59bd1
Compare
Applied your last 2 suggestions, thanks for the help. |
Hi, just bumping this PR. |
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.
LGTM
Result of nixpkgs-review pr 102675 1
5 packages built:
- coconut (python38Packages.coconut)
- python27Packages.cpyparsing
- python37Packages.coconut
- python37Packages.cpyparsing
- python38Packages.cpyparsing
Thanks for reviewing, merging, and more generally maintaining Nixpkgs. uwu |
Add Coconut interpreter.
Includes a patch that affected building: evhub/coconut#560
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)