From 43aa75dc89c60a994ed89a93b44622f01e9b90ba Mon Sep 17 00:00:00 2001
From: Robin Schneider <ypid@riseup.net>
Date: Tue, 31 Dec 2019 15:13:43 +0100
Subject: [PATCH] Fix cross platform build support, thanks to @dani-garcia for
 the review

---
 docker/Dockerfile.j2               | 47 ++++++++++++++++--------------
 docker/aarch64/mysql/Dockerfile    |  2 +-
 docker/aarch64/sqlite/Dockerfile   |  2 +-
 docker/amd64/mysql/Dockerfile      |  2 +-
 docker/amd64/postgresql/Dockerfile |  2 +-
 docker/amd64/sqlite/Dockerfile     |  2 +-
 docker/armv6/mysql/Dockerfile      |  2 +-
 docker/armv6/sqlite/Dockerfile     |  2 +-
 docker/armv7/mysql/Dockerfile      |  2 +-
 docker/armv7/sqlite/Dockerfile     |  2 +-
 10 files changed, 34 insertions(+), 31 deletions(-)

diff --git a/docker/Dockerfile.j2 b/docker/Dockerfile.j2
index 0298a334..f8cb0297 100644
--- a/docker/Dockerfile.j2
+++ b/docker/Dockerfile.j2
@@ -2,33 +2,37 @@
 # 	https://docs.docker.com/develop/develop-images/multistage-build/
 # 	https://whitfin.io/speeding-up-rust-docker-builds/
 ####################### VAULT BUILD IMAGE  #######################
+{% set build_stage_base_image = "rust:1.40" %}
+{% set vault_stage_base_image = build_stage_base_image %}
 {% if "alpine" in target_file %}
-{%   set preferred_base_image = "alpine:3.11" %}
+{%   set build_stage_base_image = "clux/muslrust:nightly-2019-12-19" %}
+{%   set runtime_stage_base_image = "alpine:3.11" %}
+{%   set vault_stage_base_image = runtime_stage_base_image %}
 {%   set package_arch_name = "" %}
 {% elif "amd64" in target_file %}
-{%   set preferred_base_image = "debian:buster-slim" %}
+{%   set runtime_stage_base_image = "debian:buster-slim" %}
 {%   set package_arch_name = "" %}
 {% elif "aarch64" in target_file %}
-{%   set preferred_base_image = "balenalib/aarch64-debian:buster" %}
+{%   set runtime_stage_base_image = "balenalib/aarch64-debian:buster" %}
 {%   set package_arch_name = "arm64" %}
 {% elif "armv6" in target_file %}
-{%   set preferred_base_image = "balenalib/rpi-debian:buster" %}
+{%   set runtime_stage_base_image = "balenalib/rpi-debian:buster" %}
 {%   set package_arch_name = "armel" %}
 {% elif "armv7" in target_file %}
-{%   set preferred_base_image = "balenalib/armv7hf-debian:buster" %}
+{%   set runtime_stage_base_image = "balenalib/armv7hf-debian:buster" %}
 {%   set package_arch_name = "armhf" %}
 {% endif %}
 {% set package_arch_prefix = ":" + package_arch_name %}
 {% if package_arch_name == "" %}
 {%   set package_arch_prefix = "" %}
 {% endif %}
-FROM {{ preferred_base_image }} as vault
+FROM {{ vault_stage_base_image }} as vault
 
 ENV VAULT_VERSION "v2.12.0b"
 
 ENV URL "https://github.com/dani-garcia/bw_web_builds/releases/download/$VAULT_VERSION/bw_web_$VAULT_VERSION.tar.gz"
 
-{% if "alpine" in target_file %}
+{% if "alpine" in vault_stage_base_image %}
 RUN apk add --no-cache --upgrade \
     curl \
     tar
@@ -47,7 +51,7 @@ RUN apt update -y \
 RUN mkdir /web-vault
 WORKDIR /web-vault
 
-{% if "alpine" in target_file %}
+{% if "alpine" in vault_stage_base_image %}
 SHELL ["/bin/ash", "-o", "nounset", "-o", "pipefail", "-o", "errexit", "-c"]
 {% else %}
 SHELL ["/bin/bash", "-o", "nounset", "-o", "pipefail", "-o", "errexit", "-c"]
