Opened 2 years ago

Last modified 5 months ago

#30207 closed enhancement

Deprecate sage.misc.dist.install_scripts — at Version 11

Reported by: Matthias Köppe Owned by:
Priority: major Milestone: sage-9.7
Component: build Keywords: sd111
Cc: Emmanuel Charpentier, John Palmieri, Julian Rüth Merged in:
Authors: John Palmieri Reviewers:
Report Upstream: N/A Work issues:
Branch: u/jhpalmieri/deprecate-install-scripts (Commits, GitHub, GitLab) Commit: c77ecb07a599287d5e6577fa87f1faf71bc5e6ef
Dependencies: Stopgaps:

Status badges

Description (last modified by Matthias Köppe)

This function does not belong into sagelib. If it is still needed at all, it should become a script in sage-the-distribution.

install_scripts is also mentioned in one place in the sage installation guide, src/doc/en/installation/source.rst. We update it.

https://groups.google.com/d/msg/sage-devel/M6S1bL_qU3Y/twdtVp0rBgAJ

Change History (11)

comment:1 Changed 2 years ago by Matthias Köppe

Milestone: sage-9.2sage-9.3

comment:2 Changed 22 months ago by Matthias Köppe

Keywords: sd111 added

Hoping we can make progress on this ticket this week - https://wiki.sagemath.org/days111

comment:3 Changed 19 months ago by Matthias Köppe

Milestone: sage-9.3sage-9.4

Sage development has entered the release candidate phase for 9.3. Setting a new milestone for this ticket based on a cursory review of ticket status, priority, and last modification date.

comment:4 Changed 15 months ago by Matthias Köppe

Milestone: sage-9.4sage-9.5

comment:5 Changed 10 months ago by Matthias Köppe

Milestone: sage-9.5sage-9.6

comment:6 Changed 6 months ago by Matthias Köppe

Milestone: sage-9.6sage-9.7

comment:7 Changed 5 months ago by John Palmieri

Branch: u/jhpalmieri/deprecate-install-scripts

comment:8 Changed 5 months ago by John Palmieri

Authors: John Palmieri
Commit: c77ecb07a599287d5e6577fa87f1faf71bc5e6ef
Status: newneeds_review

Here's a branch. I removed hg, kash, and M2 from the scripts to be installed, since those are not connected to Sage packages anymore. Questions:

  • should we remove from .dist import install_scripts from sage.misc.all?
  • install_scripts is used in docker/Dockerfile. Is this necessary?

New commits:

c77ecb0trac 30207: deprecate install_scripts (from sage.misc.dist)

comment:9 Changed 5 months ago by Matthias Köppe

Cc: Julian Rüth added

Cc'ing the maintainer of docker/Dockerfile

comment:11 Changed 5 months ago by Matthias Köppe

Description: modified (diff)
Note: See TracTickets for help on using tickets.