#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:6 jdemeyer]:
> Replying to [comment:1 leif]:
> > IMHO Bash 3.0 is old enough to make it a prerequisite anyway.
>
> -1 to this idea, since OS X 10.4 has bash version 2.
I don't care about it since it is dead old and only causes problems. If
someone is able to build Sage on MacOS X 10.4, Bash 3.x or 4.x should also
build, be it an optional Sage package for boot-strapping.
> The question I posed in the ticket remains: why should there be two
cases (the second case should work always, no matter the bash version).
Having both in `pipestatus` doesn't make ''much'' sense. As said, if at
all I would make a distinction in the scripts where `pipestatus` is needed
/ called, and / or use a variable for the command, which could simply be a
shell function. (`spkg/standard/deps` e.g. should IMHO be ''generated''
anyway.)
> By the way: fixing `eval "$1 | $2"` is easy: `eval "$1" | eval "$2"`.
Sure.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10339#comment:8>
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.