@@ -57,14 +61,13 @@ RUN curl -L $URL | tar xz
 RUN ls
 
 ########################## BUILD IMAGE  ##########################
-{% if "alpine" in target_file %}
+{% if "musl" in build_stage_base_image %}
 # Musl build image for statically compiled binary
-FROM clux/muslrust:nightly-2019-12-19 as build
 {% else %}
 # We need to use the Rust build image, because
 # we need the Rust compiler and Cargo tooling
-FROM rust:1.40 as build
 {% endif %}
+FROM {{ build_stage_base_image }} as build
 
 {% if "sqlite" in target_file %}
 # set sqlite as default for DB ARG for backward compatibility
@@ -199,12 +202,12 @@ RUN cargo build --features ${DB} --release --target=aarch64-unknown-linux-gnu
 ######################## RUNTIME IMAGE  ########################
 # Create a new stage with a minimal image
 # because we already have a binary built
-FROM {{ preferred_base_image }}
+FROM {{ runtime_stage_base_image }}
 
 ENV ROCKET_ENV "staging"
 ENV ROCKET_PORT=80
 ENV ROCKET_WORKERS=10
-{% if "alpine" in target_file %}
+{% if "alpine" in runtime_stage_base_image %}
 ENV SSL_CERT_DIR=/etc/ssl/certs
 {% endif %}
 
@@ -213,17 +216,17 @@ RUN [ "cross-build-start" ]
 {% endif %}
 
 # Install needed libraries
-{% if "alpine" in target_file %}
+{% if "alpine" in runtime_stage_base_image %}
 RUN apk add --no-cache \
         openssl \
         curl \
-{% if "sqlite" in target_file %}
+{%   if "sqlite" in target_file %}
         sqlite \
-{% elif "mysql" in target_file %}
+{%   elif "mysql" in target_file %}
         mariadb-connector-c \
-{% elif "postgresql" in target_file %}
+{%   elif "postgresql" in target_file %}
         postgresql-libs \
-{% endif %}
+{%   endif %}
         ca-certificates
 {% else %}
 RUN apt-get update && apt-get install -y \
@@ -231,13 +234,13 @@ RUN apt-get update && apt-get install -y \
     openssl \
     ca-certificates \
     curl \
-{% if "sqlite" in target_file %}
+{%   if "sqlite" in target_file %}
     sqlite3 \
-{% elif "mysql" in target_file %}
+{%   elif "mysql" in target_file %}
     libmariadbclient-dev \
-{% elif "postgresql" in target_file %}
+{%   elif "postgresql" in target_file %}
     libpq5 \
