Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change handling of 'sage --APP' #29930

Open
jhpalmieri opened this issue Jun 21, 2020 · 16 comments
Open

Change handling of 'sage --APP' #29930

jhpalmieri opened this issue Jun 21, 2020 · 16 comments

Comments

@jhpalmieri
Copy link
Member

Rather than handling sage --gp, sage --singular, etc. as separate if blocks in the sage script, treat them more uniformly.

Component: scripts

Branch/Commit: u/jhpalmieri/deprecate-sage-APP @ 0ad0e4f

Issue created by migration from https://trac.sagemath.org/ticket/29930

@jhpalmieri jhpalmieri added this to the sage-9.2 milestone Jun 21, 2020
@jhpalmieri
Copy link
Member Author

comment:1

Should we also deprecate these commands? Right now:

sage --gap ARGS

If we deprecate, is the alternative

sage --sh -c "gap ARGS"

? I think the quotes are necessary. Is this too unwieldy, or are we happy to essentially discourage people from using these commands at all?

@jhpalmieri
Copy link
Member Author

Branch: u/jhpalmieri/deprecate-sage-APP

@jhpalmieri
Copy link
Member Author

comment:3

Here's a branch to start the conversation.


Last 10 new commits:

885092esrc/doc/bootstrap: Generate src/doc/en/reference/repl/options.txt
05ba86aMerge branch 't/29884/src_doc_bootstrap__generate_src_doc_en_reference_repl_options_rst' into t/29111/specify_a_subset_of_sage_command_line_options_that_are_supported_by_sagelib___rather_than_sage_the_distribution
72c7e23src/doc/en/reference/repl/options.rst: Replace copypasta by include of generated file options.txt
e9a883dtrac 29111: revising "sage --advanced" message.
5dca421trac 29111: more reorganization of "sage --advanced" message
8f0ee05trac 29111: delete sage-fix-pkg-checksums
21fc231trac 29111: re "sage --advanced" message:
875940dtrac 29111: more tinkering. Fix doctests in tests/cmdline.py.
4a3d36eMove 'sage -app' back to src/bin/sage
9dab957trac 29930: change handling of 'sage --APP', and deprecate this usage.

@jhpalmieri
Copy link
Member Author

Commit: 9dab957

@jhpalmieri
Copy link
Member Author

comment:4

axiom is now removed at #29111.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 22, 2020

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

3a0193csrc/bin/sage: Remove handling of 'sage -axiom'
0dca2f9trac 29930: change handling of 'sage --APP', and deprecate this usage.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 22, 2020

Changed commit from 9dab957 to 0dca2f9

@mkoeppe
Copy link
Contributor

mkoeppe commented Aug 13, 2020

comment:6

needs rebase

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 20, 2020

Changed commit from 0dca2f9 to 92888fd

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 20, 2020

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

92888fdtrac 29930: change handling of 'sage --APP',

@jhpalmieri
Copy link
Member Author

comment:8

Here's a new version. Not a strict rebase: the earlier version deprecated commands like sage --gap, while this one keeps them. It keeps those for standard packages, deprecates their usage for optional and experimental packages like sage --M2. It also removes sage --gdb-ipython, which looks like it's been broken for a while.

@mkoeppe mkoeppe modified the milestones: sage-9.2, sage-9.3 Sep 5, 2020
@mkoeppe
Copy link
Contributor

mkoeppe commented Sep 14, 2020

comment:10

See #30563 regarding sage -maxima and what is necessary from the viewpoint of distributions

@mkoeppe
Copy link
Contributor

mkoeppe commented Feb 13, 2021

comment:11

Setting new milestone based on a cursory review of ticket status, priority, and last modification date.

@mkoeppe mkoeppe modified the milestones: sage-9.3, sage-9.4 Feb 13, 2021
@mkoeppe mkoeppe modified the milestones: sage-9.4, sage-9.5 Jul 19, 2021
@mkoeppe mkoeppe modified the milestones: sage-9.5, sage-9.6 Dec 14, 2021
@mkoeppe mkoeppe modified the milestones: sage-9.6, sage-9.7 Apr 1, 2022
@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 1, 2022

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

0ad0e4ftrac 29930: change handling of "sage --APP" for various apps

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 1, 2022

Changed commit from 92888fd to 0ad0e4f

@jhpalmieri
Copy link
Member Author

comment:16

Rebased. Still don't know if this a good approach or worth spending much time on.

@mkoeppe mkoeppe modified the milestones: sage-9.7, sage-9.8 Sep 19, 2022
@mkoeppe mkoeppe removed this from the sage-9.8 milestone Jan 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants