src/setup.py: Replace use of is_package_installed_and_updated by sage_conf or features
mkoeppe opened this issue · 11 comments
Currently (after #29701), src/setup.py determines what "distributions" to build using is_package_installed_and_updated, which inspects the installation records found in $SAGE_SPKG_INST = $SAGE_LOCAL/var/lib/sage/installed and also the information in $SAGE_PKGS = $SAGE_ROOT/build/pkgs.
Instead, we make it configurable via sage_conf, falling back to testing features.
(Note that after #30779, src/setup.py is intended to be invoked by the user for user-defined venvs, and is separate from build/pkgs/sagelib/setup.py, which is used by the Sage distribution.)
Alternative approach:
- #29119 Add an autoconf configure script for
sagelib
Depends on #30779
CC: @jdemeyer @antonio-rojas @kiwifb @dimpase @embray @timokau @isuruf @jhpalmieri
Component: build
Keywords: sd111
Issue created by migration from https://trac.sagemath.org/ticket/28815
Ticket retargeted after milestone closed
Description changed:
---
+++
@@ -1,3 +1,8 @@
-Follow-up from #25828, where the `bliss` package is handled in this way.
+Tickets:
+- #25828 (`bliss`)
+
+Alternative approaches:
+- #29701 Meta-ticket: Replace use of OptionalExtension by namespace packages
+- #29119 Add an autoconf configure script for sagelib
Description changed:
---
+++
@@ -2,7 +2,7 @@
- #25828 (`bliss`)
Alternative approaches:
-- #29701 Meta-ticket: Replace use of OptionalExtension by namespace packages
-- #29119 Add an autoconf configure script for sagelib
+- #29701 Meta-ticket: Replace use of `OptionalExtension` by namespace packages
+- #29119 Add an autoconf configure script for `sagelib`
Description changed:
---
+++
@@ -1,8 +1,10 @@
-Tickets:
-- #25828 (`bliss`)
+Currently (after #29701), `src/setup.py` determines what "distributions" to build using `is_package_installed_and_updated`, which inspects the installation records found in `$SAGE_SPKG_INST` = `$SAGE_LOCAL/var/lib/sage/installed` and also the information in `$SAGE_PKGS` = `$SAGE_ROOT/build/pkgs`.
-Alternative approaches:
-- #29701 Meta-ticket: Replace use of `OptionalExtension` by namespace packages
+Instead, we make it configurable via `sage_conf`, falling back to testing features.
+
+(Note that after #30779, `src/setup.py` is intended to be invoked by the user for user-defined venvs, and is separate from `build/pkgs/sagelib/setup.py`, which is used by the Sage distribution.)
+
+Alternative approach:
- #29119 Add an autoconf configure script for `sagelib`
Hoping we can make progress on this ticket this week - https://wiki.sagemath.org/days111
Changed keywords from none to sd111
Setting new milestone based on a cursory review of ticket status, priority, and last modification date.