-{% endif %}
+{%   endif %}
     && rm -rf /var/lib/apt/lists/*
 {% endif %}
 
diff --git a/docker/aarch64/mysql/Dockerfile b/docker/aarch64/mysql/Dockerfile
index e0b06554..6562d4d8 100644
--- a/docker/aarch64/mysql/Dockerfile
+++ b/docker/aarch64/mysql/Dockerfile
@@ -2,7 +2,7 @@
 # 	https://docs.docker.com/develop/develop-images/multistage-build/
 # 	https://whitfin.io/speeding-up-rust-docker-builds/
 ####################### VAULT BUILD IMAGE  #######################
-FROM balenalib/aarch64-debian:buster as vault
+FROM rust:1.40 as vault
 
 ENV VAULT_VERSION "v2.12.0b"
 
diff --git a/docker/aarch64/sqlite/Dockerfile b/docker/aarch64/sqlite/Dockerfile
index e63e755e..cea96491 100644
--- a/docker/aarch64/sqlite/Dockerfile
+++ b/docker/aarch64/sqlite/Dockerfile
@@ -2,7 +2,7 @@
 # 	https://docs.docker.com/develop/develop-images/multistage-build/
 # 	https://whitfin.io/speeding-up-rust-docker-builds/
 ####################### VAULT BUILD IMAGE  #######################
-FROM balenalib/aarch64-debian:buster as vault
+FROM rust:1.40 as vault
 
 ENV VAULT_VERSION "v2.12.0b"
 
diff --git a/docker/amd64/mysql/Dockerfile b/docker/amd64/mysql/Dockerfile
index 8d789c82..d59c8953 100644
--- a/docker/amd64/mysql/Dockerfile
+++ b/docker/amd64/mysql/Dockerfile
@@ -2,7 +2,7 @@
 # 	https://docs.docker.com/develop/develop-images/multistage-build/
 # 	https://whitfin.io/speeding-up-rust-docker-builds/
 ####################### VAULT BUILD IMAGE  #######################
-FROM debian:buster-slim as vault
+FROM rust:1.40 as vault
 
 ENV VAULT_VERSION "v2.12.0b"
 
diff --git a/docker/amd64/postgresql/Dockerfile b/docker/amd64/postgresql/Dockerfile
index 7b9818a6..bf1aa2f7 100644
--- a/docker/amd64/postgresql/Dockerfile
+++ b/docker/amd64/postgresql/Dockerfile
@@ -2,7 +2,7 @@
 # 	https://docs.docker.com/develop/develop-images/multistage-build/
 # 	https://whitfin.io/speeding-up-rust-docker-builds/
 ####################### VAULT BUILD IMAGE  #######################
-FROM debian:buster-slim as vault
+FROM rust:1.40 as vault
 
 ENV VAULT_VERSION "v2.12.0b"
 
diff --git a/docker/amd64/sqlite/Dockerfile b/docker/amd64/sqlite/Dockerfile
index 55a5dc04..56cf4bee 100644
--- a/docker/amd64/sqlite/Dockerfile
+++ b/docker/amd64/sqlite/Dockerfile
@@ -2,7 +2,7 @@
 # 	https://docs.docker.com/develop/develop-images/multistage-build/
 # 	https://whitfin.io/speeding-up-rust-docker-builds/
 ####################### VAULT BUILD IMAGE  #######################
-FROM debian:buster-slim as vault
+FROM rust:1.40 as vault
 
 ENV VAULT_VERSION "v2.12.0b"
 
diff --git a/docker/armv6/mysql/Dockerfile b/docker/armv6/mysql/Dockerfile
index d05c21aa..ed389253 100644
--- a/docker/armv6/mysql/Dockerfile
+++ b/docker/armv6/mysql/Dockerfile
@@ -2,7 +2,7 @@
 # 	https://docs.docker.com/develop/develop-images/multistage-build/
 # 	https://whitfin.io/speeding-up-rust-docker-builds/
 ####################### VAULT BUILD IMAGE  #######################
-FROM balenalib/rpi-debian:buster as vault
+FROM rust:1.40 as vault
 
 ENV VAULT_VERSION "v2.12.0b"
 
diff --git a/docker/armv6/sqlite/Dockerfile b/docker/armv6/sqlite/Dockerfile
index 64255111..98afd9ba 100644
--- a/docker/armv6/sqlite/Dockerfile
+++ b/docker/armv6/sqlite/Dockerfile
@@ -2,7 +2,7 @@
 # 	https://docs.docker.com/develop/develop-images/multistage-build/
 # 	https://whitfin.io/speeding-up-rust-docker-builds/
 ####################### VAULT BUILD IMAGE  #######################
-FROM balenalib/rpi-debian:buster as vault
+FROM rust:1.40 as vault
 
 ENV VAULT_VERSION "v2.12.0b"
 
diff --git a/docker/armv7/mysql/Dockerfile b/docker/armv7/mysql/Dockerfile
index 90bb9d6a..6e32c8c9 100644
--- a/docker/armv7/mysql/Dockerfile
+++ b/docker/armv7/mysql/Dockerfile
@@ -2,7 +2,7 @@
 # 	https://docs.docker.com/develop/develop-images/multistage-build/
 # 	https://whitfin.io/speeding-up-rust-docker-builds/
 ####################### VAULT BUILD IMAGE  #######################
-FROM balenalib/armv7hf-debian:buster as vault
+FROM rust:1.40 as vault
 
 ENV VAULT_VERSION "v2.12.0b"
 
diff --git a/docker/armv7/sqlite/Dockerfile b/docker/armv7/sqlite/Dockerfile
index 28851b67..ba9bb117 100644
--- a/docker/armv7/sqlite/Dockerfile
+++ b/docker/armv7/sqlite/Dockerfile
@@ -2,7 +2,7 @@
 # 	https://docs.docker.com/develop/develop-images/multistage-build/
 # 	https://whitfin.io/speeding-up-rust-docker-builds/
 ####################### VAULT BUILD IMAGE  #######################
-FROM balenalib/armv7hf-debian:buster as vault
+FROM rust:1.40 as vault
 
 ENV VAULT_VERSION "v2.12.0b"