#10339: Simplify spkg/pipestatus
------------------------+---------------------------------------------------
   Reporter:  jdemeyer  |       Owner:  GeorgSWeber        
       Type:  defect    |      Status:  new                
   Priority:  major     |   Milestone:  sage-4.6.1         
  Component:  build     |    Keywords:  pipestatus Makefile
     Author:            |    Upstream:  N/A                
   Reviewer:            |      Merged:                     
Work_issues:            |  
------------------------+---------------------------------------------------

Comment(by leif):

 Replying to [comment:11 jdemeyer]:
 > How did a discussion of pipestatus turn in a discussion about a bash
 spkg?

 That's actually a continued discussion...

 > The point is, it is easy to implement something like pipestatus on OS X
 10.4 ''without'' a bash spkg, so what's the problem?

 Go ahead. (If it doesn't make worse what we currently have, which is
 really suboptimal).

 But it's obviously the most easiest thing to simply use a useful feature
 of a shell we already require anyway.

 I'd say define a variable name (e.g. `PIPE_STATUS`) for what makes up
 `pipestatus` and then let `$PIPE_STATUS cmd1 cmd2` either call a Bash-3.0
 function (using the `pipefail` shell option) or call a `pipestatus`
 script, from which we can remove the first case. Of course the second
 implementation could also be made into a shell function, though more
 complex.

 As mentioned elsewhere, I would also move the logging (`tee`s) from `deps`
 to `sage-spkg`, which simplifies things.

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