summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPeter Tyser <ptyser@xes-inc.com>2010-10-29 17:59:06 -0500
committerWolfgang Denk <wd@denx.de>2010-11-14 23:49:37 +0100
commitcd57b0bb8b86ef52dc89ce8191b95bb85134634a (patch)
treee838e7bcf1e799595c59d88a914af349b9b8c03d
parent009884ae35362d6a4edf04c4af8aad78d415a261 (diff)
downloadu-boot-imx-cd57b0bb8b86ef52dc89ce8191b95bb85134634a.zip
u-boot-imx-cd57b0bb8b86ef52dc89ce8191b95bb85134634a.tar.gz
u-boot-imx-cd57b0bb8b86ef52dc89ce8191b95bb85134634a.tar.bz2
MAKEALL: Do a sanity check on user-supplied arguments
Add a check to make sure that the user's arguments actually find a board in boards.cfg. Previously, if a user misspelled an argument the argument would be discarded without warning. For example, running 'MAKEALL -c 85xx' with the intention of compiling all Freescale 85xx boards would instead silently discard the '-c 85xx' argument since the proper cpu name is 'mpc85xx' and then proceed to compile all PowerPC boards (MAKEALL's default). Also fix an unrelated typo. Signed-off-by: Peter Tyser <ptyser@xes-inc.com>
-rwxr-xr-xMAKEALL8
1 files changed, 7 insertions, 1 deletions
diff --git a/MAKEALL b/MAKEALL
index c54c6e8..767d561 100755
--- a/MAKEALL
+++ b/MAKEALL
@@ -11,7 +11,7 @@
# line; without any arguments, MAKEALL defaults to building all Power
# Architecture systems (i. e. same as for "MAKEALL powerpc").
#
-# With the iontroduction of the board.cfg file, it has become possible
+# With the introduction of the board.cfg file, it has become possible
# to provide additional selections. We use standard command line
# options for this:
#
@@ -125,6 +125,12 @@ FILTER="\$1 !~ /^#/"
if [ "$SELECTED" ] ; then
SELECTED=$(awk '('"$FILTER"') { print $1 }' boards.cfg)
+
+ # Make sure some boards from boards.cfg are actually found
+ if [ -z "$SELECTED" ] ; then
+ echo "Error: No boards selected, invalid arguments"
+ exit 1
+ fi
fi
#########################################################################