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
cvxopt: 1.1.7 -> 1.1.9, add optional extensions #38780
Conversation
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 add a little bit of white space, i.e. blank lines.
export CVXOPT_FFTW_INC_DIR=${fftw.dev}/include | ||
''; | ||
checkPhase = '' | ||
python -m unittest discover -s tests |
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.
Replace python
by ${python.interpreter}
buildInputs = [ | ||
openblasCompat | ||
suitesparse | ||
] ++ stdenv.lib.optionals withGlpk [ |
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.
stdenv.lib
can be replaced by lib
everywhere
@@ -0,0 +1,71 @@ | |||
{ stdenv |
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.
Use lib
here
I don't see nothing moved out of pythonPackages. |
I addressed your comments. I don't know what you mean with the last one: |
The commit message needs to be amended here as well. @GrahamcOfBorg build python2.pkgs.cvxopt python3.pkgs.cvxopt |
Success on aarch64-linux (full log) Attempted: python2.pkgs.cvxopt, python3.pkgs.cvxopt Partial log (click to expand)
|
Done. |
glpk | ||
] ++ lib.optionals withGsl [ | ||
gsl | ||
]; |
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.
You should be able to leave buildInputs
out completely because they are all mentioned in preConfigure
.
Also adds support for the optional extensions glpk, gsl, fftw and enables tests.
@GrahamcOfBorg build python2.pkgs.cvxopt python3.pkgs.cvxopt |
Success on x86_64-linux (full log) Attempted: python2.pkgs.cvxopt, python3.pkgs.cvxopt Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: python2.pkgs.cvxopt, python3.pkgs.cvxopt Partial log (click to expand)
|
Motivation for this change
python-packages
Maintainer: @edwtjo
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)