Messages in this thread | | | Subject | Re: [PATCH 1/2] merge_config.sh: use signal names compatible with dash and bash | From | John Stultz <> | Date | Mon, 21 Nov 2011 12:05:37 -0800 |
| |
On Mon, 2011-11-21 at 11:42 -0800, Darren Hart wrote: > The SIGHUP SIGINT and SIGTERM names caused failures when running > merge_config.sh with the dash shell. Dropping the "SIG" component > makes the script work in both bash and dash. > > Signed-off-by: Darren Hart <dvhart@linux.intel.com> Acked-by: John Stultz <john.stultz@linaro.org>
> --- > scripts/kconfig/merge_config.sh | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/scripts/kconfig/merge_config.sh b/scripts/kconfig/merge_config.sh > index 890276b..b91015d 100755 > --- a/scripts/kconfig/merge_config.sh > +++ b/scripts/kconfig/merge_config.sh > @@ -24,7 +24,7 @@ clean_up() { > rm -f $TMP_FILE > exit > } > -trap clean_up SIGHUP SIGINT SIGTERM > +trap clean_up HUP INT TERM > > usage() { > echo "Usage: $0 [OPTIONS] [CONFIG [...]]"
| |