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 ra...@rab.in wrote: On Sat, Nov 01, 2014 at 09:12:37AM -0600, Simon Glass wrote: On 29 October 2014 16:21, Rabin Vincent ra...@rab.in wrote: + assert(run_command(setenv ut_var '\'; setenv ut_var2 \${ut_var}\, 0) == 0); +

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 ra...@rab.in wrote: + assert(run_command(setenv ut_var '\'; setenv ut_var2 \${ut_var}\, 0) == 0); + assert(!strcmp(getenv(ut_var2), \)); + + assert(run_command(setenv

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 ra...@rab.in 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

[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 setenv