[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#16940: [PATCH] tests: Do not access /dev/tty if it does not exist
From: |
Bernhard Voelker |
Subject: |
bug#16940: [PATCH] tests: Do not access /dev/tty if it does not exist |
Date: |
Wed, 05 Mar 2014 23:02:04 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.3.0 |
On 03/05/2014 08:05 PM, Pádraig Brady wrote:
> On 03/05/2014 04:46 PM, Pádraig Brady wrote:
>> Note the ':>' is problematic though as the : always returns true.
>> Well it does for solaris, freebsd, dash, bash-4.3 at least.
>> In ksh and bash<=4.2 it does return an error as you assumed.
>> The ':>' construct is only needed for csh anyway I think
>> which we don't support.
>>
>> I notice a few more cases: git grep ": *>.*"
>> which I'll fix in a follow up patch with maybe a syntax check.
>
> Attached.
Great, looks good.
Thanks & have a nice day,
Berny
- bug#16940: [PATCH] tests: Do not access /dev/tty if it does not exist, Cyril Roelandt, 2014/03/04
- bug#16940: [PATCH] tests: Do not access /dev/tty if it does not exist, Pádraig Brady, 2014/03/04
- bug#16940: [PATCH] tests: Do not access /dev/tty if it does not exist, Bernhard Voelker, 2014/03/05
- bug#16940: [PATCH] tests: Do not access /dev/tty if it does not exist, Pádraig Brady, 2014/03/05
- bug#16940: [PATCH] tests: Do not access /dev/tty if it does not exist, Bernhard Voelker, 2014/03/05
- bug#16940: [PATCH] tests: Do not access /dev/tty if it does not exist, Pádraig Brady, 2014/03/05
- bug#16940: [PATCH] tests: Do not access /dev/tty if it does not exist, Pádraig Brady, 2014/03/05
- bug#16940: [PATCH] tests: Do not access /dev/tty if it does not exist,
Bernhard Voelker <=