[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [SIMPLE PATCH] {maint} Add test for `AM_WITH_DMALLOC' macro.
From: |
Stefano Lattarini |
Subject: |
Re: [SIMPLE PATCH] {maint} Add test for `AM_WITH_DMALLOC' macro. |
Date: |
Sun, 26 Sep 2010 22:35:18 +0200 |
User-agent: |
KMail/1.13.3 (Linux/2.6.30-2-686; KDE/4.4.4; i686; ; ) |
On Sunday 26 September 2010, Ralf Wildenhues wrote:
> * Stefano Lattarini wrote on Sun, Sep 26, 2010 at 09:51:11PM CEST:
> > On Sunday 26 September 2010, Ralf Wildenhues wrote:
> > > * Stefano Lattarini wrote on Tue, Sep 21, 2010 at 12:57:10PM
CEST:
> > > > +dnl Let's show some good practices ;-)
> > > > +AC_SUBST([AM_LDFLAGS], [$LDFLAGS])
> > >
> > > This is not needed, nor would I recommend it. LDFLAGS is
> > > AC_SUBSTed by Autoconf already, and used in link flags already
> > > anyway.
> >
> > You sure? If I remove the AC_SUBST of LDFLAGS, the test fails
>
> > with:
> Well yes, you need to adjust the test to grep LDFLAGS, of course.
I did that, of course ;-) But LDFLAGS is not AC_SUBT'd by default,
leading to the failure I reported. If I add "AC_SUBST([LDFLAGS])"
to configure.in, the test passes.
Regards,
Stefano