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
python2Packages.pyezminc: init at 1.2 #41086
Conversation
buildPythonPackage rec { | ||
pname = "pyezminc"; | ||
version = "1.2"; | ||
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.
drop this line
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 this comment
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.
name
is set automatically by buildPythonPackage
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.
removed
src = fetchFromGitHub { | ||
owner = "BIC-MNI"; | ||
repo = "pyezminc"; | ||
rev = "b9fc331c5ad296348fa1ac39df8b0e26ed667d85"; |
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.
rev = "release-${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.
done
|
||
buildPythonPackage rec { | ||
pname = "pyezminc"; | ||
version = "1.2"; |
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.
1.2.01
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.
done
buildInputs = [ netcdf hdf5 libminc ezminc ]; | ||
propagatedBuildInputs = [ numpy scipy ]; | ||
|
||
preBuild = "export CFLAGS='-fpermissive'"; |
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 think there's some more standard way to do this. Please grep through Nixpkgs.
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.
changed to NIX_CFLAGS_COMPILE
version = "1.2"; | ||
name = "${pname}-${version}"; | ||
|
||
disabled = isPy3k; # ?? |
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.
README says
BUILD REQUIREMENTS:
- python2-devel
so I think it should remain disabled for Python 3. Please update the comment.
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.
comment removed
description = "Python API for libminc using EZMINC"; | ||
license = stdenv.lib.licenses.gpl2; | ||
maintainers = with stdenv.lib.maintainers; [ bcdarwin ]; | ||
platforms = stdenv.lib.platforms.unix; |
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.
drop this line
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 this comment
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.
platforms
is set automatically by buildPythonPackage
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.
removed
|
||
preBuild = "export CFLAGS='-fpermissive'"; | ||
|
||
doCheck = false; # e.g., expects test data in /opt |
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.
Probably needs patching in postPatch
: https://github.com/BIC-MNI/pyezminc/search?q=opt
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.
Is there a package that includes share/icbm152_model_09c
?
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 think they're available as a tarball somewhere so in principle they could be included in this package or another one (e.g. BEAST, not yet Nixified)
474473c
to
5d06aa7
Compare
@GrahamcOfBorg build python2Packages.pyezminc python3Packages.pyezminc |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: python2Packages.pyezminc, python3Packages.pyezminc Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: python2Packages.pyezminc The following builds were skipped because they don't evaluate on aarch64-linux: python3Packages.pyezminc Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: python2Packages.pyezminc The following builds were skipped because they don't evaluate on x86_64-linux: python3Packages.pyezminc Partial log (click to expand)
|
Motivation for this change
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)