#10187: Update ecl and maxima
-------------------------------------------------------------------------------+
   Reporter:  vbraun                                                           
|       Owner:  tbd                                     
       Type:  defect                                                           
|      Status:  needs_review                            
   Priority:  major                                                            
|   Milestone:  sage-4.6.1                              
  Component:  packages                                                         
|    Keywords:                                          
     Author:  Volker Braun, David Kirkby                                       
|    Upstream:  Workaround found; Bug reported upstream.
   Reviewer:  Karl-Dieter Crisman, David Kirkby, Volker Braun, Leif Leonhardy  
|      Merged:                                          
Work_issues:                                                                   
|  
-------------------------------------------------------------------------------+

Comment(by leif):

 I had (''"in principle"''^TM^) already been ok with Dave's ECL package,
 then he made further changes.

 As said before, I consider the "spaces in `SAGE_ROOT`" issue a currently
 rather minor one (as long as our `spkg-install` doesn't fail in an
 unexpected manner), since I doubt upstream (at least some other packages)
 will support that.

 However, Dave claimed he had fixed that (with a change that effectively
 changes nothing ;-) ).
 While `echo $CPPFLAGS` will almost always work, running `gcc` with spaces
 in the include path won't as is. So if at all, I'd change it to
 {{{
 #!sh
 CPPFLAGS="$CPPFLAGS -I\"$SAGE_LOCAL/include\""
 LDFLAGS="$LDFLAGS -L\"$SAGE_LOCAL/lib\""
 }}}
 which at least `gcc` does support, but not necessarily other parts of
 `configure` etc.

 ----

 There are some typos in `SPKG.txt` (IMHO minor as well):
  * s/sa'''f'''ter/safer/ (plus spurious ":w" ?)
  * s/user defined/user'''-'''defined/
  * s/overwritten, as they would have done/over'''ridden''', as they would
 have '''been'''/
  * s/it will cause'''s'''/it will cause/
  * s/Remove/Remove'''d'''/

 But `SPKG.txt` is not the ''User'' or ''Developer's Guide''...

 ----

 I cannot guarantee (nor test) the new parameters to `configure` work on
 all platforms as expected: `--with-system-gmp=yes` is IMHO (at least)
 superfluous if we also pass `--with-gmp-prefix="$SAGE_LOCAL"`.

 ----

 I'd prefer all error messages (in `spkg-install`) starting with
 ''"Error"'' to ease grepping for such, consistent with other packages.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10187#comment:113>
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.

Reply via email to