Opened 3 years ago

Last modified 13 months ago

#21569 new enhancement

Install src/bin/* scripts via setup.py (scripts, console_scripts)

Reported by: mkoeppe Owned by:
Priority: major Milestone: sage-7.4
Component: build Keywords:
Cc: was, vbraun, jdemeyer, embray, fbissey, leif, thansen, saraedum Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by mkoeppe)

Currently the scripts in src/bin are installed by build/make/Makefile.

As part of #21507, for those scripts that should be considered part of sagelib, the installation should be done by src/setup.py instead, using the scripts and console-scripts features.

(Example, received from was: ​https://github.com/sagemathinc/smc/tree/master/src/smc_pyutil. See, in particular, the console_scripts section of setup.py: ​https://github.com/sagemathinc/smc/blob/master/src/smc_pyutil/setup.py#L58)

(Those scripts that don't belong into src/bin are moved to build/bin by #21559. See also: #21570: Move non-scripts of src/bin/ elsewhere (and also move their install location; #21707: Split sage-env into sage-build-env and sage-env)

Change History (8)

comment:1 Changed 3 years ago by mkoeppe

  • Description modified (diff)

comment:2 Changed 3 years ago by mkoeppe

  • Type changed from PLEASE CHANGE to enhancement

comment:3 Changed 3 years ago by mkoeppe

  • Description modified (diff)

comment:4 Changed 3 years ago by embray

+1

comment:5 Changed 3 years ago by mkoeppe

Erik, Jeroen - should this be done on top of #21604 and #21600; or does it have to wait for other tickets?

comment:6 Changed 3 years ago by embray

Normally this shouldn't be dependent on those tickets, and I can't think of any reason it would be. But I've come not to expect normalcy with sage.

comment:7 Changed 3 years ago by mkoeppe

  • Description modified (diff)

comment:8 Changed 13 months ago by mkoeppe

  • Cc thansen saraedum added

In #22731 comment 28, it is proposed to move scripts to $SAGE_LOCAL/libexec. Could a setuptools expert comment whether setup.sh would be able to do this, or can it only install into bin (via the scripts and console_scripts features)?

Note: See TracTickets for help on using tickets.