#11993: Fix output of `sage --version`
-----------------------------+----------------------------------------------
   Reporter:  leif           |          Owner:  leif            
       Type:  defect         |         Status:  needs_work      
   Priority:  minor          |      Milestone:  sage-4.8        
  Component:  scripts        |       Keywords:  banner sage-sage
Work_issues:                 |       Upstream:  N/A             
   Reviewer:  John Palmieri  |         Author:  Leif Leonhardy  
     Merged:                 |   Dependencies:                  
-----------------------------+----------------------------------------------
Changes (by jdemeyer):

  * status:  positive_review => needs_work


Comment:

 I disagree with makeing here the change
 {{{
 if [ "$1" = '-v' -o "$1" = '-version' -o "$1" = '--version' ]; then
 }}}
 to
 {{{
 case "$1" in
     -v|-version|--version)
 }}}

 I don't disagree with the change itself, just with the fact that
  1. you do it on this ticket
  2. only for this command line option

 If you want to make this change, please do it on a ''different'' ticket
 and then for ''all'' command-line options.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11993#comment:7>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to