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
hisat2: init 2.1.0 #45102
hisat2: init 2.1.0 #45102
Conversation
version = "2.1.0"; | ||
|
||
src = fetchFromGitHub { | ||
owner = "infphilo"; |
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.
Why do you not get the source from the official release tarball?
ftp://ftp.ccb.jhu.edu/pub/infphilo/hisat2/downloads
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 think this is any more "official" since they link to both the tarball and their github page for source. I have no strong desire to use github over the tarball, so I've switched to the tarball.
@GrahamcOfBorg build hisat2 |
Failure on aarch64-linux (full log) Attempted: hisat2 Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: hisat2 Partial log (click to expand)
|
Failure on x86_64-darwin (full log) Attempted: hisat2 Partial log (click to expand)
|
Please fix the darwin build or modify the platforms attribute. |
e131a31
to
a9ef705
Compare
@GrahamcOfBorg build hisat2 |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: hisat2 Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: hisat2 Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: hisat2 Partial log (click to expand)
|
Sorry keep getting the platforms slightly wrong, seems it assumes sse so has to be x86. I've tried to restrict to x86 linux only, is this the cleanest way? |
You can see the available platforms here - nixpkgs/lib/systems/for-meta.nix Line 9 in e4d9ce9
|
Thanks, doesn't look any cleaner than just listing the two platforms. |
Motivation for this change
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)