Opened 9 years ago

Closed 8 years ago

#13996 closed enhancement (fixed)

Clean up spkg/bin/sage

Reported by: jdemeyer Owned by: leif
Priority: minor Milestone: sage-5.7
Component: scripts Keywords:
Cc: jhpalmieri Merged in: sage-5.7.beta2
Authors: Jeroen Demeyer Reviewers: Volker Braun
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #5155, #12719 Stopgaps:

Status badges

Description (last modified by jdemeyer)

  1. There is code in spkg/bin/sage which doesn't serve any purpose anymore, or options which aren't supported anymore. Clean this up.
  1. Rename the shell function sage() to interactive_sage() to minimize confusion (we already have $SAGE_ROOT/sage and $SAGE_ROOT/spkg/bin/sage)
  1. Maximize the use of exec.
  1. Change "python foo.py" to "foo.py"
  1. After installing new packages, run sage-location.

Some parts have been left untouched to avoid merge conflicts with existing patches such as #12719 and #12415.

Apply:

  1. 13996_cleanup_sage.patch to $SAGE_ROOT
  2. 13996_scripts_cleanup.patch to $SAGE_LOCAL/bin
  3. 13996_help.patch to devel/sage

Attachments (3)

13996_help.patch (1.5 KB) - added by jdemeyer 8 years ago.
13996_scripts_cleanup.patch (1.9 KB) - added by jdemeyer 8 years ago.
13996_cleanup_sage.patch (16.8 KB) - added by jdemeyer 8 years ago.

Download all attachments as: .zip

Change History (16)

comment:1 Changed 9 years ago by jdemeyer

  • Description modified (diff)

comment:2 Changed 9 years ago by jdemeyer

  • Description modified (diff)

comment:3 Changed 9 years ago by jdemeyer

  • Description modified (diff)

comment:4 Changed 9 years ago by jdemeyer

  • Description modified (diff)
  • Summary changed from Remove obsolete stuff from spkg/bin/sage to Clean up spkg/bin/sage

comment:5 Changed 9 years ago by jdemeyer

  • Description modified (diff)

comment:6 Changed 9 years ago by jdemeyer

  • Dependencies set to #5155
  • Description modified (diff)

comment:7 Changed 9 years ago by jdemeyer

  • Description modified (diff)

comment:8 Changed 9 years ago by jhpalmieri

  • Cc jhpalmieri added

comment:9 Changed 8 years ago by jdemeyer

  • Description modified (diff)

Changed 8 years ago by jdemeyer

comment:10 Changed 8 years ago by jdemeyer

  • Status changed from new to needs_review

Changed 8 years ago by jdemeyer

Changed 8 years ago by jdemeyer

comment:11 Changed 8 years ago by jdemeyer

  • Dependencies changed from #5155 to #5155, #12719

comment:12 Changed 8 years ago by vbraun

  • Reviewers set to Volker Braun
  • Status changed from needs_review to positive_review

Looks good to me.

comment:13 Changed 8 years ago by jdemeyer

  • Merged in set to sage-5.7.beta2
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.