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
pythonPackages.mpi4py: 2.0.0 -> 3.0.0 #31695
Conversation
Note that Python 3 already is supported for 2.0.0. Maybe that should be backported? |
Tests run with sandboxing enabled again and we can remove some of the fixes as they aren't necessary anymore. PyPy and Python 3 are also supported.
7945130
to
fb8ff9e
Compare
@GrahamcOfBorg build pythonPackages.mpi4py |
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.
Failure for system: x86_64-darwin
/nix/store/vgapfdgsbh4znd2c396hk2c27ggvs0rz-mpi4py-3.0.0.tar.gz.drv
/nix/store/bgvd1bswbklb1claf4s831szznqk07nc-python2.7-mpi4py-3.0.0.drv
these paths will be fetched (5.44 MiB download, 16.02 MiB unpacked):
/nix/store/7ikakkxayhv0p6c7rv276q6sbz2kpa07-openmpi-1.10.7
fetching path ‘/nix/store/7ikakkxayhv0p6c7rv276q6sbz2kpa07-openmpi-1.10.7’...
unable to connect to ‘mac1’
unable to open SSH connection to ‘mac1’, trying other available machines...
building path(s) ‘/nix/store/l3xa1p4mzsh0nd52g8lhxaj4ybc90znp-mpi4py-3.0.0.tar.gz’
killing process 8776
error: a ‘x86_64-darwin’ is required to build ‘/nix/store/vgapfdgsbh4znd2c396hk2c27ggvs0rz-mpi4py-3.0.0.tar.gz.drv’, but I am a ‘x86_64-linux’
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.
Success for system: x86_64-linux
[0@localhost] Python 2.7 (/nix/store/f6pmgza1hha574hm76p98mc2s6gsn3y8-python-2.7.14/bin/python2.7)
[0@localhost] MPI 3.0 (Open MPI 1.10.7)
[0@localhost] mpi4py 3.0.0 (build/lib.linux-x86_64-2.7/mpi4py)
....s.........................................................................................................................................................................................................................................................s.s.s.s.........................................ssssssssssssssssssssssss..........................................................ss...................................................................ssssssssssss..................................s.ss.........s............ssss............................................................................................................................................................s.s..........ssssssssssssss........................ssssss...........ssssss......s...s................................................................................................................................ss......s...ss..............................................s..s...s..s...ssssssssssssssssssssssssssssssssssssssss..........................................................................................................................
----------------------------------------------------------------------
Ran 1134 tests in 13.042s
OK (skipped=130)
warning: SQLite database is busy
/nix/store/j8z8sh11rgfc44laym4vplhg0zax5mn5-python2.7-mpi4py-3.0.0
Looks good, merging! @FRidh do you think 17.09 should enable python 3 for this package based on @Moredread comments above? |
Supporting more on 17.09 than now is fine as long as it doesn't break anything. It just requires someone making the changes and testing it. |
Tests run with sandboxing enabled again and we can remove some of the fixes as
they aren't necessary anymore.
PyPy and Python 3 are also supported.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)