build/pkgs/{sage_conf,sage_docbuild,sage_setup,sage_sws2rst}/spkg-ins…

…tall: Handle SAGE_WHEELS
sagemath · Oct 3, 2022 · 05b44d7 · 05b44d7
1 parent f2d1576
commit 05b44d7
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 5 deletions.
diff --git a/build/pkgs/sage_conf/spkg-install b/build/pkgs/sage_conf/spkg-install
@@ -12,6 +12,9 @@ fi
 cd src
 if [ "$SAGE_EDITABLE" = yes ]; then
     sdh_pip_editable_install .
+    if [ "$SAGE_WHEELS" = yes ]; then
+        sdh_setup_bdist_wheel && sdh_store_wheel .
+    fi
 else
     sdh_pip_install .
 fi
diff --git a/build/pkgs/sage_docbuild/spkg-install b/build/pkgs/sage_docbuild/spkg-install
@@ -12,6 +12,9 @@ fi
 cd src
 if [ "$SAGE_EDITABLE" = yes ]; then
     sdh_pip_editable_install .
+    if [ "$SAGE_WHEELS" = yes ]; then
+        sdh_setup_bdist_wheel && sdh_store_wheel .
+    fi
 else
     sdh_pip_install .
 fi
diff --git a/build/pkgs/sage_setup/spkg-install b/build/pkgs/sage_setup/spkg-install
@@ -12,6 +12,9 @@ fi
 cd src
 if [ "$SAGE_EDITABLE" = yes ]; then
     sdh_pip_editable_install .
+    if [ "$SAGE_WHEELS" = yes ]; then
+        sdh_setup_bdist_wheel && sdh_store_wheel .
+    fi
 else
     sdh_pip_install .
 fi
diff --git a/build/pkgs/sage_sws2rst/spkg-install b/build/pkgs/sage_sws2rst/spkg-install
@@ -3,16 +3,21 @@
 # For type=script packages, the build rule in build/make/Makefile sources
 # sage-env but not sage-dist-helpers.
 lib="$SAGE_ROOT/build/bin/sage-dist-helpers"
-. "$lib"
+source "$lib"
 if [ $? -ne 0 ]; then
     echo >&2 "Error: failed to source $lib"
     echo >&2 "Is $SAGE_ROOT the correct SAGE_ROOT?"
     exit 1
 fi
-set -e
-# We build the wheel directly with "setup.py bdist_wheel", not with "pip wheel",
-# because pip does not handle our symlinks correctly.
-(cd src && sdh_setup_bdist_wheel && sdh_store_and_pip_install_wheel .)
+cd src
+if [ "$SAGE_EDITABLE" = yes ]; then
+    sdh_pip_editable_install .
+    if [ "$SAGE_WHEELS" = yes ]; then
+        sdh_setup_bdist_wheel && sdh_store_wheel .
+    fi
+else
+    sdh_pip_install .
+fi
 # For type=script packages, spkg-check is not run
 case "$SAGE_CHECK" in
     yes)