[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
- [PATCH] tests: avoid problematic $(< file) shell construct, Pádraig Brady, 2013/12/07
- Re: [PATCH] tests: avoid problematic $(< file) shell construct,
Bernhard Voelker <=
- Re: [PATCH] tests: avoid problematic $(< file) shell construct, Pádraig Brady, 2013/12/07
- Re: [PATCH] tests: avoid problematic $(< file) shell construct, Pádraig Brady, 2013/12/07
- Re: [PATCH] tests: avoid problematic $(< file) shell construct, Pádraig Brady, 2013/12/12
- Re: [PATCH] tests: avoid problematic $(< file) shell construct, Bernhard Voelker, 2013/12/15
- Re: [PATCH] tests: avoid problematic $(< file) shell construct, Eric Blake, 2013/12/16
- Re: [PATCH] tests: avoid problematic $(< file) shell construct, Bernhard Voelker, 2013/12/16
- Re: [PATCH] tests: avoid problematic $(< file) shell construct, Pádraig Brady, 2013/12/16
- Re: [PATCH] tests: avoid problematic $(< file) shell construct, Bernhard Voelker, 2013/12/16