Re: [bug-gettext] [PATCH] Add a --disable-tools option

[Thomas Petazzoni]

Dear Stefano Lattarini,

On Tue, 28 May 2013 20:40:32 +0200, Stefano Lattarini wrote:

> > -SUBDIRS = gnulib-local gettext-runtime gettext-tools
> > +if ENABLE_TOOLS
> > +TOOLS_SUBDIR = gettext-tools
> > +else
> > +TOOLS_SUBDIR =
> > +endif
> > +
> > +SUBDIRS = gnulib-local gettext-runtime $(TOOLS_SUBDIR)
> >
> Or even simply:
> 
>   -SUBDIRS = gnulib-local gettext-runtime gettext-tools
>   +SUBDIRS = gnulib-local gettext-runtime
>   +if ENABLE_TOOLS
>   +SUBDIRS += gettext-tools
>   +endif
> 
> maybe?

I am by far not a Makefile.am expert, but I believe the += thing
doesn't work. Most of the Makefile.am I have seen do this trick of
defining an intermediate variable.

What I did is the way that is documented in the automake documentation:
http://www.gnu.org/software/automake/manual/html_node/Subdirectories-with-AM_005fCONDITIONAL.html.

Thanks for your feedback,

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com