Skip to content

Commit 00ab9a3

Browse files
author
Matthias Koeppe
committed
build/bin/sage-get-system-packages: Add/update comments
1 parent 04dfcfc commit 00ab9a3

File tree

1 file changed

+9
-3
lines changed

1 file changed

+9
-3
lines changed

build/bin/sage-get-system-packages

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -49,18 +49,24 @@ for PKG_BASE in $SPKGS; do
4949

5050
case "$SYSTEM:$ENABLE_SYSTEM_SITE_PACKAGES" in
5151
install-requires*|pip*)
52+
# This is output for installation of packages into a Python environment.
53+
# So it's OK to use any Python packages.
5254
;;
5355
*:)
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).
56+
# --enable-system-site-packages was NOT passed to configure
57+
# (or script was run outside of the sage-build-env).
58+
#
59+
# Skip this package if it uses the SAGE_PYTHON_PACKAGE_CHECK macro.
60+
#
5761
SPKG_CONFIGURE="${SAGE_ROOT}/build/pkgs/${PKG_BASE}/spkg-configure.m4"
5862
if grep -q SAGE_PYTHON_PACKAGE_CHECK "${SPKG_CONFIGURE}" 2>/dev/null;
5963
then
6064
continue
6165
fi
6266
;;
6367
*)
68+
# --enable-system-site-packages was passed to configure.
69+
# So any package (Python, non-Python) is fine.
6470
;;
6571
esac
6672

0 commit comments

Comments
 (0)