[Top][All Lists]

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

Re: AC_PATH_PROGS_FEATURE_CHECK results in namespace invasion

From: Russ Allbery
Subject: Re: AC_PATH_PROGS_FEATURE_CHECK results in namespace invasion
Date: Wed, 26 Feb 2014 16:38:15 -0800
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/23.4 (gnu/linux)

Peter Rosin <address@hidden> writes:
> On 2014-02-27 01:20, Russ Allbery wrote:

>> Wouldn't you name the variable FOO_FEATURE?  In other words, I think
>> the example in the Autoconf manual is not the best, but the
>> functionality you need is there.  I would rewrite the example as:
>>     AC_CACHE_CHECK([for m4 that supports indir], [ac_cv_path_M4_indir],
>>       [AC_PATH_PROGS_FEATURE_CHECK([M4_indir], [m4 gm4],
>>           ...
>>     AC_SUBST([M4], [$ac_cv_path_M4_indir])

> That doesn't work, because then you can't shortcut the test with the
> natural M4=/some/gnu/place/m4, you'd be forced to write M4_indir=...
> instead. This is even worse than saving the ac_cv_path_FOO variable
> during the AC_PATH_PROGS_FEATURE_CHECK macro, since it would be visible
> to the end user.

Good point.  However, couldn't you just put:

    : ${ac_cv_path_M4_indir:=$M4}

before the above code?

Russ Allbery (address@hidden)              <http://www.eyrie.org/~eagle/>

reply via email to

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