#9160: Singular - change timestamp of file and sort out SAGE64 isssue.
------------------------------------------------------+---------------------
   Reporter:  drkirkby                                |       Owner:  
GeorgSWeber 
       Type:  defect                                  |      Status:  
needs_review
   Priority:  blocker                                 |   Milestone:  
sage-4.4.4  
  Component:  build                                   |    Keywords:            
  
     Author:  David Kirkby and Willem Jan Palenstijn  |    Upstream:  N/A       
  
   Reviewer:                                          |      Merged:            
  
Work_issues:                                          |  
------------------------------------------------------+---------------------

Comment(by drkirkby):

 OK, I've made some changes. I hope these are ok with you.

  * Changed the timestamp on the configure script with 'touch'
  * Moved all the 'export' commands to one place - it seemed a bit
 pointless exporting LDFLAGS and DYNAMIC_KERNEL from inside the SAGE64
 code, whereas others were exported multiple times on other lines. Now all
 variables are exported once, and only once and done in one place.
  * Changed $RM to 'rm', $CHMOD to 'chmod' etc. I've done quite a few of
 these in the past, as it makes the code easier to read.
  * The 'SAGE64' test was actually inside an 'else' part of {{{if [
 $SAGE_DEBUG = 1 ] ; then }}} bit of code, so would have not been executed
 had someone set SAGE_DEBUG to 1. As such it would have been impossible to
 debug the code in 64-bit mode on systems where SAGE64 needed to be set.
 The SAGE64 test is now done irrespective of whether debugging or not.
  * Ensured $SAGE_LOCAL/include is the first CPPFLAG, so the Sage include
 files are loaded before others. A failure to do this was causing some old
 headers on Solaris to be included, instead of the newer ones in Sage. (The
 same is happening with mathplotlib I believe)
  * Removed "-O2 -g -m64" from CPPFLAGS, as these should not be passed to
 the pre-processor, but the compiler. They should go into CFLAGS and
 CXXFLAGS, not CPPFLAGS.
  * I did '''not''' change DYNAMIC_KERNEL to be only used on OS X (Darwin),
 as the Singular documentation specifically says this should be used on
 Solaris.

 I've tested on Solaris 32-bit (SPARC) on several systems, Linux and OS X -
 all work fine. It does not work fully on !OpenSolaris x64, as the CFLAGS
 and CXXFLAGS are not getting passed every time the compiler is invoked -
 there are about 6 cases where the flag does not get passed. This needs to
 be resolved at a later date.

 Hopefully the package is a bit cleaner now. It's still a mess, but it's a
 start to a cleaner build system.

 The updated package is in the original place -
 http://boxen.math.washington.edu/home/kirkby/patches/singular-3.1.0.4.p6.spkg
 An updated patch file is attached to the ticket.

 Test results are below for one Solaris system, one Linux system and one OS
 X system.

 Dave

  == On bsd.math (OS X) ==
 {{{
 Successfully installed singular-3.1.0.4.p6
 Now cleaning up tmp files.
 Making Sage/Python scripts relocatable...
 Making script relocatable
 Finished installing singular-3.1.0.4.p6.spkg
 [kir...@bsd sage-4.4.2]$ uname -a
 Darwin bsd.local 10.3.0 Darwin Kernel Version 10.3.0: Fri Feb 26 11:58:09
 PST 2010; root:xnu-1504.3.12~1/RELEASE_I386 i386 i386 MacPro1,
 1 Darwin
 [kir...@bsd sage-4.4.2]$
 }}}

  == On sage.math (Linux) ==
 {{{
 Successfully installed singular-3.1.0.4.p6
 Now cleaning up tmp files.
 Making Sage/Python scripts relocatable...
 Making script relocatable
 Finished installing singular-3.1.0.4.p6.spkg
 kir...@sage:~/sage-4.4.2$ uname -a
 Linux sage.math.washington.edu 2.6.24-26-server #1 SMP Tue Dec 1 18:26:43
 UTC 2009 x86_64 GNU/Linux
 }}}

  == On t2.math (Solaris 10 on SPARC) ==
 {{{
 sys     4m38.647s
 Successfully installed singular-3.1.0.4.p6
 Now cleaning up tmp files.
 rm: Cannot remove any directory in the path of the current working
 directory
 /rootpool2/local/kirkby/sage-4.4.3/spkg/build/singular-3.1.0.4.p6
 Making Sage/Python scripts relocatable...
 Making script relocatable
 Finished installing singular-3.1.0.4.p6.spkg
 }}}

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