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:  sage9.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/deprecateinstallscripts (Commits, GitHub, GitLab)  Commit:  c77ecb07a599287d5e6577fa87f1faf71bc5e6ef 
Dependencies:  Stopgaps: 
Description (last modified by )
This function does not belong into sagelib. If it is still needed at all, it should become a script in sagethedistribution.
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/sagedevel/M6S1bL_qU3Y/twdtVp0rBgAJ
Change History (11)
comment:1 Changed 2 years ago by
Milestone:  sage9.2 → sage9.3 

comment:2 Changed 22 months ago by
Keywords:  sd111 added 

comment:3 Changed 19 months ago by
Milestone:  sage9.3 → sage9.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
Milestone:  sage9.4 → sage9.5 

comment:5 Changed 10 months ago by
Milestone:  sage9.5 → sage9.6 

comment:6 Changed 6 months ago by
Milestone:  sage9.6 → sage9.7 

comment:7 Changed 5 months ago by
Branch:  → u/jhpalmieri/deprecateinstallscripts 

comment:8 Changed 5 months ago by
Authors:  → John Palmieri 

Commit:  → c77ecb07a599287d5e6577fa87f1faf71bc5e6ef 
Status:  new → needs_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
fromsage.misc.all
? install_scripts
is used indocker/Dockerfile
. Is this necessary?
New commits:
c77ecb0  trac 30207: deprecate install_scripts (from sage.misc.dist)

comment:10 Changed 5 months ago by
I did not do any updating of https://doc.sagemath.org/html/en/installation/source.html#installationinamultiuserenvironment, by the way.
comment:11 Changed 5 months ago by
Description:  modified (diff) 

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