#3999: [with patch, needs work] Wrapper class to treat additive groups as
multiplicative goups
------------------------------+---------------------------------------------
 Reporter:  robertwb          |        Owner:  somebody  
     Type:  defect            |       Status:  new       
 Priority:  major             |    Milestone:  sage-3.2.2
Component:  basic arithmetic  |   Resolution:            
 Keywords:                    |  
------------------------------+---------------------------------------------
Changes (by was):

  * summary:  [with patch, needs review] Wrapper class to treat additive
              groups as multiplicative goups => [with patch,
              needs work] Wrapper class to treat additive
              groups as multiplicative goups

Comment:

 REFEREE REPORT:

 This bitrotted.  I couldn't apply cleanly because of setup.py being
 refactored.  I fixed that by hand, but then this wouldn't compile:
 {{{
 [EMAIL PROTECTED]:~/build/sage-3.2.1.alpha1$ ./sage -br

 ----------------------------------------------------------
 sage: Building and installing modified Sage library files.


 Installing c_lib
 scons: `install' is up to date.
 Updating Cython code....
 Building modified file sage/groups/multiplicative_wrapper.pyx.
 Execute 1 commands (using 1 cpus)
 python2.5 `which cython` --embed-positions --incref-local-binop
 -I/home/was/build/sage-3.2.1.alpha1/devel/sage-main -o
 sage/groups/multiplicative_wrapper.c
 sage/groups/multiplicative_wrapper.pyx

 Error converting Pyrex file to C:
 ------------------------------------------------------------
 ...
         except:
             if x == 1:
                 return self.one
             raise

     cpdef bint _has_coerce_map_from_(self, S) except -2:
          ^
 ------------------------------------------------------------

 /home/was/build/sage-3.2.1.alpha1/devel/sage-
 main/sage/groups/multiplicative_wrapper.pyx:113:10: C method
 '_has_coerce_map_from_' not previously declared in definition part of
 extension type

 Error converting Pyrex file to C:
 ------------------------------------------------------------
 ...
         cdef MultWrapperElement e =
 <MultWrapperElement>PY_NEW(MultWrapperElement)
         e._parent = self._parent
         e._elt = elt
         return e

     cdef MonoidElement _mul_c_impl(self, MonoidElement right):
         ^
 ------------------------------------------------------------

 /home/was/build/sage-3.2.1.alpha1/devel/sage-
 main/sage/groups/multiplicative_wrapper.pyx:191:9: C method '_mul_c_impl'
 not previously declared in definition part of extension type

 Error converting Pyrex file to C:
 ------------------------------------------------------------
 ...
         return e

     cdef MonoidElement _mul_c_impl(self, MonoidElement right):
         return
 self._new(self._elt._add_c((<MultWrapperElement>right)._elt))

     cdef MultiplicativeGroupElement _div_c_impl(self,
 MultiplicativeGroupElement right):
         ^
 ------------------------------------------------------------

 /home/was/build/sage-3.2.1.alpha1/devel/sage-
 main/sage/groups/multiplicative_wrapper.pyx:194:9: C method '_div_c_impl'
 not previously declared in definition part of extension type
 Parallel build failed with status 256.
 sage: There was an error installing modified sage library code.
 }}}

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/3999#comment:4>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of 
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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