Re: [Patch V2] libgfortran: do not assume libm

2012-04-02 Thread Tristan Gingold
On Mar 30, 2012, at 2:39 PM, Paolo Bonzini wrote: > Il 30/03/2012 12:22, Tristan Gingold ha scritto: >> >> On Mar 27, 2012, at 10:38 AM, Janne Blomqvist wrote: >> >>> On Tue, Mar 27, 2012 at 11:01, Tristan Gingold wrote: Hi, this patch fixes this issue. Is it OK ? >>> >>> Ok.

Re: [Patch V2] libgfortran: do not assume libm

2012-03-30 Thread Paolo Bonzini
Il 30/03/2012 12:22, Tristan Gingold ha scritto: > > On Mar 27, 2012, at 10:38 AM, Janne Blomqvist wrote: > >> On Tue, Mar 27, 2012 at 11:01, Tristan Gingold wrote: >>> Hi, >>> >>> this patch fixes this issue. Is it OK ? >> >> Ok. >> >>> Maybe we should include the AC_DEFINE action within GCC_C

Re: [Patch V2] libgfortran: do not assume libm

2012-03-30 Thread Tristan Gingold
On Mar 27, 2012, at 10:38 AM, Janne Blomqvist wrote: > On Tue, Mar 27, 2012 at 11:01, Tristan Gingold wrote: >> Hi, >> >> this patch fixes this issue. Is it OK ? > > Ok. > >> Maybe we should include the AC_DEFINE action within GCC_CHECK_MATH_FUNC. >> Will try to do that. > > That looks li

Re: [Patch V2] libgfortran: do not assume libm

2012-03-27 Thread Tristan Gingold
On Mar 27, 2012, at 10:38 AM, Janne Blomqvist wrote: > On Tue, Mar 27, 2012 at 11:01, Tristan Gingold wrote: >> Hi, >> >> this patch fixes this issue. Is it OK ? > > Ok. No check-gfortran regressions on x86_64-darwin, committed. >> Maybe we should include the AC_DEFINE action within GCC_CHE

Re: [Patch V2] libgfortran: do not assume libm

2012-03-27 Thread Janne Blomqvist
On Tue, Mar 27, 2012 at 11:01, Tristan Gingold wrote: > Hi, > > this patch fixes this issue.  Is it OK ? Ok. > Maybe we should include the AC_DEFINE action within GCC_CHECK_MATH_FUNC.   > Will try to do that. That looks like a cleaner solution, yes, and less chance for typos to sneak in. > Sor

Re: [Patch V2] libgfortran: do not assume libm

2012-03-27 Thread Tristan Gingold
On Mar 27, 2012, at 9:51 AM, Tristan Gingold wrote: > > On Mar 26, 2012, at 11:43 PM, Janne Blomqvist wrote: > [...] > >> On x86_64-unknown-linux-gnu bootstrap I see the following: >> >> >> make[1]: Entering directory >> `/home/janne/src/gfortran/trunk/objdir-git/x86_64-unknown-linux-gnu/libg

Re: [Patch V2] libgfortran: do not assume libm

2012-03-27 Thread Tristan Gingold
On Mar 26, 2012, at 11:43 PM, Janne Blomqvist wrote: [...] > On x86_64-unknown-linux-gnu bootstrap I see the following: > > > make[1]: Entering directory > `/home/janne/src/gfortran/trunk/objdir-git/x86_64-unknown-linux-gnu/libgfortran' > make check-am > make[2]: Entering directory > `/home/ja

Re: [Patch V2] libgfortran: do not assume libm

2012-03-26 Thread Janne Blomqvist
On Mon, Mar 26, 2012 at 11:44, Tristan Gingold wrote: > > On Mar 22, 2012, at 11:06 AM, Paolo Bonzini wrote: > >> Il 22/03/2012 09:30, Tristan Gingold ha scritto: >>> Hi, >>> >>> this is version 2 of the patch. >>> >>> The initial problem is that libgfortran configure.ac used >>> AC_CHECK_LIB([m]

Re: [Patch V2] libgfortran: do not assume libm

2012-03-26 Thread Tristan Gingold
On Mar 22, 2012, at 11:06 AM, Paolo Bonzini wrote: > Il 22/03/2012 09:30, Tristan Gingold ha scritto: >> Hi, >> >> this is version 2 of the patch. >> >> The initial problem is that libgfortran configure.ac used AC_CHECK_LIB([m]…) >> to check wether several math functions are available. That d

Re: [Patch V2] libgfortran: do not assume libm

2012-03-22 Thread Tobias Burnus
Paolo Bonzini wrote: Il 22/03/2012 09:30, Tristan Gingold ha scritto: Bootstrapped without gfortran regressions on x86_64-darwin. Ok for trunk ? (I will submit a follow-up change in libquadmath once this change is approved). config/ 2012-03-22 Tristan Gingold * math.m4: New file. li

Re: [Patch V2] libgfortran: do not assume libm

2012-03-22 Thread Paolo Bonzini
Il 22/03/2012 09:30, Tristan Gingold ha scritto: > Hi, > > this is version 2 of the patch. > > The initial problem is that libgfortran configure.ac used AC_CHECK_LIB([m]…) > to check wether several math functions are available. That doesn't work on > VMS, because there is no such things as a l

[Patch V2] libgfortran: do not assume libm

2012-03-22 Thread Tristan Gingold
Hi, this is version 2 of the patch. The initial problem is that libgfortran configure.ac used AC_CHECK_LIB([m]…) to check wether several math functions are available. That doesn't work on VMS, because there is no such things as a libm. It seems to me that there are no autoconf macro to check