#13389: Minor fix to LiE optional SPKG
-------------------------------------+--------------------------------------
Reporter: kini | Owner: tbd
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-5.3
Component: optional packages | Resolution:
Keywords: | Work issues:
Report Upstream: N/A | Reviewers:
Authors: Keshav Kini | Merged in:
Dependencies: | Stopgaps:
-------------------------------------+--------------------------------------
Comment (by jhpalmieri):
Also, the use of `-i` with sed is not portable, and breaks installation on
Solaris: the file `local/bin/lie` still refers to the build directory.
Something like this will fix it:
{{{
#!diff
diff --git a/spkg-install b/spkg-install
--- a/spkg-install
+++ b/spkg-install
@@ -22,7 +22,8 @@ make CC="$CC" || die "Error building LiE
# relocating
cd ..
-sed -i -e "s'$PWD/src'$SAGE_LOCAL/lib/LiE'" src/lie
+sed -e "s'$PWD/src'$SAGE_LOCAL/lib/LiE'" src/lie > src/lie_new
+mv src/lie_new src/lie
rm -rf "$SAGE_LOCAL"/lib/lie # clean up old versions
rm -rf "$SAGE_LOCAL"/bin/lie "$SAGE_LOCAL"/lib/LiE
mv src/lie "$SAGE_LOCAL"/bin/
}}}
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13389#comment:3>
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.
