Re: Help with src/yleval

[Gary V. Vaughan]

On Sun, Sep 30, 2001 at 11:04:24AM +0100, Gary V. Vaughan wrote:
> On Sat, Sep 29, 2001 at 06:38:50PM +0200, address@hidden wrote:
> > 
> > I think in a previous mail I meant
> > 
> > Index: src/Makefile.am
> > --- src/Makefile.am Tue, 18 Sep 2001 20:57:18 +0200 akim
> > +++ src/Makefile.am Sat, 29 Sep 2001 16:20:46 +0200 akim
> > @@ -22,7 +22,7 @@
> >  MAINTAINERCLEANFILES = Makefile.in
> >  
> >  INCLUDES       = -I$(top_builddir) -I$(top_srcdir) \
> > -                 -I$(top_builddir)/m4 -I$(top_srcdir)/m4 $(INCLTDL) 
> > $(INTLINCL)
> > +                 -I$(top_builddir)/m4 -I$(top_srcdir)/m4 $(INTLINCL)
> >  
> >  noinst_HEADERS = gnu-getopt.h
> 
> $(INCLTDL) belongs in m4/Makefile.am rather than src/Makefile.am,
> since the ltdl dependency is in libm4.la (where libtool can track the
> dependency and automatically add it to the link line of applications
> that link with libm4.la).

Done.

> > but I am still confused with the various occurrences of LIBADD_DL, sometimes
> > as a Makefile macro, something as an AC_SUBST pattern:
> 
> I wasn't aware of Automake's AC_SUBST handling when I submitted some
> of my early patches to Rene... but I was for my more recent changes.
> I should spend some time cleaning this up.

Done.

> > once in m4_LDADD and another time in libm4_la_LIBADD.
> 
> Hmm.  That is wrong.

Now fixed in CVS m4.

> > Also, naming is not consitant between
> > INTLINCL and INCLLTDL.

Now fixed in CVS HEAD and branch-1-4 libtool.

Cheers,
        Gary.
-- 
  ())_. Gary V. Vaughan     gary@(oranda.demon.co.uk|gnu.org)
  ( '/  Research Scientist  http://www.oranda.demon.co.uk       ,_())____
  / )=  GNU Hacker          http://www.gnu.org/software/libtool  \'      `&
`(_~)_  Tech' Author        http://sources.redhat.com/autobook   =`---d__/