[U-Boot] [PATCH v2] mkconfig: deny messed up ARCH definition

Refuse to setup a platform if the command line ARCH= is not the same as the one required for the board. This prevents any user with prehistoric aliases from messing up their builds.
Reported in thread: http://old.nabble.com/-U-Boot--Build-breaks-on-some-OMAP3-configs-to26132721...
Inputs from: Mike Frysinger and Wolfgang Denk: http://lists.denx.de/pipermail/u-boot/2009-November/063642.html
Cc: Wolfgang Denk wd@denx.de Cc: Mike Frysinger vapier@gentoo.org Cc: Anand Gadiyar gadiyar@ti.com Cc: Dirk Behme dirk.behme@googlemail.com Signed-off-by: Nishanth Menon nm@ti.com --- mkconfig | 5 +++++ 1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/mkconfig b/mkconfig index 4c5675b..1075510 100755 --- a/mkconfig +++ b/mkconfig @@ -27,6 +27,11 @@ done [ $# -lt 4 ] && exit 1 [ $# -gt 6 ] && exit 1
+if [ "${ARCH}" -a "${ARCH}" != "$2" ]; then + echo "Failed: $ARCH=${ARCH}, should be '$2' for ${BOARD_NAME}" 1>&2 + exit 1 +fi + echo "Configuring for ${BOARD_NAME} board..."
#

Dear Nishanth Menon,
In message 1257176418-11624-1-git-send-email-nm@ti.com you wrote:
Refuse to setup a platform if the command line ARCH= is not the same as the one required for the board. This prevents any user with prehistoric aliases from messing up their builds.
Reported in thread: http://old.nabble.com/-U-Boot--Build-breaks-on-some-OMAP3-configs-to26132721...
Inputs from: Mike Frysinger and Wolfgang Denk: http://lists.denx.de/pipermail/u-boot/2009-November/063642.html
Cc: Wolfgang Denk wd@denx.de Cc: Mike Frysinger vapier@gentoo.org Cc: Anand Gadiyar gadiyar@ti.com Cc: Dirk Behme dirk.behme@googlemail.com Signed-off-by: Nishanth Menon nm@ti.com
mkconfig | 5 +++++ 1 files changed, 5 insertions(+), 0 deletions(-)
Applied, thanks.
Best regards,
Wolfgang Denk
participants (2)
-
Nishanth Menon
-
Wolfgang Denk