Adding simple pip-based wheel build instead of complicated delvewheel

peekxc · Dec 5, 2023 · dcf8f54 · dcf8f54
1 parent cd22404
commit dcf8f54
Show file tree

Hide file tree

Showing 5 changed files with 18 additions and 6 deletions.
diff --git a/.github/workflows/build_windows.yml b/.github/workflows/build_windows.yml
@@ -64,4 +64,16 @@ jobs:
     - name: Test with pytest
       run: |
         python -m pytest tests/ --cov=primate --benchmark-skip
-        coverage report -m
+        coverage report -m
+    - name: Build a wheel
+      run: | 
+        python -m pip wheel . -w dist
+    - name: Test the wheel 
+      run: | 
+        pip uninstall primate -y
+        python -m pip install dist/primate*.whl
+        python -m pytest tests/ --cov=primate --benchmark-skip
+        coverage report -m
+    # - name: Make wheel w/ delvewheel 
+    #   run: |
+    #     bash ./tools/repair_windows.sh
diff --git a/more_itertools-10.1.0-py3-none-any.whl b/more_itertools-10.1.0-py3-none-any.whl
diff --git a/numpy-1.26.2-cp311-cp311-macosx_10_9_x86_64.whl b/numpy-1.26.2-cp311-cp311-macosx_10_9_x86_64.whl
diff --git a/primate-0.2.3-cp311-cp311-macosx_10_15_x86_64.whl b/primate-0.2.3-cp311-cp311-macosx_10_15_x86_64.whl
diff --git a/tools/cibw_linux.sh b/tools/cibw_linux.sh
@@ -9,11 +9,11 @@ if [ -n "$(command -v yum)" ]; then
   yum install python39-devel
   alias python=python3.9
 elif [ -n "$(command -v apt-get)" ]; then 
-  apt-get update -y
-  apt-get install -y clang libomp-dev
-  apt-get update -y
-  apt-get install -y libopenblas-dev
-  apt-get install -y python3-dev
+  sudo apt-get update -y
+  sudo apt-get install -y clang libomp-dev
+  sudo apt-get update -y
+  sudo apt-get install -y libopenblas-dev
+  sudo apt-get install -y python3-dev
 elif [ -n "$(command -v apk)" ]; then 
   apk update
   apk add clang # looks like Alpine only supports up to clang 10 ?