Ticket #9527 (closed enhancement: fixed)
improve "sage -sh" prompt so it doesn't confuse everybody
| Reported by: | was | Owned by: | jason |
|---|---|---|---|
| Priority: | minor | Milestone: | sage-4.5.2 |
| Component: | misc | Keywords: | |
| Cc: | malb | Work issues: | |
| Report Upstream: | N/A | Reviewers: | Martin Albrecht |
| Authors: | William Stein | Merged in: | sage-4.5.2.alpha1 |
| Dependencies: | Stopgaps: |
Description
When doing sage -sh, the result is very, very confusing, since it has the SAGE_ROOT path displayed unlabeled, which causes confusion, and does not show the current path, which also causes confusion. This patch adds a label and the current path, to help reduce confusion some.
Attachments
Change History
comment:2 follow-up: ↓ 3 Changed 3 years ago by malb
- Status changed from needs_review to positive_review
- Reviewers set to Martin Albrecht
- Authors set to William Stein
The patch looks fine. I'd have preferred to have my PS1 to be embedded in the new PS1, i.e. something like this:
this_is_my_prompt$ sage -sh SAGE_ROOT=/usr/local/sage-4.3 (sage subshell) this_is_my_prompt$
But it seems this just doesn't work (I tried). This new one is definitely an improvement thus I'll give it a positive review.
comment:3 in reply to: ↑ 2 Changed 3 years ago by was
Replying to malb:
The patch looks fine. I'd have preferred to have my PS1 to be embedded in the new PS1, i.e. something like this:
this_is_my_prompt$ sage -sh SAGE_ROOT=/usr/local/sage-4.3 (sage subshell) this_is_my_prompt$But it seems this just doesn't work (I tried).
I tried too, but couldn't figure it out. This patch is meant to be a big improvement, that's all.

