-
Notifications
You must be signed in to change notification settings - Fork 21
update submodules to latest versions #73
Changes from all commits
e19398c
130bc10
c9ab82e
52f4b91
1e6dae6
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -7,44 +7,42 @@ function build_wheel { | |
local lib_plat=$PLAT | ||
if [ -n "$IS_OSX" ]; then | ||
install_gfortran | ||
else | ||
# For manylinux2010 builds with manylinux1 openblas builds | ||
$use_sudo yum install -y libgfortran-4.4.7 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is needed to use the manylinux2010 openblas builds on the manylinux1 wheel builds. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sounds like the comment is backward. Maybe "Needed for manylinux1 wheels using libraries built for manylinux2010." |
||
fi | ||
build_libs $lib_plat | ||
# Fix version error for development wheels by using bdist_wheel | ||
build_bdist_wheel $@ | ||
} | ||
|
||
function build_libs { | ||
local plat=${1:-$PLAT} | ||
# Force 64-bit OpenBLAS library for macOS intel (dual arch) | ||
# builds. For these builds, we pretend to be dual arch, but in | ||
# fact we're only using the 64-bit build of OpenBLAS | ||
if [ -n $IS_OSX ] && [ $plat == intel ]; then | ||
plat=x86_64 | ||
fi | ||
local tar_fname=$(get_gf_lib "openblas-${OPENBLAS_VERSION}" "$plat") | ||
local tar_path=$(abspath $tar_fname) | ||
# Sudo needed for macOS | ||
local use_sudo="" | ||
[ -n "$IS_OSX" ] && use_sudo="sudo" | ||
(cd / && $use_sudo tar zxf $tar_path) | ||
# Use the same incantation as numpy/tools/travis-before-install.sh to | ||
# download and un-tar the openblas libraries. The python call returns | ||
# the un-tar root directory, then the files are copied into /usr/local. | ||
# Could utilize a site.cfg instead to prevent the copy. | ||
python -mpip install urllib3 | ||
basedir=$(python numpy/tools/openblas_support.py) | ||
$use_sudo cp -r $basedir/lib/* /usr/local/lib | ||
$use_sudo cp $basedir/include/* /usr/local/include | ||
} | ||
|
||
function get_test_cmd { | ||
local extra_argv=${1:-$EXTRA_ARGV} | ||
echo "import sys; import numpy; \ | ||
sys.exit(not numpy.test('full', \ | ||
extra_argv=[${extra_argv}]))" | ||
extra_argv=['-vv', ${extra_argv}]))" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. For debugging? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. yes |
||
} | ||
|
||
function run_tests { | ||
# Runs tests on installed distribution from an empty directory | ||
# We only run the 64 bit tests as of NumPy 1.16. | ||
if [ -z "$IS_OSX" ]; then | ||
apt-get -y update && apt-get install -y gfortran | ||
fi | ||
python -c "$(get_test_cmd)" | ||
# Check bundled license file | ||
python ../check_license.py | ||
# Show BLAS / LAPACK used | ||
# Show BLAS / LAPACK used. Since this uses a wheel we cannot use | ||
# tools/openblas_config.py; tools is not part of what is shipped | ||
python -c 'import numpy; numpy.show_config()' | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
# Environment variables for 32-bit build. | ||
# The important difference from the 64-bit build is `-msse2` to | ||
# compile sse loops for ufuncs. | ||
OPENBLAS_VERSION="v0.3.7" | ||
MACOSX_DEPLOYMENT_TARGET=10.9 | ||
CFLAGS="-msse2 -std=c99 -fno-strict-aliasing" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is comes from There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The needed bit is the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Added a line to the README, as well as explaining the OpenBLAS download |
+1 −1 | .travis.yml | |
+31 −2 | gfortran_utils.sh | |
+20 −0 | test_gf_utils.sh |
+26 −0 | .gitignore | |
+17 −16 | .travis.yml | |
+21 −7 | README.rst | |
+154 −14 | common_utils.sh | |
+10 −1 | configure_build.sh | |
+11 −5 | docker_build_wrap.sh | |
+7 −0 | docker_test_wrap.sh | |
+0 −32 | manylinux_utils.sh | |
+3 −41 | osx_utils.sh | |
+4 −10 | supported_wheels.py | |
+4 −0 | tests/config.sh | |
+1 −4 | tests/test_fill_pypy_ver.sh | |
+4 −1 | tests/test_library_builders.sh | |
+6 −0 | tests/test_multibuild.sh | |
+37 −2 | tests/test_supported_wheels.sh | |
+9 −2 | travis_linux_steps.sh |
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.
lines 19-here update the confusing travis.yml: we don't have any python3.6 jobs to exclude,
sudo
is no longer needed,matrix
should be replaced byjobs
.