Re: [U-Boot] [PATCH 6/6] hush: add some tests for quoting

2014-11-06 Thread Simon Glass
Hi, On 5 November 2014 13:11, Rabin Vincent wrote: > On Sat, Nov 01, 2014 at 09:12:37AM -0600, Simon Glass wrote: >> On 29 October 2014 16:21, Rabin Vincent wrote: >> > + assert(run_command("setenv ut_var '\"'; setenv ut_var2 >> > \"${ut_var}\"", 0) == 0); >> > + assert(!strcmp(gete

Re: [U-Boot] [PATCH 6/6] hush: add some tests for quoting

2014-11-05 Thread Rabin Vincent
On Sat, Nov 01, 2014 at 09:12:37AM -0600, Simon Glass wrote: > On 29 October 2014 16:21, Rabin Vincent wrote: > > + assert(run_command("setenv ut_var '\"'; setenv ut_var2 > > \"${ut_var}\"", 0) == 0); > > + assert(!strcmp(getenv("ut_var2"), "\"")); > > + > > + assert(run_command

Re: [U-Boot] [PATCH 6/6] hush: add some tests for quoting

2014-11-01 Thread Simon Glass
Hi Rabin, On 29 October 2014 16:21, Rabin Vincent wrote: > Add a couple of tests for quoting. The indirect variable read test is > a fixed version of the following expression from Fedora ARM's boot.scr. > This unfixed expression stopped working after fe9ca3d ("hush: fix some > quoted variable ex

[U-Boot] [PATCH 6/6] hush: add some tests for quoting

2014-10-29 Thread Rabin Vincent
Add a couple of tests for quoting. The indirect variable read test is a fixed version of the following expression from Fedora ARM's boot.scr. This unfixed expression stopped working after fe9ca3d ("hush: fix some quoted variable expansion issues"): setenv catcat setenv catout\;'setenv catX "sete