#10303: clean up sage-check-64 and use of SAGE64
-----------------------------+----------------------------------------------
   Reporter:  jhpalmieri     |       Owner:  GeorgSWeber 
       Type:  defect         |      Status:  needs_review
   Priority:  minor          |   Milestone:  sage-4.6.1  
  Component:  build          |    Keywords:  64          
     Author:  John Palmieri  |    Upstream:  N/A         
   Reviewer:                 |      Merged:              
Work_issues:                 |  
-----------------------------+----------------------------------------------

Comment(by leif):

 Replying to [comment:6 jhpalmieri]:
 > Here's a new version of the "v2" patch.

 Hmmm, now we get ''"32-bit build, so writing 'no' to
 SAGE_LOCAL/lib/sage-64.txt."'' on every non-OS X / non-[Open]Solaris
 system...

 (And if `SAGE64` is not set, we usually read the flag file twice.)

 [[BR]]


 > > SAGE64=yes should simply only be used on 64-bit MacOS X and
 [Open]Solaris; we could give an error message on 32-bit systems, I think
 no need to give a warning on 64-bit Linuces.
 >
 > I don't know how to check for 32-bit systems in a shell script, so I'm
 not doing this, just checking for "Darwin" or "SunOS".

 You could check if {{{... `uname -m` in i[3456]86|ppc) ...}}}, Dave will
 know what in addition to look for on [Open]Solaris (e.g. "x86") and
 especially SPARC.

 (On Darwin PPCs, `uname -m` might report more weird "processor names" like
 "Power Macintosh", but that's IMHO a minor issue since the build would
 fail later on 32-bit OSs anyway if we pass `-m64`. We do not support
 64-bit builds on PowerPCs at all, so looking for
 `i[3456]86|ppc*|[Pp]ower*` would be ok, too.)

 ----

 If you edit `sage-check-64` again, you could remove the persistent
 superfluous semicolon from `echo "SAGE_LOCAL undefined ... exiting";` ;-)
 and replace {{{`uname`}}} by `"$UNAME"`.

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