coreutils
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] tests: avoid problematic $(< file) shell construct


From: Bernhard Voelker
Subject: Re: [PATCH] tests: avoid problematic $(< file) shell construct
Date: Sat, 07 Dec 2013 18:39:21 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130329 Thunderbird/17.0.5

On 12/07/2013 04:39 PM, Pádraig Brady wrote:
> bash and ksh were fine with this, but dash 0.5.6 wasn't so...
> 
> commit 645102c94cf8fddf1251940b2af884871befb388
> Author: Pádraig Brady <address@hidden>
> Date:   Sat Dec 7 15:36:33 2013 +0000
> 
>     tests: avoid problematic $(< file) shell construct
> 
>     * tests/cp/link-deref.sh: This was seen to cause an issue
>     with dash v0.5.6 at least, so change to a simpler expression.
> 
> diff --git a/tests/cp/link-deref.sh b/tests/cp/link-deref.sh
> index 91a6c5d..c12dfc8 100755
> --- a/tests/cp/link-deref.sh
> +++ b/tests/cp/link-deref.sh
> @@ -55,7 +55,7 @@ for src in dirlink filelink danglink; do
>          "$result"   \
>          "$ino_dst"  \
>          "$typ_dst"  \
> -        "$(< err)"  \
> +        "$(cat err)"  \
>          > out
> 
>        # What was expected?
> 

+1, thanks.

I'm afraid I'll be tempted to use the $(<file) construct next time
again.  Do you this it's worth adding a new syntax-check?

BTW: how did you generate the mail? It seems it got a bit mangled
this time.

Have a nice day,
Berny



reply via email to

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