Skip to content
This repository was archived by the owner on Feb 1, 2023. It is now read-only.

Commit d28b75a

Browse files
author
Matthias Koeppe
committed
Replace SAGE_STANDARD_PACKAGES by use of SAGE_OPTIONAL_INSTALLED_PACKAGES
1 parent b17e145 commit d28b75a

File tree

2 files changed

+12
-26
lines changed

2 files changed

+12
-26
lines changed

build/make/Makefile.in

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -86,17 +86,12 @@ GCC_DEP = @SAGE_GCC_DEP@
8686

8787
@SAGE_PACKAGE_DEPENDENCIES@
8888

89-
# All standard packages
90-
STANDARD_PACKAGES = @SAGE_STANDARD_PACKAGES@
91-
STANDARD_PACKAGE_INSTS = \
92-
$(foreach pkgname,$(STANDARD_PACKAGES),$(inst_$(pkgname)))
93-
94-
# All optional installed packages (triggers the auto-update)
89+
# All standard/optional/experimental installed packages (triggers the auto-update)
9590
OPTIONAL_INSTALLED_PACKAGES = @SAGE_OPTIONAL_INSTALLED_PACKAGES@
9691
OPTIONAL_INSTALLED_PACKAGE_INSTS = \
9792
$(foreach pkgname,$(OPTIONAL_INSTALLED_PACKAGES),$(inst_$(pkgname)))
9893

99-
# All previously installed optional packages that are to be uninstalled
94+
# All previously installed standard/optional/experimental packages that are to be uninstalled
10095
OPTIONAL_CLEANED_PACKAGES = @SAGE_OPTIONAL_CLEANED_PACKAGES@
10196
OPTIONAL_CLEANED_PACKAGES_CLEANS = $(OPTIONAL_CLEANED_PACKAGES:%=%-clean)
10297

@@ -211,7 +206,6 @@ base-toolchain: _clean-broken-gcc base
211206
# All targets except for the base packages
212207
all-sage: \
213208
sagelib \
214-
$(STANDARD_PACKAGE_INSTS) \
215209
$(OPTIONAL_INSTALLED_PACKAGE_INSTS) \
216210
$(OPTIONAL_CLEANED_PACKAGES_CLEANS)
217211

@@ -268,7 +262,7 @@ build-start: all-build
268262
# We make this depend on all standard packages because running
269263
# sage-starts runs sage-location, which should be run after installing
270264
# any package.
271-
$(STARTED): $(STANDARD_PACKAGE_INSTS)
265+
$(STARTED): $(OPTIONAL_INSTALLED_PACKAGE_INSTS)
272266
$(AM_V_at)"$(SAGE_ROOT)/build/bin/sage-starts"
273267

274268

m4/sage_spkg_collect.m4

Lines changed: 9 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -20,13 +20,11 @@
2020
# platform, or the dependency on them is satisfied by an existing
2121
# system package.
2222
#
23-
# - SAGE_STANDARD_PACKAGES - lists the names of all packages that have
24-
# the "standard" type. All "standard" packages are installed by
25-
# default (if they are listed in SAGE_DUMMY_PACKAGES "installed" in
26-
# this case is a no-op).
23+
# - SAGE_OPTIONAL_INSTALLED_PACKAGES - lists the names of packages with the
24+
# "standard", "optional", or "experimental" type that should be installed.
2725
#
28-
# - SAGE_OPTIONAL_PACKAGES - lists the names of packages with the
29-
# "optional" type that should be installed.
26+
# - SAGE_OPTIONAL_CLEANED_PACKAGES - lists the names of packages with the
27+
# "standard", "optional", or "experimental" type that should be installed.
3028
#
3129
# - SAGE_SDIST_PACKAGES - lists the names of all packages whose sources
3230
# need to be downloaded to be included in the source distribution.
@@ -107,9 +105,6 @@ SAGE_BUILT_PACKAGES=''
107105
# underlying system.
108106
SAGE_DUMMY_PACKAGES=''
109107
110-
# Standard packages
111-
SAGE_STANDARD_PACKAGES=''
112-
113108
# List of currently installed and to-be-installed optional packages - filled in SAGE_SPKG_ENABLE
114109
#SAGE_OPTIONAL_INSTALLED_PACKAGES
115110
# List of optional packages to be uninstalled - filled in SAGE_SPKG_ENABLE
@@ -155,26 +150,24 @@ for DIR in $SAGE_ROOT/build/pkgs/*; do
155150
base)
156151
message="came preinstalled with the SageMath tarball"
157152
;;
158-
standard)
159-
SAGE_STANDARD_PACKAGES="${SAGE_STANDARD_PACKAGES} \\$(printf '\n ')${SPKG_NAME}"
160-
in_sdist=yes
161-
message="will be installed as an SPKG"
162-
;;
163-
optional|experimental)
153+
standard|optional|experimental)
164154
AS_VAR_IF([SAGE_ENABLE_]${SPKG_NAME}, [yes], [
165155
message="$SPKG_TYPE, will be installed as an SPKG"
166156
], [
167157
message="$SPKG_TYPE, use \"$srcdir/configure --enable-$SPKG_NAME\" to install"
168158
])
169-
uninstall_message=", use \"$srcdir/configure --disable-$SPKG_NAME\" to uninstall"
170159
;;
171160
*)
172161
AC_MSG_ERROR([The content of "$SPKG_TYPE_FILE" must be 'base', 'standard', 'optional', or 'experimental'])
173162
;;
174163
esac
175164
176165
case "$SPKG_TYPE" in
166+
standard)
167+
in_sdist=yes
168+
;;
177169
optional|experimental)
170+
uninstall_message=", use \"$srcdir/configure --disable-$SPKG_NAME\" to uninstall"
178171
stampfile=""
179172
for f in "$SAGE_SPKG_INST/$SPKG_NAME"-*; do
180173
AS_IF([test -r "$f"], [
@@ -325,7 +318,6 @@ AC_SUBST([SAGE_PIP_PACKAGES])
325318
AC_SUBST([SAGE_SCRIPT_PACKAGES])
326319
AC_SUBST([SAGE_BUILT_PACKAGES])
327320
AC_SUBST([SAGE_DUMMY_PACKAGES])
328-
AC_SUBST([SAGE_STANDARD_PACKAGES])
329321
AC_SUBST([SAGE_OPTIONAL_INSTALLED_PACKAGES])
330322
AC_SUBST([SAGE_OPTIONAL_CLEANED_PACKAGES])
331323
AC_SUBST([SAGE_SDIST_PACKAGES])

0 commit comments

Comments
 (0)