[PATCH] dbd.m4: use LDFLAGS instead of LIBS is required on Solaris

[Henry Jen <he...@ztune.net>]

Hi,

AC_CHECK_LIB is performed as "$CC $CFLAGS $LDFLAGS -llib $LIBS conftest.c"

With Sun Studio, -L needs to be in front of -l flag, which makes sense 
when you need control on which lib to be linked with.

In dbd.m4, using LIBS to define the -L flag, which should really be 
LDFLAGS, the attached patch does just that.

Please help to review and commit the patch. Much appreciated.

Cheers,
Henry

Re: [PATCH] dbd.m4: use LDFLAGS instead of LIBS is required on Solaris

[Garrett Rooney <ro...@electricjellyfish.net>]

On 3/30/06, William A. Rowe, Jr. <wr...@rowe-clan.net> wrote:
> Henry Jen wrote:
> > @@ -40,7 +40,7 @@
> >        apu_have_pgsql=0
> >      else
> >        CPPFLAGS="-I$withval/include"
> > -      LIBS="-L$withval/lib "
> > +      LDFLAGS="-L$withval/lib "
>
> +1 - this was LIBS abuse - your solution is correct :)

Agreed, committed in r390268.

Thanks!

-garrett

Re: [PATCH] dbd.m4: use LDFLAGS instead of LIBS is required on Solaris

["William A. Rowe, Jr." <wr...@rowe-clan.net>]

Henry Jen wrote:
> @@ -40,7 +40,7 @@
>        apu_have_pgsql=0
>      else
>        CPPFLAGS="-I$withval/include"
> -      LIBS="-L$withval/lib "
> +      LDFLAGS="-L$withval/lib "

+1 - this was LIBS abuse - your solution is correct :)

Bill