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
samtools-old: init at 0.1.19 #34530
samtools-old: init at 0.1.19 #34530
Conversation
@mimadrid as maintainer of the more recent samtools version, feedback is appreciated. Cheers. |
@GrahamcOfBorg build samtools-old |
Success on x86_64-linux (full log) Partial log (click to expand)
|
Success on aarch64-linux (full log) Partial log (click to expand)
|
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 need a new derivation? Isn't it possible to override samtools
and add samtools-old = samtools.overrideAttrs ...
?
I don't know enough about nix yet to follow your suggestion. However, from what I gathered I'd have to override almost every attribute in the existing samtools derivation. |
Then it doesn't make much sense, you're right. To learn more about |
samtools-1.5 already exists but some software requires the older version
@unode Awesome, thanks a lot. I think it's ok now to be merged |
Motivation for this change
nixpkgs already contains samtools-1.5 but some software still requires the older 0.1.19 version.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)