#7243: [with patch, needs review] bashisms in
extcode-4.1.2/pari/dokchitser/testall
-----------------------+----------------------------------------------------
   Reporter:  tabbott  |       Owner:  tbd                    
       Type:  defect   |      Status:  needs_work             
   Priority:  major    |   Milestone:  sage-4.3               
  Component:  porting  |    Keywords:                         
Work_issues:           |      Author:  Tim Abbott, Mike Hansen
   Upstream:  N/A      |    Reviewer:                         
     Merged:           |  
-----------------------+----------------------------------------------------
Changes (by drkirkby):

 * cc: david.kir...@…, mhansen (added)
  * status:  needs_review => needs_work
  * upstream:  => N/A


Comment:

 I think the

 {{{

 #!/usr/bin/env bash
 }}}

 (from memory, so might not be right)

 is better, as bash is not always installed in /bin

 {{{
 bash-2.04$ uname -a
 HP-UX hpbox B.11.11 U 9000/785 2016698240 unlimited-user license
 bash-2.04$ ls /bin/bash
 /bin/bash not found
 bash-2.04$ which bash
 /opt/OpenSource/bin/bash
 bash-2.04$ which env
 /usr/bin/env
 bash-2.04$
 }}}

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7243#comment:2>
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