#8059: update Singular SPKG to newest upstream release
-------------------------------+--------------------------------------------
   Reporter:  malb             |       Owner:                                   
         
       Type:  enhancement      |      Status:  needs_work                       
         
   Priority:  major            |   Milestone:  sage-4.5                         
         
  Component:  packages         |    Keywords:                                   
         
     Author:  Martin Albrecht  |    Upstream:  Fixed upstream, in a later 
stable release.
   Reviewer:                   |      Merged:                                   
         
Work_issues:                   |  
-------------------------------+--------------------------------------------

Comment(by drkirkby):

 Replying to [comment:53 malb]:
 > I've updated the Singular SPKG to 3-1-1-3 which fixes the genus issue.
 However, this release segfaults a lot!

 Hi Martin,

 do you see this warning message about {{{warning: returning reference to
 temporary}}}? I got that warnings when building on !OpenSolaris - see
 below.

 {{{
 g++ -m64 -O2 -g -m64 -fPIC -pipe -fno-implicit-templates -I. -I../kernel
 -I/export/home/drkirkby/sage-4.5.alpha4/local/include
 -I/export/home/drkirkby/sage-4.5.alpha4/local/include
 -I/export/home/drkirkby/sage-4.5.alpha4/local/include -O2 -g -m64 -DNDEBUG
 -DOM_NDEBUG -Dix86_SunOS -DHAVE_CONFIG_H -c iparith.cc
 In file included from ../kernel/ring.h:13,
                  from subexpr.h:16,
                  from ipid.h:13,
                  from iparith.cc:20:
 ../kernel/polys-impl.h:177:1: warning: "pPolyAssumeReturn" redefined
 ../kernel/polys-impl.h:176:1: warning: this is the location of the
 previous definition
 In file included from ../kernel/walkMain.h:5,
                  from ../kernel/walkProc.h:3,
                  from iparith.cc:56:
 ../kernel/int64vec.h: In member function 'const int&
 int64vec::operator[](int) const':
 ../kernel/int64vec.h:51: warning: returning reference to temporary
 }}}

 That sounds the sort of error which could result in segfaults.

 I'm attaching a patch which gives much improved error checking.

 Dave

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