Re: [Bug-gsl] Test result for test release of gel 1.16

[Patrick Alken]

Hi Steve, thanks for the reports - I've fixed the issues (hopefully) 
with rng/test.c, specfunc/ellint.c and specfunc/coupling.c that you 
reported.

Just for clarification, you're saying that the 'clang' compiler passes 
the SV_decomp_mod, and the default cc does not? I can certainly add a 
note about this. As an aside a few days ago I ran make check on Mac OS 
10.6, using a port of gcc, and everything went ok.

Patrick

On 06/02/2013 11:58 AM, Steve Brosnan wrote:
> Test results on a mid-2011 iMac (quad-core i5) running MacOS 10.8.3 follow…
>
> As exists for gsl-1.15, (./configure; make; make check) shows multiple errors 
> for SV_decomp_mod. This has been investigated previously in this forum and was 
> found to be related to optimization problems with the default cc compiler on 
> Apple products. Apple's preferred compiler is now 'clang'.
>
> Repeating the build (after make clean) using (./configure CC=clang; make; make 
> check) results in the following:
> Warnings issued in 'make':
> ...
> coupling.c:167:17: warning: implicit declaration of function 'gsl_sf_exp_err_e' 
> is invalid in C99
>        [-Wimplicit-function-declaration]
>        status += gsl_sf_exp_err_e(bc1.val + bc2.val + bc3.val + lnorm.val,
> ...
> ellint.c:124:9: warning: type specifier missing, defaults to 'int' 
> [-Wimplicit-int]
>    const nmax = 10000;
> ...
> Warning issued in 'make check':
> Making check in rng
> make  test
> clang -DHAVE_CONFIG_H -I. -I.. -I..    -g -O2 -MT test.o -MD -MP -MF 
> .deps/test.Tpo -c -o test.o test.c
> test.c:639:16: warning: implicit declaration of function 'rng_sanity_test' is 
> invalid in C99 [-Wimplicit-function-declaration]
>        status = rng_sanity_test (r);
>                 ^
> 1 warning generated.
> ...
>
> Apparently clang is more picky than the standard compiler. I do not recall any 
> such warnings in gsl 1.15, but I could be mistaken. I saw your comment about 
> waiting to fix issues with strict compiler flags until another release. I 
> understand your caution.
>
> Since the root cause for the optimization issue has not been identified, there 
> was previously some discussion about adding a note to the INSTALL file 
> recommending that Mac users specify the 'clang' compiler. I did not see any 
> such comment in the existing INSTALL file. I think it would be a good idea.
>
> Steve
>
>
> On May 31, 2013, at 9:00 AM, address@hidden wrote:
>
> > Date: Thu, 30 May 2013 11:11:16 -0600
> > From: Patrick Alken <address@hidden>
> > To: "address@hidden" <address@hidden>, "address@hidden"
> >         <address@hidden>
> > Subject: [Bug-gsl] test release for gsl 1.16
> > Message-ID: <address@hidden>
> > Content-Type: text/plain; charset=ISO-8859-1; format=flowed
> >
> > I have uploaded a test release for gsl 1.16 here:
> >
> > http://alpha.gnu.org/gnu/gsl/gsl-1.15.90.tar.gz
> > http://alpha.gnu.org/gnu/gsl/gsl-1.15.90.tar.gz.sig
> >
> > All reports are welcome - anyone who can test on various platforms would
> > be appreciated (BSD, Mac OS, Windows).
> >
> > Please try testing the build and also the documentation (I upgraded the
> > texinfo.tex file in doc/ so I want to make sure that works with
> > everyone's configuration)
> >
> > 1) ./configure ; make ; make check
> > 2) cd doc ; make ps
> >
> > Report any successes/failures to address@hidden