Merge pull request #351 from aminya/test [skip test]

fix: use logged in bash as the entry point of docker + add more docker tests

Author: aminya

.vscode/settings.json

@@ -72,5 +72,8 @@
   "prettier.enable": false,
   "javascript.preferences.importModuleSpecifierEnding": "js",
   "typescript.preferences.importModuleSpecifierEnding": "js",
-  "eslint.useESLintClass": true
+  "eslint.useESLintClass": true,
+  "yaml.schemas": {
+    "https://json.schemastore.org/container-structure-test.json": "/dev/docker/ci/tests/*.yml"
+  }
 }

README.md

@@ -338,15 +338,15 @@ RUN apt-get update -qq && \
     rm -rf /var/lib/apt/lists/* && \
     rm -rf /tmp/*
 
-ENTRYPOINT ["/bin/bash"]
+SHELL ["/bin/bash", "-l", "-c"]
+ENTRYPOINT ["/bin/bash", "-l"]
 
 #### Building (example)
 FROM setup-cpp-ubuntu AS builder
 
 COPY ./dev/cpp_vcpkg_project /home/app
 WORKDIR /home/app
-RUN bash -c 'source ~/.cpprc \
-    && task build'
+RUN task build
 
 #### Running environment
 # use a fresh image as the runner

dev/docker/ci/arch-gcc.dockerfile

@@ -8,4 +8,5 @@ RUN node --enable-source-maps /usr/lib/setup-cpp/setup-cpp.js \
     rm -rf /var/cache/pacman/pkg/* && \
     rm -rf /tmp/*
 
-ENTRYPOINT ["/bin/bash"]
+SHELL ["/bin/bash", "-l", "-c"]
+ENTRYPOINT ["/bin/bash", "-l"]

dev/docker/ci/arch-gcc.yml

@@ -8,4 +8,5 @@ RUN node --enable-source-maps /usr/lib/setup-cpp/setup-cpp.js \
     rm -rf /var/cache/pacman/pkg/* && \
     rm -rf /tmp/*
 
-ENTRYPOINT ["/bin/bash"]
+SHELL ["/bin/bash", "-l", "-c"]
+ENTRYPOINT ["/bin/bash", "-l"]

dev/docker/ci/arch-llvm.dockerfile

@@ -9,4 +9,5 @@ RUN node --enable-source-maps /usr/lib/setup-cpp/setup-cpp.js \
     rm -rf /var/cache/pacman/pkg/* && \
     rm -rf /tmp/*
 
-ENTRYPOINT ["/bin/bash"]
+SHELL ["/bin/bash", "-l", "-c"]
+ENTRYPOINT ["/bin/bash", "-l"]

dev/docker/ci/arch-llvm.yml

@@ -36,4 +36,5 @@ RUN pacman -Syuu --noconfirm && \
     rm -rf /var/cache/pacman/pkg/* && \
     rm -rf /tmp/*
 
-ENTRYPOINT ["/bin/bash"]
+SHELL ["/bin/bash", "-l", "-c"]
+ENTRYPOINT ["/bin/bash", "-l"]