bug-coreutils
[Top][All Lists]
Advanced

[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





reply via email to

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