freifunk/Firmware-building Changeset - 73323e3e12b0 · Chaos Computer Club Lëtzebuerg Projects

Changeset - 73323e3e12b0

Parent rev.

Child rev.

[Not reviewed]

0 1 0

x - 20 days ago 2025-10-15 14:54:44
xbr@c3l.lu

fix: build.sh, a lot of shellcheck + enable functions

1 file changed with 7 insertions and 19 deletions:

build_script/build.sh

0 comments (0 inline, 0 general)

build_script/build.sh

➞

Show inline comments

 #!/usr/bin/env bash
 USER_DIR=/home/user
 GLUON_DIR=$USER_DIR/gluon
 SITE_DIR=$USER_DIR/Firmware-building/site_config
 EXPERIMENTAL_SITE=$SITE_DIR/experimental/site
 BETA_SITE=$SITE_DIR/beta/site
 STABLE_SITE=$SITE_DIR/stable/site
 # $1: site directory
 set_site_directory() {
     pushd $GLUON_DIR
+    pushd "$GLUON_DIR"
     # if site file exists, remove it
     if [ -a site ]; then
         unlink site
     fi
     ln -s $1
+    ln -s "$1"
     popd
+}
 build() {
     pushd $GLUON_DIR
+    pushd "$GLUON_DIR"
     for TARGET in $(make list-targets); do
         make -j16 GLUON_TARGET=$TARGET
+        make -j16 GLUON_TARGET="$TARGET"
     done
     popd
+}
 # $1: autoupdater branch
 move_output_images() {
     pushd $GLUON_DIR
+    pushd "$GLUON_DIR"
     mv output/images output-images-$1
+    mv output/images "output-images-$1"
     popd
+}
 #unset multiple env vars
 unset BUILD_EXPERIMENTAL BUILD_BETA BUILD_STABLE
 case $1 in
     exp)
         BUILD_EXPERIMENTAL=1
         ;;
@@ @@ -64,40 +64,28 @@ case $1 in @@
     *)
         echo "fflux firmware building script"
         echo "Usage: build.sh <all|exp|beta|stable>"
         exit 1
         ;;
 esac
 pushd $GLUON_DIR
 if [ -d $GLUON_DIR/output/images ]; then
     echo "Error: output images directory contains images. Please remove them before building."
-    echo "Delete `gluon/output/images` directory and try again."
+    echo "Delete 'gluon/output/images' directory and try again."
     exit 1
 fi
 set_site_directory() {
     echo "Linking $1"
+}
 build() {
     echo "Building images"
+}
 move_output_images() {
     echo "Moving output images to output-images-$1"
+}
 if [ -n "$BUILD_EXPERIMENTAL" ]; then
     set_site_directory $EXPERIMENTAL_SITE
     build
     move_output_images experimental
 fi
 if [ -n "$BUILD_BETA" ]; then
     set_site_directory $BETA_SITE
     build
     move_output_images beta
 fi

0 comments (0 inline, 0 general)