[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/4] AC_PROG_LEX: Rephrase LEX is set test
From: |
Bernhard Reutner-Fischer |
Subject: |
Re: [PATCH 2/4] AC_PROG_LEX: Rephrase LEX is set test |
Date: |
Thu, 9 Apr 2015 15:29:36 +0200 |
On 9 April 2015 at 15:21, Eric Blake <address@hidden> wrote:
> On 04/09/2015 06:43 AM, Bernhard Reutner-Fischer wrote:
>> 2015-04-08 Bernhard Reutner-Fischer <address@hidden>
>> [AC_CHECK_PROGS(LEX, flex lex, :)
>> -if test "x$LEX" != "x:"; then
>> +if [[ ${LEX:+y} ]]; then
>
> NACK. This is checking if $LEX is specifically something other than ':',
> not whether it is empty.
oops, managed to miss the ':'.
So scratch that one please.
- [PATCH] Replace 'test "${var+set}" = set' with 'test -n "${var+set}"', Bernhard Reutner-Fischer, 2015/04/08
- Re: [PATCH] Replace 'test "${var+set}" = set' with 'test -n "${var+set}"', Paul Eggert, 2015/04/08
- Re: [PATCH] Replace 'test "${var+set}" = set' with 'test -n "${var+set}"', Eric Blake, 2015/04/08
- Re: [PATCH] Replace 'test "${var+set}" = set' with 'test -n "${var+set}"', Eric Blake, 2015/04/08
- Re: [PATCH] Replace 'test "${var+set}" = set' with 'test -n "${var+set}"', Paul Eggert, 2015/04/09
- Re: [PATCH] Replace 'test "${var+set}" = set' with 'test -n "${var+set}"', Eric Blake, 2015/04/09
- [PATCH 2/4] AC_PROG_LEX: Rephrase LEX is set test, Bernhard Reutner-Fischer, 2015/04/09
- Re: [PATCH 2/4] AC_PROG_LEX: Rephrase LEX is set test, Eric Blake, 2015/04/09
- Re: [PATCH 2/4] AC_PROG_LEX: Rephrase LEX is set test,
Bernhard Reutner-Fischer <=
- [PATCH 0/4] misc test(1) invocation tweaks, Bernhard Reutner-Fischer, 2015/04/09
- Re: [PATCH 0/4] misc test(1) invocation tweaks, Eric Blake, 2015/04/09
- [PATCH 4/4] _AC_OUTPUT_FILES_PREPARE: Rephrase CONFIG_FILES test, Bernhard Reutner-Fischer, 2015/04/09
- Re: [PATCH 4/4] _AC_OUTPUT_FILES_PREPARE: Rephrase CONFIG_FILES test, Eric Blake, 2015/04/09
- Re: [PATCH 4/4] _AC_OUTPUT_FILES_PREPARE: Rephrase CONFIG_FILES test, Bernhard Reutner-Fischer, 2015/04/09
- [PATCH 3/4] AC_PROG_MKDIR_P: Also accept BusyBox mkdir -p, Bernhard Reutner-Fischer, 2015/04/09
- Re: [PATCH 3/4] AC_PROG_MKDIR_P: Also accept BusyBox mkdir -p, Eric Blake, 2015/04/09
- Re: [PATCH 3/4] AC_PROG_MKDIR_P: Also accept BusyBox mkdir -p, Eric Blake, 2015/04/09
- [PATCH v2 1/4] Replace 'test "${var+set}" = set' with '[ ${var+y} ]', Bernhard Reutner-Fischer, 2015/04/09
- Re: [PATCH v2 1/4] Replace 'test "${var+set}" = set' with '[ ${var+y} ]', Eric Blake, 2015/04/09