Merge pull request #401 from Dekken/pip_segfault

call import_array statically for linux/mac

lib/swig/tick/array/array_module.i

@@ -9,9 +9,19 @@
 #include <system_error>
 #include "numpy/arrayobject.h"
 #include "numpy/npy_math.h"
+
+#ifndef _WIN32
+int tick_import_numpy(){
+  return _import_array();
+}
+static const int tick_numpy_import = tick_import_numpy();
+#endif
+
 #include "tick/base/debug.h"
 #include "tick/array/carray_python.h"
 #include "tick/array/vector_operations.h"
+
+
 %}
 
 

setup.py

@@ -353,7 +353,11 @@ def add_dir_name(dir_name, filenames):
             lib = os.path.join(build_dir, mod.build, "_"+mod.extension_name)
             lib += os.path.splitext(sysconfig.get_config_var("EXT_SUFFIX"))[0]
             libraries.append(lib)
-        # names (i.e. not lib<name>.so) we specify the full library path
+        elif platform.system() == 'Linux':
+            lib_dir = os.path.abspath(os.path.join(build_dir, mod.build))
+            extra_link_args.append("-L"+lib_dir)
+            extra_link_args.append("-Wl,-rpath,"+lib_dir)
+            extra_link_args.append("-l:"+mod.lib_filename)
         else:
             extra_link_args.append(os.path.abspath(
                 os.path.join(build_dir, mod.build, mod.lib_filename)))

tools/docker/build_wheels.sh

@@ -2,14 +2,14 @@
 
 set -e -x
 
-python_versions=(cp36-cp36m)
+python_versions=(cp36-cp36m cp37-cp37m)
 
 for PYVER in ${python_versions[@]}; do
     PYBIN=/opt/python/${PYVER}/bin
 
     cd /io
-
-    "${PYBIN}/python3" -mpip install -r requirements.txt
+    "${PYBIN}/python3" -m pip install pip --upgrade
+    "${PYBIN}/python3" -m pip install -r requirements.txt
     "${PYBIN}/python3" setup.py bdist_wheel --dist-dir=/tick/wheelhouse
 done