Skip to content

Commit 04dfcfc

Browse files
author
Matthias Koeppe
committed
build/bin/sage-get-system-packages (install-requires[-toml], pip): Do not require ENABLE_SYSTEM_SITE_PACKAGES=yes
1 parent e249bef commit 04dfcfc

File tree

2 files changed

+18
-12
lines changed

2 files changed

+18
-12
lines changed

bootstrap

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -108,8 +108,8 @@ EOF
108108
if test -f "$DIR/requirements.txt" -o -f "$DIR/install-requires.txt"; then
109109
# A Python package
110110
SPKG_TREE_VAR=SAGE_VENV
111-
echo "define(>>>SPKG_INSTALL_REQUIRES_${pkgname}<<<, >>>$(echo $(ENABLE_SYSTEM_SITE_PACKAGES=yes sage-get-system-packages install-requires ${pkgname}))<<<)dnl" >> m4/sage_spkg_versions.m4
112-
echo "define(>>>SPKG_INSTALL_REQUIRES_${pkgname}<<<, >>>$(echo $(ENABLE_SYSTEM_SITE_PACKAGES=yes sage-get-system-packages install-requires-toml ${pkgname}))<<<)dnl" >> m4/sage_spkg_versions_toml.m4
111+
echo "define(>>>SPKG_INSTALL_REQUIRES_${pkgname}<<<, >>>$(echo $(sage-get-system-packages install-requires ${pkgname}))<<<)dnl" >> m4/sage_spkg_versions.m4
112+
echo "define(>>>SPKG_INSTALL_REQUIRES_${pkgname}<<<, >>>$(echo $(sage-get-system-packages install-requires-toml ${pkgname}))<<<)dnl" >> m4/sage_spkg_versions_toml.m4
113113
fi
114114
fi
115115
spkg_finalizes="$spkg_finalizes

build/bin/sage-get-system-packages

Lines changed: 16 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -47,16 +47,22 @@ case "$SYSTEM" in
4747
esac
4848
for PKG_BASE in $SPKGS; do
4949

50-
# Skip this package if it uses the SAGE_PYTHON_PACKAGE_CHECK
51-
# macro and if --enable-system-site-packages was NOT passed
52-
# to ./configure (or if ./configure has not yet been run).
53-
SPKG_CONFIGURE="${SAGE_ROOT}/build/pkgs/${PKG_BASE}/spkg-configure.m4"
54-
if [ -z "${ENABLE_SYSTEM_SITE_PACKAGES}" ]; then
55-
if grep -q SAGE_PYTHON_PACKAGE_CHECK "${SPKG_CONFIGURE}" 2>/dev/null;
56-
then
57-
continue;
58-
fi
59-
fi
50+
case "$SYSTEM:$ENABLE_SYSTEM_SITE_PACKAGES" in
51+
install-requires*|pip*)
52+
;;
53+
*:)
54+
# Skip this package if it uses the SAGE_PYTHON_PACKAGE_CHECK
55+
# macro and if --enable-system-site-packages was NOT passed
56+
# to ./configure (or if ./configure has not yet been run).
57+
SPKG_CONFIGURE="${SAGE_ROOT}/build/pkgs/${PKG_BASE}/spkg-configure.m4"
58+
if grep -q SAGE_PYTHON_PACKAGE_CHECK "${SPKG_CONFIGURE}" 2>/dev/null;
59+
then
60+
continue
61+
fi
62+
;;
63+
*)
64+
;;
65+
esac
6066

6167
for NAME in $SYSTEM_PACKAGES_FILE_NAMES; do
6268
SYSTEM_PACKAGES_FILE="$SAGE_ROOT"/build/pkgs/$PKG_BASE/$NAME

0 commit comments

Comments
 (0)