Rtree Wheel Infrastructure

.github/workflows/wheels.yml

@@ -0,0 +1,50 @@
+name: Build Wheels
+
+on: [push, pull_request]
+
+jobs:
+  build_wheels:
+    name: Build wheel on ${{ matrix.os }}
+    runs-on: ${{ matrix.os }}
+    env:
+      CIBW_TEST_REQUIRES: pytest numpy
+      CIBW_TEST_COMMAND: pytest {project}/tests
+      CIBW_BEFORE_BUILD_LINUX: "pip install cmake; bash {project}/ci/install_libspatialindex.bash"
+      CIBW_BEFORE_BUILD_MACOS: "bash {project}/ci/install_libspatialindex.bash"
+    strategy:
+      matrix:
+        os: [ubuntu-18.04] #, windows-latest, macos-latest]
+    steps:
+    - uses: actions/checkout@v1
+    - uses: actions/setup-python@v1
+      name: Install Python
+      with:
+        python-version: '3.7'
+    - name: Install cibuildwheel
+      run: |
+        python -m pip install cibuildwheel==1.3.0
+    - name: Install Homebrew Packages
+      if: startsWith(matrix.os, 'macos')
+      run: |
+        # provides sha256sum
+        brew install coreutils
+    - name: Install Visual C++ for Python 2.7
+      if: startsWith(matrix.os, 'windows')
+      run: |
+        choco install vcpython27 -f -y
+    - name: Build wheels
+      run: |
+        python -m cibuildwheel --output-dir wheelhouse
+    - uses: actions/upload-artifact@v1
+      with:
+        name: wheels
+        path: ./wheelhouse
+    - name: Upload To PyPi
+      env:
+        TWINE_USERNAME: ${{ secrets.PYPI_USERNAME }}
+        TWINE_PASSWORD: ${{ secrets.PYPI_PASSWORD }}
+      # TODO : remove `if false` statement after secrets are set in Github UI
+      if: false
+      run: |
+        pip install twine
+        twine upload ./wheelhouse/*

ci/install_libspatialindex.bash

@@ -0,0 +1,26 @@
+#!/bin/bash
+set -xe
+
+# A simple script to install libspatialindex from a Github Release
+VERSION=1.9.3
+SHA256=63a03bfb26aa65cf0159f925f6c3491b6ef79bc0e3db5a631d96772d6541187e
+
+
+rm $VERSION.zip || true
+curl -L -O https://github.com/libspatialindex/libspatialindex/archive/$VERSION.zip
+
+# check the file hash
+echo "${SHA256}  ${VERSION}.zip" | sha256sum --check
+
+rm -rf "libspatialindex-${VERSION}" || true
+unzip $VERSION
+cd libspatialindex-${VERSION}
+
+mkdir build
+cd build
+cmake ..
+make -j 4
+make install
+
+cd ../..
+rm -rf "libspatialindex-${VERSION}"

rtree/core.py

@@ -2,13 +2,9 @@
 import sys
 import platform
 import ctypes
-from ctypes.util import find_library
-
-
-class RTreeError(Exception):
-    "RTree exception, indicates a RTree-related error."
-    pass
 
+from . import finder
+from .exceptions import RTreeError
 
 def check_return(result, func, cargs):
     "Error checking for Error calls"
@@ -76,97 +72,8 @@ def free_error_msg_ptr(result, func, cargs):
     rt.Index_Free(p)
     return retvalue
 
-
-if os.name == 'nt':
-
-    def _load_library(dllname, loadfunction, dllpaths=('', )):
-        """Load a DLL via ctypes load function. Return None on failure.
-        Try loading the DLL from the current package directory first,
-        then from the Windows DLL search path.
-        """
-        try:
-            dllpaths = (os.path.abspath(os.path.dirname(__file__)),
-                        ) + dllpaths
-        except NameError:
-            pass  # no __file__ attribute on PyPy and some frozen distributions
-        for path in dllpaths:
-            if path:
-                # temporarily add the path to the PATH environment variable
-                # so Windows can find additional DLL dependencies.
-                try:
-                    oldenv = os.environ['PATH']
-                    os.environ['PATH'] = path + ';' + oldenv
-                except KeyError:
-                    oldenv = None
-            try:
-                return loadfunction(os.path.join(path, dllname))
-            except (WindowsError, OSError):
-                pass
-            finally:
-                if path and oldenv is not None:
-                    os.environ['PATH'] = oldenv
-        return None
-
-    base_name = 'spatialindex_c'
-    if '64' in platform.architecture()[0]:
-        arch = '64'
-    else:
-        arch = '32'
-
-    lib_name = '%s-%s.dll' % (base_name, arch)
-    rt = None
-    if 'SPATIALINDEX_C_LIBRARY' in os.environ:
-        lib_path, lib_name = \
-            os.path.split(os.environ['SPATIALINDEX_C_LIBRARY'])
-        rt = _load_library(lib_name, ctypes.cdll.LoadLibrary, (lib_path,))
-    # try wheel location
-    if not rt:
-        lib_path = os.path.abspath(os.path.join(
-                       os.path.dirname(__file__), "lib"))
-        rt = _load_library(lib_name, ctypes.cdll.LoadLibrary, (lib_path,))
-    # try conda location
-    if not rt:
-        if 'conda' in sys.version:
-            lib_path = os.path.join(sys.prefix, "Library", "bin")
-            rt = _load_library(lib_name, ctypes.cdll.LoadLibrary, (lib_path,))
-    if not rt:
-        rt = _load_library(lib_name, ctypes.cdll.LoadLibrary)
-
-    if not rt:
-        raise OSError("could not find or load %s" % lib_name)
-
-elif os.name == 'posix':
-
-    if 'SPATIALINDEX_C_LIBRARY' in os.environ:
-        lib_name = os.environ['SPATIALINDEX_C_LIBRARY']
-        rt = ctypes.CDLL(lib_name)
-    else:
-        try:
-            # try loading libspatialindex from the wheel location
-            # inside the package
-
-            lib_path = os.path.abspath(os.path.join(
-                       os.path.dirname(__file__), "lib"))
-            old_dir = os.getcwd()
-            os.chdir(lib_path)
-            full_path = os.path.join(lib_path, "libspatialindex_c.so")
-            rt = ctypes.cdll.LoadLibrary(full_path)
-
-            # Switch back to the original working directory
-            os.chdir(old_dir)
-            if not rt:
-                raise FileNotFoundError("%s not loaded" % full_path)
-        except FileNotFoundError:
-            lib_name = find_library('spatialindex_c')
-            rt = ctypes.CDLL(lib_name)
-            if not rt:
-                raise FileNotFoundError("%s not loaded" % full_path)
-
-    if not rt:
-        raise OSError("Could not load libspatialindex_c library")
-
-else:
-    raise RTreeError('Unsupported OS "%s"' % os.name)
+# load the shared library by looking in likely places
+rt, _rt_path = finder.load()
 
 rt.Error_GetLastErrorNum.restype = ctypes.c_int
 

rtree/exceptions.py

@@ -0,0 +1,4 @@
+
+class RTreeError(Exception):
+    "RTree exception, indicates a RTree-related error."
+    pass