[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

From: Stefano Lattarini
Subject: Re: [bug-gettext] [PATCH] Add a --disable-tools option
Date: Tue, 28 May 2013 21:19:29 +0200

Hi Thomas.

On 05/28/2013 09:07 PM, Thomas Petazzoni wrote:
> Dear Stefano Lattarini,
> On Tue, 28 May 2013 20:40:32 +0200, Stefano Lattarini wrote:
>>> -SUBDIRS = gnulib-local gettext-runtime gettext-tools
>>> +TOOLS_SUBDIR = gettext-tools
>>> +else
>>> +endif
>>> +
>>> +SUBDIRS = gnulib-local gettext-runtime $(TOOLS_SUBDIR)
>> Or even simply:
>>   -SUBDIRS = gnulib-local gettext-runtime gettext-tools
>>   +SUBDIRS = gnulib-local gettext-runtime
>>   +SUBDIRS += gettext-tools
>>   +endif
>> maybe?
> I am by far not a Makefile.am expert, but I believe the += thing
> doesn't work.
It does --- it's not portable to *make*, but Automake handles it
correctly and spits out an output that is portable to POSIX make
(and more :-)

> Most of the Makefile.am I have seen do this trick of
> defining an intermediate variable.
I can't answer about that.  Maybe they have good reasons to do so,
or maybe they too have just copied the idiom from other makefiles,
that too have copied from other older makefiles, ...  You get the

> 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.
I think the example is in that form to minimize spurious difference
with the following example "Subdirectories with AC_SUBST":

Anyway, what you did is *not wrong* -- I just suggested a briefer
idiom to obtain the same effect.  I have no strong feelings in the
issue, so if the Gettext maintainer can absolutely take your patch
if he prefers.  No objection at all from me!

> Thanks for your feedback,
> Thomas


reply via email to

[Prev in Thread] Current Thread [Next in Thread]