#9858: FLINT 1.5.0.p5's test suite fails to build with new GMP/MPIR
------------------------+---------------------------------------------------
   Reporter:  leif      |          Owner:  leif                 
       Type:  defect    |         Status:  needs_review         
   Priority:  critical  |      Milestone:  sage-4.7.3           
  Component:  packages  |       Keywords:  SAGE_CHECK mpz_random
Work_issues:            |       Upstream:  N/A                  
   Reviewer:            |         Author:  Leif Leonhardy       
     Merged:            |   Dependencies:                       
------------------------+---------------------------------------------------
Changes (by newvalueoldvalue):

 * cc: davidloeffler (added)
  * status:  new => needs_review
  * author:  => Leif Leonhardy


Old description:

> Due to the use of deprecated/obsolete GMP functions, the test suite of
> flint-1.5.0.p5 fails to build since merging the new MPIR 2.1.3 (#8664),
> or if one uses any other recent version of MPIR or GMP.  This means that
> the package's installation fails if {{{SAGE_CHECK=yes}}}:
>
> {{{
> Successfully installed flint-1.5.0.p5
> Running the test suite.
> *************************************************
> Running test suite. This should take 6-20 minutes
>    Please report all failures to sage-devel
> *************************************************
> gcc -std=c99
> -I/mnt/usb1/scratch/jdemeyer/sage-4.7.2.alpha3-baseline/local/include/
> -I/mnt/usb1/scratch/jdemeyer/sage-4.7.2.alpha3-baselin
> e/local/include -mtune=opteron -march=opteron -fPIC -funroll-loops  -O2
> -c F_mpz-test.c -o F_mpz-test.o
> [...]
> gcc -std=c99
> -I/mnt/usb1/scratch/jdemeyer/sage-4.7.2.alpha3-baseline/local/include/
> -I/mnt/usb1/scratch/jdemeyer/sage-4.7.2.alpha3-baseline/local/include
> -mtune=opteron -march=opteron -fPIC -funroll-loops  -O2 -c fmpz-test.c -o
> fmpz-test.o
> fmpz-test.c: In function âtest___fmpz_normaliseâ:
> fmpz-test.c:1379:8: warning: implicit declaration of function
> âmpz_randomâ
> g++
> -I/mnt/usb1/scratch/jdemeyer/sage-4.7.2.alpha3-baseline/local/include/
> -I/mnt/usb1/scratch/jdemeyer/sage-4.7.2.alpha3-baseline/local/include
> -mtune=opteron -march=opteron -fPIC -funroll-loops  -O2 fmpz-test.o test-
> support.o -o fmpz-test zn_mod.o misc.o mul_ks.o pack.o mul.o mulmid.o
> mulmid_ks.o ks_support.o mpn_mulmid.o nuss.o pmf.o pmfvec_fft.o tuning.o
> mul_fft.o mul_fft_dft.o array.o invert.o mpn_extras.o mpz_extras.o
> memory-manager.o ZmodF.o ZmodF_mul.o ZmodF_mul-tuning.o fmpz.o
> fmpz_poly.o mpz_poly-tuning.o mpz_poly.o ZmodF_poly.o long_extras.o
> zmod_poly.o theta.o zmod_mat.o F_mpz.o tinyQS.o factor_base.o poly.o
> sieve.o linear_algebra.o block_lanczos.o NTL-interface.o
> -L/mnt/usb1/scratch/jdemeyer/sage-4.7.2.alpha3-baseline/local/lib/
> -L/mnt/usb1/scratch/jdemeyer/sage-4.7.2.alpha3-baseline/local/lib/  -lgmp
> -lpthread -lntl -lm
> fmpz-test.o: In function `test___fmpz_normalise':
> fmpz-test.c:(.text+0x368d): undefined reference to `mpz_random'
> collect2: ld returned 1 exit status
> make: *** [fmpz-test] Error 1
> Error building the test suite for FLINT
> *************************************
> Error testing package ** flint-1.5.0.p5 **
> *************************************
> sage: An error occurred while testing flint-1.5.0.p5
> }}}
>
> I guess Bill has already fixed that in FLINT 1.5.2, which was released
> over a year ago. So simply updating the spkg might suffice.
>
> Btw, at least the current FLINT in Sage ignores user-provided
> {{{CFLAGS}}}; one could perhaps "move" them into some other environment
> variable used by FLINT.
>
> ----
>
> Upstream: http://www.flintlib.org/

New description:

 Due to the use of deprecated/obsolete GMP functions, the test suite of
 flint-1.5.0.p5 fails to build since merging the new MPIR 2.1.3 (#8664), or
 if one uses any other recent version of MPIR or GMP.  This means that the
 package's installation fails if {{{SAGE_CHECK=yes}}}:

 {{{
 Successfully installed flint-1.5.0.p5
 Running the test suite.
 *************************************************
 Running test suite. This should take 6-20 minutes
    Please report all failures to sage-devel
 *************************************************
 gcc -std=c99
 -I/mnt/usb1/scratch/jdemeyer/sage-4.7.2.alpha3-baseline/local/include/
 -I/mnt/usb1/scratch/jdemeyer/sage-4.7.2.alpha3-baselin
 e/local/include -mtune=opteron -march=opteron -fPIC -funroll-loops  -O2 -c
 F_mpz-test.c -o F_mpz-test.o
 [...]
 gcc -std=c99
 -I/mnt/usb1/scratch/jdemeyer/sage-4.7.2.alpha3-baseline/local/include/
 -I/mnt/usb1/scratch/jdemeyer/sage-4.7.2.alpha3-baseline/local/include
 -mtune=opteron -march=opteron -fPIC -funroll-loops  -O2 -c fmpz-test.c -o
 fmpz-test.o
 fmpz-test.c: In function âtest___fmpz_normaliseâ:
 fmpz-test.c:1379:8: warning: implicit declaration of function âmpz_randomâ
 g++
 -I/mnt/usb1/scratch/jdemeyer/sage-4.7.2.alpha3-baseline/local/include/
 -I/mnt/usb1/scratch/jdemeyer/sage-4.7.2.alpha3-baseline/local/include
 -mtune=opteron -march=opteron -fPIC -funroll-loops  -O2 fmpz-test.o test-
 support.o -o fmpz-test zn_mod.o misc.o mul_ks.o pack.o mul.o mulmid.o
 mulmid_ks.o ks_support.o mpn_mulmid.o nuss.o pmf.o pmfvec_fft.o tuning.o
 mul_fft.o mul_fft_dft.o array.o invert.o mpn_extras.o mpz_extras.o memory-
 manager.o ZmodF.o ZmodF_mul.o ZmodF_mul-tuning.o fmpz.o fmpz_poly.o
 mpz_poly-tuning.o mpz_poly.o ZmodF_poly.o long_extras.o zmod_poly.o
 theta.o zmod_mat.o F_mpz.o tinyQS.o factor_base.o poly.o sieve.o
 linear_algebra.o block_lanczos.o NTL-interface.o
 -L/mnt/usb1/scratch/jdemeyer/sage-4.7.2.alpha3-baseline/local/lib/
 -L/mnt/usb1/scratch/jdemeyer/sage-4.7.2.alpha3-baseline/local/lib/  -lgmp
 -lpthread -lntl -lm
 fmpz-test.o: In function `test___fmpz_normalise':
 fmpz-test.c:(.text+0x368d): undefined reference to `mpz_random'
 collect2: ld returned 1 exit status
 make: *** [fmpz-test] Error 1
 Error building the test suite for FLINT
 *************************************
 Error testing package ** flint-1.5.0.p5 **
 *************************************
 sage: An error occurred while testing flint-1.5.0.p5
 }}}

 I guess Bill has already fixed that in FLINT 1.5.2, which was released
 over a year ago. So simply updating the spkg might suffice.

 Btw, at least the current FLINT in Sage ignores user-provided
 {{{CFLAGS}}}; one could perhaps "move" them into some other environment
 variable used by FLINT.

 ----

 Upstream: http://www.flintlib.org/

 ----

 '''New spkg:'''
 [http://sage.math.washington.edu/home/leif/Sage/spkgs/flint-1.5.0.p10.spkg]

 '''md5sum:''' `907a720d1b9bf9e4bc1412e08b2f8892`


 === flint-1.5.0.p10 (Leif Leonhardy, October 11th 2011) ===
   * #9858: Add an upstream patch from FLINT 1.5.2 to make FLINT's test
 suite
     build (when `SAGE_CHECK=yes`) with MPIR 1.3.x, 2.x (cf. #8664) and GMP
 5.x.
     The patch just substitutes deprecated random functions in one file
     (`fmpz-test.c`) which have been removed from the mentioned GMP / MPIR
     versions.

--

Comment:

 I've made a FLINT 1.5.0.p10 spkg which just fixes the trivial test suite
 build error (see the ticket's description), since providing a ''clean''
 FLINT 1.5.2 spkg based on the 1.5.0.p9 is tedious and still takes some
 time, hopefully just until tomorrow.

 Please test (with `SAGE_CHECK=yes` of course, and an MPIR >=1.3.x or GMP
 5.x, e.g. with Sage 4.7.2.alpha'''3''') and review!

 (Unless you're happy with unmerging the overdue MPIR upgrade from Sage
 4.7.2 just for this reason.)

 Thanks in advance.

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