Re: expand: Fix ghost fields with unquoted $@/$*

[Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>]

On Fri, Mar 23, 2018 at 09:27:37AM +0100, Harald van Dijk wrote:
> 
> Also the above. But it breaks a traditional ash extension:
> 
>   unset IFS
>   set -- A1 B2 C3
>   echo ${@%2 C3}
> 
> This used to print A1 B, but prints A1 B2 C3 with your patch.
> 
>   echo ${@%2}
> 
> This used to print A1 B2 C3, but prints A1 B with your patch.

Hmm, it still does on my machine:

$ build/src/dash
$ unset IFS
$ set -- A1 B2 C3
$ echo ${@%2 C3}
A1 B
$

I'm testing against this commit:

commit c166b718b496da63c4df7a0972df2fc6cd38256b
Author: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
Date:   Thu Mar 15 18:27:30 2018 +0800

    parser: Fix backquote support in here-document EOF mark

Are you testing against the same commit or have you applied other
patches already?

Thanks,
-- 
Email: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
--
To unsubscribe from this list: send the line "unsubscribe dash" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html