Make dependencies of optional packages more detailed, no longer depend on all standard packages
mkoeppe opened this issue · 21 comments
#28095 (Add --enable-OPTIONALSPKG options to configure) made it possible to build Sage and a number of optional packages in one pass.
To improve opportunities for build parallelization, we should make dependencies of optional packages more detailed and no longer depend on all standard packages.
This will also make it faster to test builds from scratch to a specific optional package.
CC: @embray @dimpase @jhpalmieri
Component: build
Author: Matthias Koeppe
Branch: a5818c0
Reviewer: Jonathan Kliem
Issue created by migration from https://trac.sagemath.org/ticket/29426
pushing these forward to 9.2
Tests ran at https://github.com/mkoeppe/sage/actions/runs/77739151
Several optional packages that run their tests with the Sage doctester depend on ipywidgets when SAGE_CHECK=yes.
See for example https://github.com/mkoeppe/sage/runs/584777416?check_suite_focus=true:
sage_numerical_backends_coin, p_group_cohomology
New commits:
ebb07d1 | Optional/experimental packages: Remove order-only dependencies on all standard packages |
Branch pushed to git repo; I updated commit sha1. New commits:
d4773b8 | build/pkgs: Add ipywidgets as a dependency to packages using the sage doctester |
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
6b20b84 | trac 26018: embray's work rebased to 9.1.rc0 |
8e6f694 | Optional/experimental packages: Remove order-only dependencies on all standard packages |
a5818c0 | build/pkgs: Add ipywidgets as a dependency to packages using the sage doctester |
Author: Matthias Koeppe
New tests are running at https://github.com/mkoeppe/sage/actions/runs/81669565 (with #29341)
This seems to work well. Needs review
Reviewer: Jonathan Kliem
LGTM.
Thanks!
+1 thanks