#14151: Update GMP-ECM to 6.4.4
--------------------------------------+-------------------------------------
       Reporter:  zimmerma            |         Owner:  tbd                     
        
           Type:  enhancement         |        Status:  closed                  
        
       Priority:  major               |     Milestone:  sage-5.10               
        
      Component:  packages: standard  |    Resolution:  fixed                   
        
       Keywords:  spkg                |   Work issues:                          
        
Report Upstream:  N/A                 |     Reviewers:  François Bissey, Paul 
Zimmermann
        Authors:  Jeroen Demeyer      |     Merged in:  sage-5.10.beta0         
        
   Dependencies:                      |      Stopgaps:                          
        
--------------------------------------+-------------------------------------

Comment (by leif):

 Replying to [comment:36 fbissey]:
 > Replying to [comment:35 zimmerma]:
 > > sorry Francois was quicker than myself. Here are my 2 cents.
 > >
 > > In SPKG.txt:
 > > {{{
 > >  * We currently work around a linker bug on MacOS X 10.5 PPC (with
 > >    GCC 4.2.1) which breaks 'configure' if debug symbols are enabled.
 > >    This *might* get fixed in later upstream releases.
 > > }}}
 > > Is that still true, since there is no patch any more?

 ''"We work around ..."'' != ''"We patch ...''", and the comment was in
 "Special !Update/Build Instructions", not its "Patches" subsection.

 [[BR]]

 > I believe it is fixed as from the description of the patch:
 > {{{
 > * configure.patch:
 >          - Disable "asm-redc" on 32-bit Darwin PPCs (upstream revision
 1516 / bug
 >            #10646).
 >            (Note that this upstream patch is likely to slow down GMP-ECM
 on 64-bit
 >            PPC CPUs running (32-bit) MacOS X, since the extended
 instruction set of
 >            the CPU is no longer exploitet.  A proper fix would just pass
 an option
 >            to Apple's assembler to allow the use of the extended
 instruction set.)
 >                  - Fix compilation error on x86 CPUs supporting SSE2.
 (Sage trac #10252,
 >            upstream revision 1546).
 > }}}
 > So there is mention of two revisions in gmp-ecm taking care of the
 problem from that patch. But good spotting, SPKG.txt needs cleaning to
 remove that statement.

 [[BR]]

 The comment referred to the following part of `spkg-install` (which is
 still in, and presumably still necessary -- perhaps ask Dima, so should
 have been kept):

 {{{
 #!sh
     case "`uname -srm | tr ' ' '-'`" in
         Darwin-9*-[Pp]ower*)
             # Don't add debug symbols because configure otherwise
             # fails due to a bus error in Apple's 'ld' when trying
             # to determine if global symbols are prefixed with an
             # underscore (cf.
 http://trac.sagemath.org/sage_trac/ticket/5847#comment:35 ff.):
             echo >&2 "Warning: Disabling debug symbols on MacOS X 10.5" \
                 "PowerPC because of a linker (?) bug."
             echo >&2 "See
 http://trac.sagemath.org/sage_trac/ticket/5847#comment:35"; \
                 "ff. for details."
             echo >&2
             CFLAGS="-O3 $CFLAGS"
             ;;
         *)
             CFLAGS="-g -O3 $CFLAGS"
     esac
 }}}

 By the way, the attached diff doesn't reflect the latest spkg version.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14151#comment:43>
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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to