Update docker.yml (#23)

* Update docker.yml

Add support for bionic

* Update for bionic

* Updates

* Add git to install build tools

* Updates after shell script linter

* Undo script changes from linter since they broke script

* Address script linter warnings

* Update to .sh extension inline with google style guide

.github/workflows/docker.yml

@@ -13,6 +13,9 @@ jobs:
     strategy:
       matrix:
         include:
+          - architecture: x64
+            ubuntu-codename: bionic
+            target: x64-version
           - architecture: x64
             ubuntu-codename: focal
             target: x64-version

Dockerfile

@@ -15,7 +15,9 @@ RUN apt-get update
 FROM base AS x64-version
 
 # add build tools including Google test and mock
-RUN apt-get install -y cmake build-essential file libgtest-dev libgmock-dev gdb git
+COPY scripts/install_build_tools.sh /install_build_tools.sh
+RUN ./install_build_tools.sh
+
 
 FROM base AS cross-compile-version
 

scripts/install_build_tools.sh

@@ -0,0 +1,27 @@
+#!/bin/bash
+set -e
+# shellcheck source=/dev/null
+. /etc/lsb-release
+echo "Distribution code name : ${DISTRIB_CODENAME}"
+apt-get update
+if [ "$DISTRIB_CODENAME" = "bionic" ];then 
+    echo "Installing bionic build tools ..."
+    DEPENDENCIES=(
+        build-essential 
+        cmake 
+        googletest 
+        google-mock
+        gdb
+        git)
+else
+    echo "Installing other linux distribution (focal or jammy) build tools ..."
+    DEPENDENCIES=(
+        build-essential
+        cmake
+        build-essential
+        libgtest-dev
+        libgmock-dev
+        gdb
+        git)
+fi
+DEBIAN_FRONTEND=noninteractive apt install --no-install-recommends --yes --quiet "${DEPENDENCIES[@]}"