# HG changeset patch
# User David Kirkby <david.kirkby@onetel.net>
# Date 1291235691 0
# Node ID daebf2c3ce1a8ec30a26a364dcd660939d6dfb10
# Parent  3c32f4059a1c22f3cd3efc15fbe7024595a222af
#10187 Document problems there may be if quoting SAGE_LOCAL.

diff -r 3c32f4059a1c -r daebf2c3ce1a SPKG.txt
--- a/SPKG.txt	Wed Dec 01 18:30:24 2010 +0000
+++ b/SPKG.txt	Wed Dec 01 20:34:51 2010 +0000
@@ -42,13 +42,17 @@
  * Delete the contents of the src/contrib/encodings/
  * TODO - Add the ECL test suite, and an spkg-check file to run it. 
  * Note, the way we configure Sage, CXX and CXXFLAGS are unused. 
+ * Do NOT quote SAGE_LOCAL when setting CPPFLAGS and/or LDFLAGS, 
+   in spkg-install as this caused the build to break. See 
+   http://trac.sagemath.org/sage_trac/ticket/10187#comment:117
+ 
 
 == Changelog ==
 
 === ecl-10.4.1 (David Kirkby, 11th November 2010) ===
   * #10187 Update to the latest stable release of ECL, which is needed
     to build on Fedora 14 (see #10185). 
-  * Addred the word "Error" in front of all error messages. 
+  * Added the word "Error" in front of all error messages. 
   * Added a TODO to add an spkg-check file. Doing this is not
     trivial. The ECL developer has said the test code on the 
     ECL site is outdated, and ECL is not really designed to 
diff -r 3c32f4059a1c -r daebf2c3ce1a spkg-install
--- a/spkg-install	Wed Dec 01 18:30:24 2010 +0000
+++ b/spkg-install	Wed Dec 01 20:34:51 2010 +0000
@@ -14,6 +14,9 @@
    CXXFLAG64=-m64
 fi
 
+# Do NOT quote SAGE_LOCAL here, as has caused problems.
+# See: http://trac.sagemath.org/sage_trac/ticket/10187#comment:117
+
 CPPFLAGS="$CPPFLAGS -I$SAGE_LOCAL/include"
 LDFLAGS="$LDFLAGS -L$SAGE_LOCAL/lib"