Opened 17 months ago

Last modified 2 weeks ago

#29386 new enhancement

Install script packages via sage-spkg — at Version 12

Reported by: mkoeppe Owned by:
Priority: major Milestone: sage-9.5
Component: build Keywords:
Cc: dimpase, jhpalmieri, embray, mjo Merged in:
Authors: Matthias Koeppe Reviewers:
Report Upstream: N/A Work issues:
Branch: u/mkoeppe/install_script_packages_via_sage_spkg (Commits, GitHub, GitLab) Commit: 3d6fa5924f54b85a28f5c4d969afca01da8801fc
Dependencies: #29793, #30657 Stopgaps:

Status badges

Description (last modified by mkoeppe)

Follow-up from #29287.

Script packages are installed by special rules in build/make/Makefile.

This ticket installs them by sage-spkg instead. This enables script packages to participate in the staged installs (SAGE_DESTDIR) and uninstallation mechanisms.

This is useful for #29082... to clean up #27870/#29003, which introduced files in SAGE_LOCAL that do not belong to any package and are never uninstalled.

We also make the decision whether spkg-install is generated from a template or provided as is in the source tree orthogonal to the distinction source=script vs. source=pip. We simply generate spkg-install if is present.

We also rename targets SPKG-clean to SPKG-uninstall (as originally discussed in #29097), leaving a deprecated target behind. This is to unify the targets between normal and script packages (the latter used -uninstall already).

Change History (12)

comment:1 Changed 16 months ago by mkoeppe

  • Milestone changed from sage-9.1 to sage-9.2

pushing these forward to 9.2

comment:2 Changed 14 months ago by mkoeppe

  • Dependencies set to #29793

comment:3 Changed 14 months ago by mkoeppe

  • Description modified (diff)

comment:4 Changed 13 months ago by mkoeppe

  • Cc mjo added

comment:5 Changed 12 months ago by mkoeppe

  • Milestone changed from sage-9.2 to sage-9.3

comment:6 Changed 11 months ago by embray

+1 this would be a very nice improvement.

comment:7 Changed 10 months ago by mkoeppe

  • Branch set to u/mkoeppe/install_script_packages_via_sage_spkg

comment:8 Changed 10 months ago by mkoeppe

  • Commit set to fee5bc2c04387745dbb9b8265ec13be7bcd434e1
  • Dependencies changed from #29793 to #29793, #30657

New commits:

f0f03c9build/bin/sage-spkg: Remove dead code for manufacturing an spkg-install script
46e5f68build/bin/sage-spkg: Get rid of USE_LOCAL_SCRIPTS (it was always yes); prepare for script packages
fee5bc2build/bin/sage-spkg: For script packages, do not poison the proxy variables

comment:9 Changed 10 months ago by git

  • Commit changed from fee5bc2c04387745dbb9b8265ec13be7bcd434e1 to e4c27f6f307c06d6a1c9fc368cb72a461e4c6b20

Branch pushed to git repo; I updated commit sha1. Last 10 new commits:

9ee2110build/bin/sage-dist-helpers: Also use $sudo for storing the wheel file
d7aac84src/doc/en/developer/packaging.rst: Update sdh_... documentation
9b7c7a0build/bin/sage-pip-{install,uninstall}: Fix typo in comment
4135e8bbuild/bin/sage-pip-install: Remove an outdated comment
f2e7075Merge tag '9.2.beta13' into t/29500/install_all_python_packages_via_pip_wheel__create_pep_503_simple_repository_for_wheels
bd0fd2fMerge branch 't/29500/install_all_python_packages_via_pip_wheel__create_pep_503_simple_repository_for_wheels' into t/30657/fix_up__sage__p_
2e20f23build/bin/sage-spkg: Move SAGE_SPKG_{INST,SCRIPTS,WHEELS} setting here from build/bin/install, make them relative to SAGE_INST_LOCAL
ac09b14Merge branch 't/30657/fix_up__sage__p_' into t/29386/install_script_packages_via_sage_spkg
bf0c3dcbuild/bin/sage-spkg: Exit with error for some old-style script invocations
e4c27f6build/bin/sage-spkg: Remove error message when checksums.ini does not exist (script package)

comment:10 Changed 10 months ago by git

  • Commit changed from e4c27f6f307c06d6a1c9fc368cb72a461e4c6b20 to 81a63df3107793d067f1bd5a9807bfff8c0cd054

Branch pushed to git repo; I updated commit sha1. New commits:

3ceeaa7build/bin/sage-spkg: Fix up for normal and script packages
8c4fb4dbuild/bin/sage-spkg: Handle script packages without package-version.txt
137365abuild/make/ Replace 'SPKG-clean' by 'SPKG-uninstall', with deprecation
a9cbfdabuild/make/ Install script packages using sage-spkg
81a63dfsrc/bin/sage: Use 'make SPKG-uninstall' instead of -clean

comment:11 Changed 10 months ago by git

  • Commit changed from 81a63df3107793d067f1bd5a9807bfff8c0cd054 to 3d6fa5924f54b85a28f5c4d969afca01da8801fc

Branch pushed to git repo; I updated commit sha1. New commits:

3d6fa59build/make/ Replace 'SPKG-clean' by 'SPKG-uninstall', with deprecation, for pip packages too

comment:12 Changed 10 months ago by mkoeppe

  • Authors set to Matthias Koeppe
  • Description modified (diff)
Note: See TracTickets for help on using tickets.