Add shellcheck to validate bash

Help to avoid bugs

Signed-off-by: Eric Curtin <eric.curtin@docker.com>

Author: ericcurtin

Dockerfile

@@ -40,6 +40,11 @@ ARG LLAMA_SERVER_VARIANT
 # Create non-root user
 RUN groupadd --system modelrunner && useradd --system --gid modelrunner --create-home --home-dir /home/modelrunner modelrunner
 
+COPY scripts/apt-install.sh apt-install.sh
+
+# Install ca-certificates for HTTPS and vulkan
+RUN ./apt-install.sh
+
 # Install ca-certificates for HTTPS and vulkan
 RUN apt-get update && \
     packages="ca-certificates" && \

Makefile

@@ -10,7 +10,7 @@ MODELS_PATH := $(shell pwd)/models-store
 LLAMA_ARGS ?=
 
 # Main targets
-.PHONY: build run clean test docker-build docker-run help
+.PHONY: build run clean test docker-build docker-run help validate
 
 # Default target
 .DEFAULT_GOAL := help
@@ -34,6 +34,9 @@ clean:
 test:
 	go test -v ./...
 
+validate:
+	find . -name '*.sh' -exec shellcheck {} +
+
 # Build Docker image
 docker-build:
 	docker buildx build \

scripts/apt-install.sh

@@ -0,0 +1,17 @@
+#!/bin/bash
+
+main() {
+  set -eux -o pipefail
+
+  apt-get update
+  local packages=("ca-certificates")
+  if [ "${LLAMA_SERVER_VARIANT}" = "generic" ] || [ "${LLAMA_SERVER_VARIANT}" = "cpu" ]; then
+      packages+=("libvulkan1")
+  fi
+
+  apt-get install -y --no-install-recommends "${packages[@]}"
+  rm -rf /var/lib/apt/lists/*
+}
+
+main "$@"
+