Messages in this thread Patch in this message | ![/](/images/icornerl.gif) | | Date | Sat, 10 Feb 2024 22:19:32 +0100 | From | Nicolas Schier <> | Subject | Re: [PATCH 1/2] kbuild: Abort make on install failures |
| |
On Sat, Feb 10, 2024 at 10:29:00AM +0000 Russell King (Oracle) wrote: > On Sat, Feb 10, 2024 at 03:46:00PM +0800, Zhang Bingwu wrote: > > From: Zhang Bingwu <xtexchooser@duck.com> > > > > Setting '-e' flag tells shells to exit with error exit code immediately > > after any of commands fails, and causes make(1) to regard recipes as > > failed. > > > > Before this, make will still continue to succeed even after the > > installation failed, for example, for insufficient permission or > > directory does not exist. > > > > Signed-off-by: Zhang Bingwu <xtexchooser@duck.com> > > ---
Thanks for fixing!
[...] > > diff --git a/arch/arm/boot/install.sh b/arch/arm/boot/install.sh > > index 9ec11fac7d8d..34e2c6e31fd1 100755 > > --- a/arch/arm/boot/install.sh > > +++ b/arch/arm/boot/install.sh > > @@ -17,6 +17,8 @@ > > # $3 - kernel map file > > # $4 - default install path (blank if root directory) > > > > +set -e > > + > > What about #!/bin/sh -e on the first line, which is the more normal way > to do this for an entire script?
are you sure? I can find many more occurrences of 'set -e' than the shebang version in the Linux tree, especially in the kbuild scripts, thus it's bike-shedding, isn't it?
Reviewed-by: Nicolas Schier <nicolas@fjasle.eu>
Kind regards, Nicolas [unhandled content-type:application/pgp-signature] | ![\](/images/icornerr.gif) |