#9434: Stop greping for a non-existent sage-banner
------------------------+---------------------------------------------------
Reporter: drkirkby | Owner: drkirkby
Type: defect | Status: needs_review
Priority: minor | Milestone: sage-4.6
Component: build | Keywords:
Author: | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
------------------------+---------------------------------------------------
Comment(by drkirkby):
Replying to [comment:12 leif]:
> You could use
> {{{
> #!sh
> ...
> if [ -f "$SAGE_ROOT/VERSION.txt" ]; then
> sed -i -e "1iSage version: $SAGE_VERSION, Release date:
$SAGE_RELEASE_DATE\nUpdated from $OLD_VERSION"
> else
> ...
> }}}
> to avoid {{{cat}}} ... ;-)
>
> (Perhaps omit the newline, i.e. replace it by e.g. two spaces.)
>
But that would be an even bigger mistake than to use an unnecessary cat,
as you are making use of non-POSIX options to {{{sed}}} - see
[http://www.opengroup.org/onlinepubs/009695399/utilities/sed.html POSIX
specifiction of sed] I can guarantee that will fail on Solaris and AIX and
probably other Unix systems too.
Dave
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9434#comment:19>
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.
