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
ocamlPackages_latest.phylogenetics: init at unstable-2019-11-15 #74837
Conversation
eb63c7b
to
f6bc3f5
Compare
094f01b
to
0886406
Compare
6345efc
to
275a665
Compare
@@ -291,6 +295,10 @@ let | |||
|
|||
gmap = callPackage ../development/ocaml-modules/gmap { }; | |||
|
|||
gnuplot = callPackage ../development/ocaml-modules/gnuplot { | |||
inherit (pkgs) gnuplot; |
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.
Please similarly fix the input to metrics-unix
. Thanks.
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's no metrics-unix
on this branch; did you mean to comment somewhere else?
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 is one in master
. Please rebase and we’ll be good to go.
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.
Rebase done.
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.
Cool. Now there is a metrics-unix
whose input you can fix.
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 understand what you're asking.
Also, what does metrics-unix
have to do with this package? Shouldn't that be a separate issue/PR?
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 ocamlPackages.metrics-unix
package takes as input gnuplot
from the main package set. This PR changes this input: it gives ocamlPackages.gnuplot
instead of pkgs.gnuplot
.
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.
Sorry, didn't see that. Done, metrics-unix
builds correctly.
@GrahamcOfBorg build ocaml-ng.ocamlPackages_4_08.phylogenetics |
Motivation for this change
Add Biocaml, Gnuplot-Ocaml, and Phylogenetics packages.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @