Merge pull request #76 from jacobwgillespie/main

Move `apt-get` to top of Dockerfile for better caching
getmetal · Aug 23, 2023 · bc65668 · bc65668
2 parents 91a1caa + 6eb1d62
commit bc65668
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/Dockerfile b/Dockerfile
@@ -1,16 +1,16 @@
 FROM rust:1.68-bullseye as build
 
+# had to add this for open-ssl
+RUN apt-get update -y && \
+  apt-get install -y pkg-config make g++ libssl-dev ca-certificates && \
+  rustup target add x86_64-unknown-linux-gnu
+
 RUN USER=root cargo new --bin motorhead
 WORKDIR /motorhead
 
 COPY ./Cargo.lock ./Cargo.lock
 COPY ./Cargo.toml ./Cargo.toml
 
-# had to add this for open-ssl
-RUN apt-get update -y && \
-  apt-get install -y pkg-config make g++ libssl-dev ca-certificates && \
-  rustup target add x86_64-unknown-linux-gnu
-
 # cache dependencies
 ENV CARGO_NET_GIT_FETCH_WITH_CLI=true
 RUN cargo build --release