Changes between Version 6 and Version 8 of Ticket #27714


Ignore:
Timestamp:
04/23/19 16:48:19 (2 years ago)
Author:
embray
Comment:

The previous PR to flint partially fixed the problem, but not fully. See updated description.

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #27714

    • Property Report Upstream changed from Reported upstream. No feedback yet. to Fixed upstream, but not in a stable release.
  • Ticket #27714 – Description

    v6 v8  
    1414followed by a long string of similar errors in the same module.
    1515
    16 This is probably similar to the error fixed in #27713 but a different manifestation thereof:
     16This is probably similar to the error fixed in #27713 but a different manifestation thereof.
    1717
    18 In `sage/libs/gmp/types.pxd` we have:
     18This is partially fixed by the PR linked below, but then another related error arises related to flint and/or zn_poly abusing GMP type definitions:
    1919
    2020{{{
    21     ctypedef unsigned long mp_limb_t
    22     ctypedef long mp_limb_signed_t
     21[sagelib-8.8.beta3] g++ -shared -Wl,--enable-auto-image-base -L/home/embray/src/sagemath/sage/local/lib -Wl,-rpath,/home/embray/src/sagemath/sage/local/lib -L/home/embray/src/sagemath/sage/local/lib -Wl,-rpath,/home/embray/src/sagemath/sage/local/lib build/temp.cygwin-2.9.0-x86_64-2.7/build/cythonized/sage/rings/polynomial/polynomial_zz_pex.o -L/home/embray/src/sagemath/sage/local/lib -L/home/embray/src/sagemath/sage/local/lib/python2.7/config -L/home/embray/src/sagemath/sage/local/lib -lntl -lgmp -lstdc++ -lpython2.7 -o build/lib.cygwin-2.9.0-x86_64-2.7/sage/rings/polynomial/polynomial_zz_pex.dll -lpari
     22[sagelib-8.8.beta3] [380/437] gcc -I/usr/include/ncurses -fno-strict-aliasing -g -O2 -DNDEBUG -g -fwrapv -O3 -Wall -Wno-unused -I./sage/cpython -I./sage/libs/ntl -I/home/embray/src/sagemath/sage/local/lib/python2.7/site-packages/cysignals -I/home/embray/src/sagemath/sage/local/include -I/home/embray/src/sagemath/sage/src -I/home/embray/src/sagemath/sage/src/sage/ext -I/home/embray/src/sagemath/sage/local/include/python2.7 -I/home/embray/src/sagemath/sage/local/lib/python2.7/site-packages/numpy/core/include -Ibuild/cythonized -I/home/embray/src/sagemath/sage/local/include/python2.7 -c build/cythonized/sage/rings/polynomial/polynomial_zmod_flint.cpp -o build/temp.cygwin-2.9.0-x86_64-2.7/build/cythonized/sage/rings/polynomial/polynomial_zmod_flint.o -fno-strict-aliasing -DCYTHON_CLINE_IN_TRACEBACK=1 -std=gnu++11
     23[sagelib-8.8.beta3] In file included from build/cythonized/sage/rings/polynomial/polynomial_zmod_flint.cpp:697:0:
     24[sagelib-8.8.beta3] ../local/include/zn_poly/zn_poly.h:72:23: error: conflicting declaration ‘typedef long unsigned int mp_limb_t’
     25[sagelib-8.8.beta3]  typedef unsigned long  ulong;
     26[sagelib-8.8.beta3]                        ^~~~~~~
     27[sagelib-8.8.beta3] In file included from build/cythonized/sage/rings/polynomial/polynomial_zmod_flint.cpp:676:0:
     28[sagelib-8.8.beta3] /usr/include/gmp.h:138:32: note: previous declaration as ‘typedef long long unsigned int mp_limb_t’
     29[sagelib-8.8.beta3]  typedef unsigned long long int mp_limb_t;
     30...
    2331}}}
    2432
    25 when in fact the definitions of these types can be platform-dependent.  Not sure what best to do about this.
     33
     34and followed by some other errors that look related.
     35
    2636
    2737**Upstream PR:** https://github.com/wbhart/flint2/pull/557