Re: [PATCH] Fix scripts/mkuboot.sh to return status

[Sam Ravnborg]

On Thu, Mar 03, 2005 at 04:16:41PM -0500, George G. Davis wrote:
> Sam,
> 
> If `mkimage` is either not found in search path or returns non-zero status,
> `make uImage` succeeds when it should fail. This changes scripts/mkuboot.sh
> to return status so build succeeds or fails as appropriate.


As per mail from Tom Roni I dropped this patch.

Sam
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH] Fix scripts/mkuboot.sh to return status

[George G. Davis]

Sam,

If `mkimage` is either not found in search path or returns non-zero status,
`make uImage` succeeds when it should fail. This changes scripts/mkuboot.sh
to return status so build succeeds or fails as appropriate.

Source: MontaVista Software, Inc.
MR: 10761
Type: Defect Fix
Disposition: submitted to linux-kernel@vger.kernel.org
Keywords:
Description:
Fix scripts/mkuboot.sh to return non-zero status to indicate
build failure when uImage target is not created.

Signed-off-by: George G. Davis <[EMAIL PROTECTED]>

Index: linux-2.6.11-bk/scripts/mkuboot.sh
===
--- linux-2.6.11-bk.orig/scripts/mkuboot.sh
+++ linux-2.6.11-bk/scripts/mkuboot.sh
@@ -9,8 +9,10 @@
 if [ -z "${MKIMAGE}" ]; then
# Doesn't exist
echo '"mkimage" command not found - U-Boot images will not be built' >&2
-   exit 0;
+   exit -1
 fi
 
 # Call "mkimage" to create U-Boot image
 ${MKIMAGE} "$@"
+
+exit $?

--
Regards,
George
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/