[PATCH] buildscript: check selected_bsp link target

Adrian Schmutzler freifunk at adrianschmutzler.de
Fr Apr 17 14:21:13 CEST 2020


While buildscript checks whether selected_bsp exists, it does not
verify whether it is pointing to a sensible direction.

Since we use the target file name and content to derive build folders
etc., though, this patch adds a check and exits the script if the
link is broken.

Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
---
 buildscript | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/buildscript b/buildscript
index d2da0e30..9f6217c2 100755
--- a/buildscript
+++ b/buildscript
@@ -360,7 +360,14 @@ clean() {
 
 loadBSP()
 {
-    echo "Working with $(/bin/ls -l selected_bsp | awk '{ print $11 }')"
+    localbsppath=$(/bin/ls -l selected_bsp | awk '{ print $11 }')
+
+    if ! [ -s selected_bsp ]; then
+        echo "Error: $localbsppath does not exist. Aborting ..."
+        exit 1
+    fi
+
+    echo "Working with $localbsppath"
     . selected_bsp
     machine=$(basename $(realpath selected_bsp) .bsp)
 }
-- 
2.20.1



Mehr Informationen über die Mailingliste franken-dev