Re: [PATCH 4/4] _AC_OUTPUT_FILES_PREPARE: Rephrase CONFIG_FILES test

Bernhard Reutner-Fischer <[email protected]>
Newsgroups gmane.comp.sysutils.autoconf.patches
Message-ID <CAC1BbcRYmGGyvJhPhGSYX6VPsqb1M5eyX5MD2HHCJ-GQVhyawA@mail.gmail.com>
On 9 April 2015 at 15:22, Eric Blake <[email protected]> wrote:
> On 04/09/2015 06:43 AM, Bernhard Reutner-Fischer wrote:
>> 2015-04-08  Bernhard Reutner-Fischer  <[email protected]>
>>
>>       * lib/autoconf/status.m4 (_AC_OUTPUT_FILES_PREPARE): Rephrase
>>       test for empty CONFIG_FILES.
>>
>> Signed-off-by: Bernhard Reutner-Fischer <[email protected]>
>> ---
>>  lib/autoconf/status.m4 |    2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/lib/autoconf/status.m4 b/lib/autoconf/status.m4
>> index 7ccc847..6a00739 100644
>> --- a/lib/autoconf/status.m4
>> +++ b/lib/autoconf/status.m4
>> @@ -327,7 +327,7 @@ m4_define([_AC_OUTPUT_FILES_PREPARE],
>>  [# Set up the scripts for CONFIG_FILES section.
>>  # No need to generate them if there are no CONFIG_FILES.
>>  # This happens for instance with `./config.status config.h'.
>> -if test -n "$CONFIG_FILES"; then
>> +if [[ ${CONFIG_FILES:+y} ]]; then
>
> Why the ${a:+b} form here, but the ${a+b} form in patch 1?

If CONFIG_FILES is set to an empty string then there's nothing to do i assume.

thanks,
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.