#18344: fix some case issues in sage-fix-pkg-checksums
-------------------------+-------------------------------------------------
Reporter: | Owner:
jhpalmieri | Status: positive_review
Type: | Milestone: sage-6.7
defect | Resolution:
Priority: major | Merged in:
Component: build | Reviewers: Leif Leonhardy
Keywords: | Work issues:
Authors: John | Commit:
Palmieri | 02ba7045cf38725ca8cb5456df649b6bacdcbd8e
Report Upstream: N/A | Stopgaps:
Branch: |
u/jhpalmieri/case |
Dependencies: |
-------------------------+-------------------------------------------------
Comment (by leif):
... we could at least add
{{{
#!sh
if [ -z "$SAGE_ROOT" ]; then
if [ -d upstream ]; then # probably add further sanity checks
SAGE_ROOT=`pwd`
else
echo >&2 "Error: SAGE_ROOT not set."
exit 1
fi
fi
}}}
Otherwise the script does nothing and exits without an error, even when
we're in the Sage root directory but `SAGE_ROOT` isn't set (just because
`[ -d /build/pkgs/... ]` usually doesn't exist).
--
Ticket URL: <http://trac.sagemath.org/ticket/18344#comment:6>
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 unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
