Re: CVS commit: src/external/bsd/kyua-atf-compat/dist

[Julio Merino <julio%meroh.net@localhost>]

On Feb 25, 2013, at 15:40, "Valeriy E. Ushakov" <uwe%stderr.spb.ru@localhost> 
wrote:

> On Mon, Feb 25, 2013 at 18:49:51 +0000, Julio Merino wrote:
> 
>> Module Name: src
>> Committed By:        jmmv
>> Date:                Mon Feb 25 18:49:51 UTC 2013
>> 
>> Modified Files:
>>      src/external/bsd/kyua-atf-compat/dist: atf-run.sh
>> 
>> Log Message:
>> Cherry-pick upstream change d0daf9983f5a0e635f1127dbc827aa114daa90d8:
>> 
>> Fix broken variable parsing with NetBSD's /bin/sh
>> 
>> Quote the expansion of a $() command that was not properly surrounded
>> by quotes so that this runs properly with NetBSD's /bin/sh.
> 
> grep|sed pipeline is ok on a command line where one is lazy, in a
> script it should be just sed :)

Heh, very true. I'll try to play with this a bit later.

> Your $ws doesn't do what you expect it to do.  Try your command on a
> line that actually has tabs.  This is because backslash has no special
> meaning inside [], so your \t is not a tab, but either a backslash or
> a letter 't'.

Thanks; this should be fixed now.