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
SPAdes: add python dependency #89778
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.
The commit message tag needs to match the attribute name SPAdes
.
Build in #87494 fine without. Closing. |
Problem was in execution not build, python is a runtime dependency.
|
@SuperSandro2000 Confirmed, patch still required:
|
SPAdes is unusable without this patch, is there anything further I need to do to get it merged? Might be worth considering backporting to 20.09 since it's also broken there. |
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.
I agree that the python dependency allows spades.py to run in a pure environment without explicitly loading python
I read on their GitHub that they support python3 https://github.com/ablab/spades#installation. Can we please use python3 as we are currently remove python2 from as many packages as possible. |
Changed to python 3, rebased against current master, and verified functional. |
Result of 1 package built:
|
Result of 1 package built:
|
Motivation for this change
spades.py requires python.
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)