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
truvari: 1.3.4 -> 2.0.2, add new dependencies & fix build #98283
Conversation
there seem to be some conflicts :( |
Yup, this is because #98250 was merged, which was my alternate take on this problem. That's good, because I can easily fix this one up, get it in |
seeing it has no tests, use pythonImportsCheck in an attempt to detect total breakage, even though it's not intended to be used as an importable module
efbe5be
to
ecd5886
Compare
doCheck = !isPy3k; | ||
preCheck = '' | ||
substituteInPlace test/test.py \ | ||
--replace 'test_remote_file' 'dont_test_remote_file' | ||
''; |
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.
not sure why this is still being ran for python2:
https://github.com/NixOS/nixpkgs/pull/98283
1 package failed to build:
python27Packages.pytabix
10 packages built:
bwa python27Packages.acebinf python37Packages.acebinf python37Packages.bwapy python37Packages.pytabix python38Packages.acebinf python38Packages.bwapy python38Packages.pytabix tebreak truvari
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.
nevermind, i read that right
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.
That's .. strange. What was the failure? An earlier version of this PR didn't disable the tests on py27, but this version builds for me on macos 10.14 & linux under nixpkgs-review.
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 believe it was an encoding error:
....
but I can no longer reproduce it....
https://github.com/NixOS/nixpkgs/pull/98283
11 packages built:
bwa python27Packages.acebinf python27Packages.pytabix python37Packages.acebinf python37Packages.bwapy python37Packages.pytabix python38Packages.acebinf python38Packages.bwapy python38Packages.pytabix tebreak truvari
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.
diff LGTM
https://github.com/NixOS/nixpkgs/pull/98283
11 packages built:
bwa python27Packages.acebinf python27Packages.pytabix python37Packages.acebinf python37Packages.bwapy python37Packages.pytabix python38Packages.acebinf python38Packages.bwapy python38Packages.pytabix tebreak truvari
install -vD -t $out/include bntseq.h | ||
install -vD -t $out/include bwa.h | ||
install -vD -t $out/include bwamem.h | ||
install -vD -t $out/include bwt.h |
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 don't feel strongly though
install -vD -t $out/include bntseq.h | |
install -vD -t $out/include bwa.h | |
install -vD -t $out/include bwamem.h | |
install -vD -t $out/include bwt.h | |
install -vD -t $out/include bntseq.h bwa.h bwamem.h bwt.h |
doCheck = !isPy3k; | ||
preCheck = '' | ||
substituteInPlace test/test.py \ | ||
--replace 'test_remote_file' 'dont_test_remote_file' | ||
''; |
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 believe it was an encoding error:
....
but I can no longer reproduce it....
https://github.com/NixOS/nixpkgs/pull/98283
11 packages built:
bwa python27Packages.acebinf python27Packages.pytabix python37Packages.acebinf python37Packages.bwapy python37Packages.pytabix python38Packages.acebinf python38Packages.bwapy python38Packages.pytabix tebreak truvari
Motivation for this change
Possibly ZHF: #97479
This is an alternative to #98250 where I simply minimally fixed the exitsing
truvari
. This bumps it to 2.0.2 and adds new python dependenciesbwapy
,pytabix
andacebinf
. Also needed to expose the compiled library in thebwa
package.Not sure which we want to use. Could do the minimal bump for 20.09 and keep this for
master
.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)