Re: [U-Boot] 'run' no longer aborts on an error, can someone please verify

2012-04-23 Thread Simon Glass
Hi Wolfgang, On Mon, Apr 23, 2012 at 11:16 PM, Wolfgang Denk wrote: > Dear Simon, > > In message > you > wrote: >> >> Thanks for the detailed information. This might be the problem Wolfgang >> mentioned at the time...will take a look tomorrow. > > Could you please try and look into this ASAP?

Re: [U-Boot] 'run' no longer aborts on an error, can someone please verify

2012-04-23 Thread Simon Glass
Hi Wolfgang, On Mon, Apr 23, 2012 at 11:16 PM, Wolfgang Denk wrote: > Dear Simon, > > In message > you > wrote: >> >> Thanks for the detailed information. This might be the problem Wolfgang >> mentioned at the time...will take a look tomorrow. > > Could you please try and look into this ASAP?

Re: [U-Boot] 'run' no longer aborts on an error, can someone please verify

2012-04-23 Thread Wolfgang Denk
Dear Simon, In message you wrote: > > Thanks for the detailed information. This might be the problem Wolfgang > mentioned at the time...will take a look tomorrow. Could you please try and look into this ASAP? I consider this a serious bug, and would like to fix it in mainline as soon as possib

Re: [U-Boot] 'run' no longer aborts on an error, can someone please verify

2012-04-23 Thread Simon Glass
Hi Timo, On Apr 23, 2012 5:43 PM, "Timo Ketola" wrote: > > Dear Gray, Jason, Heiko, Simon, > > I added you, Jason, Heiko and Simon, into the CC-list because there is > your commits in the history which seems somewhere to change the > behaviour below: > > On 22.04.2012 00:55, Gray Remlin wrote: >

Re: [U-Boot] 'run' no longer aborts on an error, can someone please verify

2012-04-23 Thread Wolfgang Denk
Dear Timo, In message <4f9506aa.4000...@exertus.fi> you wrote: > Hi again, > > On 23.04.2012 08:43, Timo Ketola wrote: > ... > > I took a quick glance over command/main.c > > common/main.c, of course > > > 1341 rc = cmd_process(flag, argc, argv, &repeatable); > > This might

Re: [U-Boot] 'run' no longer aborts on an error, can someone please verify

2012-04-23 Thread Wolfgang Denk
Dear Gray, In message <20120423082005.8497f200...@gemini.denx.de> I wrote: > > I cannot confirm a problem with the "run' command; tested with ... > [In both my tests the hush shell was used.] I confirm the problem for the old command line interpreter. Best regards, Wolfgang Denk -- DENX Soft

Re: [U-Boot] 'run' no longer aborts on an error, can someone please verify

2012-04-23 Thread Timo Ketola
On 23.04.2012 11:20, Wolfgang Denk wrote: > I cannot confirm a problem with the "run' command; tested with > v2012.04 in "sandbox" : > > > $ echo 'setenv fail printenv foo;setenv bug echo === BUG ===;run fail bug; > reset' | ./u-boot I can confirm it: EXE402

Re: [U-Boot] 'run' no longer aborts on an error, can someone please verify

2012-04-23 Thread Wolfgang Denk
Dear Gray Remlin, In message <4f932cec.30...@gmail.com> you wrote: > U-Boot v2012.04 > > Marvell>> setenv ipaddr '192.168.1.130' > Marvell>> setenv ifhostisup 'ping 192.168.1.100' > Marvell>> setenv saywearehappy 'echo "We are happy!"' > Marvell>> run ifhostisup saywearehappy; > Using egiga0 devi

Re: [U-Boot] 'run' no longer aborts on an error, can someone please verify

2012-04-23 Thread Timo Ketola
Hi again, On 23.04.2012 08:43, Timo Ketola wrote: ... > I took a quick glance over command/main.c common/main.c, of course > 1341 rc = cmd_process(flag, argc, argv, &repeatable); This might fix it: - rc = cmd_process(flag, argc, argv, &repeatable); +

Re: [U-Boot] 'run' no longer aborts on an error, can someone please verify

2012-04-22 Thread Timo Ketola
Dear Gray, Jason, Heiko, Simon, I added you, Jason, Heiko and Simon, into the CC-list because there is your commits in the history which seems somewhere to change the behaviour below: On 22.04.2012 00:55, Gray Remlin wrote: > U-Boot v2012.04 > > Marvell>> setenv ipaddr '192.168.1.130' > Marvell>

[U-Boot] 'run' no longer aborts on an error, can someone please verify

2012-04-21 Thread Gray Remlin
U-Boot v2012.04 Marvell>> setenv ipaddr '192.168.1.130' Marvell>> setenv ifhostisup 'ping 192.168.1.100' Marvell>> setenv saywearehappy 'echo "We are happy!"' Marvell>> run ifhostisup saywearehappy; Using egiga0 device ping failed; host 192.168.1.100 is not alive "We are happy!" Marvell>> This i