Fixes Building in Docker

build.sh: fixed errors and added parameters valid_targets, release_targets to get a shortened list of available targets.
cmake/docker.sh: avoid initializing cmake on every run of ./build.sh.
Dockerfile: fixed syntax and removed unnecessary checks and instructions.

Author: Lebensgefahr

Dockerfile

@@ -2,23 +2,24 @@ FROM ubuntu:jammy
 
 ARG USER_ID
 ARG GROUP_ID
-ENV DEBIAN_FRONTEND noninteractive
+ARG GDB
+
+ENV DEBIAN_FRONTEND=noninteractive
 
 RUN apt-get update && apt-get install -y git cmake make ruby gcc python3 python3-pip gcc-arm-none-eabi ninja-build
 
 RUN if [ "$GDB" = "yes" ]; then apt-get install -y gdb; fi
 
 RUN pip install pyyaml
 
-# if either of these are already set the same as the user's machine, leave them be and ignore the error
-RUN addgroup --gid $GROUP_ID inav; exit 0;
-RUN adduser --disabled-password --gecos '' --uid $USER_ID --gid $GROUP_ID inav; exit 0;
+# add inav user and group
+RUN addgroup --gid $GROUP_ID inav
+RUN adduser --disabled-password --gecos '' --uid $USER_ID --gid $GROUP_ID inav
 
 USER inav
 
 RUN git config --global --add safe.directory /src
 
-VOLUME /src
-
 WORKDIR /src/build
+
 ENTRYPOINT ["/src/cmake/docker.sh"]

build.sh

@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -e
+set -euo pipefail
 
 if [[ $# == 0 ]]; then
   echo -e "\
@@ -8,6 +8,10 @@ Usage syntax: ./build.sh <TARGET>
 Notes:
   * You can specify multiple targets.
     ./build.sh <TARGET_1> <TARGET_2> <TARGET_N>
+  * To get a list of release targets use \"release_targets\"
+    ./build.sh release_targets
+  * To get a list of valid targets use \"valid_targets\"
+    ./build.sh valid_targets
   * To get a list of all targets use \"help\". Hint: pipe the output through a pager.
     ./build.sh help | less
   * To build all targets use \"all\"
@@ -19,10 +23,20 @@ Notes:
   exit 1
 fi
 
+run_docker() {
+    docker run --rm -it -v "$(pwd)":/src inav-build "$@"
+}
+
 if [ -z "$(docker images -q inav-build)" ]; then
-  echo -e "*** Building image\n"
-  docker build -t inav-build --build-arg USER_ID="$(id -u)" --build-arg GROUP_ID="$(id -g)" .
-  echo -ne "\n"
+    echo "*** Building Docker image"
+    docker build -t inav-build \
+                 --build-arg USER_ID="$(id -u)" \
+                 --build-arg GROUP_ID="$(id -g)" .
+else
+    docker build -q -t inav-build \
+                 --build-arg USER_ID="$(id -u)" \
+                 --build-arg GROUP_ID="$(id -g)" . >/dev/null ||
+    { echo "*** Building Docker image: ERROR"; exit 1; }
 fi
 
 if [ ! -d ./build ]; then
@@ -40,10 +54,20 @@ if [ ! -d ./tools ]; then
   mkdir ./tools && chmod 777 ./tools
 fi
 
-echo -e "*** Building targets [$@]\n"
-docker run --rm -it -v "$(pwd)":/src inav-build $@
+case "$1" in
+    release_targets)
+        run_docker targets | sed -n 's/^Release targets: \(.*\)/\1/p'|tr ' ' '\n'
+    ;;
+    valid_targets)
+        run_docker targets | sed -n 's/^Valid targets: \(.*\)/\1/p'|tr ' ' '\n'
+    ;;
+    *)
+        echo -e "*** Building targets [$@]\n"
+        run_docker "$@"
+        if ls ./build/*.hex &> /dev/null; then
+            echo -e "\n*** Built targets in ./build:"
+            stat -c "%n (%.19y)" ./build/*.hex
+        fi
+    ;;
+esac
 
-if [ -z "$(ls ./build/*.hex &> /dev/null)" ]; then
-  echo -e "\n*** Built targets in ./build:"
-  stat -c "%n (%.19y)" ./build/*.hex
-fi

cmake/docker.sh

@@ -11,7 +11,7 @@ initialize_cmake() {
 }
 
 # Check if CMake has never been initialized
-if [ ! -f Makefile ]; then
+if [ ! -f build.ninja ]; then
     initialize_cmake
 fi