diff options
author | Peter Tyser <ptyser@xes-inc.com> | 2010-10-29 17:59:06 -0500 |
---|---|---|
committer | Wolfgang Denk <wd@denx.de> | 2010-11-14 23:49:37 +0100 |
commit | cd57b0bb8b86ef52dc89ce8191b95bb85134634a (patch) | |
tree | e838e7bcf1e799595c59d88a914af349b9b8c03d | |
parent | 009884ae35362d6a4edf04c4af8aad78d415a261 (diff) | |
download | u-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-x | MAKEALL | 8 |
1 files changed, 7 insertions, 1 deletions
@@ -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 ######################################################################### |