mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Marc Reilly <marc@cpdesign.com.au>
To: Sascha Hauer <s.hauer@pengutronix.de>
Cc: barebox@lists.infradead.org
Subject: Re: handling script/init errors
Date: Tue, 12 Nov 2013 20:05:19 +1100	[thread overview]
Message-ID: <1714314.nh9XxNhNES@dev2.cpdesign> (raw)
In-Reply-To: <20131112082858.GJ24559@pengutronix.de>

Hi Sasha,

> > 
> > Is there a way to set up an error handler in the scripts? Ideally, a
> > command or script that could be called if /bin/sh encounters an error.
> 
> What would you consider an error? Is executing the 'false' command an
> error?
> Commands in scripts must be allowed to fail. You are supposed to catch
> this via
> 
> if [ $? != 0 ]; then
> 	echo "something bad happened"
> 	exit 1
> fi

"Error" is really a terrible, non-specific word. Sorry. 
And to make it worse I'm not really sure what the error, um, problem, truly 
is...

I scattered a few "false" commands in the init script and it continued onto 
the end, but when i added (on a board where there there is no bus #1):

{{{
# force reset audio dac in case audio playing during soft reset
 i2c_write -b 1 -a 0x47 -r 0x55 0x80
}}}

This causes the init script to just stop and drop to a prompt, assuming 
because bus #1 was not available. I haven't looked into how the errors/ return 
codes are different.

> 
> Of course error handling in shell is very cumbersome, even more in a
> restricted shell like hush. That's the reason I try to reduce the need
> of shell scripts in barebox. Most things that "really need to work" are
> better done in C.

I agree, especially "really need to work", (and that line above should have 
really been in C) ..
Although, one of the things that I've really loved about barebox is that 
ability to experiment and script with the shell, especially for basic testing.


> Maybe you could implement a 'catch' command. It would execute a command
> given as argument to the command. Something like:
> 
> CATCH_HANDLER=/env/bin/failure.sh
> ...
> catch <command>
> 
> Then whenever <command> fails $CATCH_HANDLER would be executed.
> 
> Don't know if that makes sense, just an idea.

I was initially thinking of something like bash's "set +e" or "trap" but I 
have neither good knowledge of how they work, or how they would/could be 
implemented in bareox.
(Just vague, fading memories :) )

Cheers,
Marc



_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

  reply	other threads:[~2013-11-12  9:21 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-12  6:07 Marc Reilly
2013-11-12  8:28 ` Sascha Hauer
2013-11-12  9:05   ` Marc Reilly [this message]
2013-11-12 10:11     ` Sascha Hauer
2013-11-13  6:51       ` Marc Reilly
2013-11-18  9:40         ` Sascha Hauer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1714314.nh9XxNhNES@dev2.cpdesign \
    --to=marc@cpdesign.com.au \
    --cc=barebox@lists.infradead.org \
    --cc=s.hauer@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox