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
dendropy: init at 4.3.0 #34533
dendropy: init at 4.3.0 #34533
Conversation
f9d45ab
to
4bd94dc
Compare
}: | ||
|
||
buildPythonPackage rec { | ||
name = "${pname}-${version}"; |
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.
no name
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 tried omitting name but that resulted in a failure when I tried locally
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.
There shouldn't be any. Did you rebase on latest master?
Btw, name
is optional only for buildPythonPackage
and buildPythonApplication
.
sha256 = "bd5b35ce1a1c9253209b7b5f3939ac22beaa70e787f8129149b4f7ffe865d510"; | ||
}; | ||
|
||
preCheck = '' |
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.
This might be better suited for postPatch
.
@GrahamcOfBorg build python2.pkgs.dendropy python3.pkgs.dendropy |
Failure on x86_64-linux (full log) Partial log (click to expand)
|
Looks like a few more tests need to be disabled. Doesn't seem like nixpkgs fault here. |
Failure on aarch64-linux (full log) Partial log (click to expand)
|
The errors are:
Solution:
|
@GrahamcOfBorg build python2.pkgs.dendropy python3.pkgs.dendropy |
Success on x86_64-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.
Just move the rm
command to prePatch
and then this should be good to go.
|
||
meta = { | ||
homepage = http://dendropy.org/; | ||
description = "a Python library for phylogenetic computing"; |
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.
Capitalize first letter please.
Success on aarch64-linux (full log) Partial log (click to expand)
|
Motivation for this change
Dendropy is a versatile python module to handle several phylogenetic formats and build phylogenetic pipelines.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)