Browse Source

Require MINECRAFT_VERSION to be set manually, make no assumptions

Mark Ide 8 years ago
parent
commit
fd1154d0f8
2 changed files with 8 additions and 2 deletions
  1. 0 2
      Dockerfile
  2. 8 0
      entrypoint.sh

+ 0 - 2
Dockerfile

@@ -8,8 +8,6 @@
 
 FROM debian:jessie
 
-ENV MINECRAFT_VERSION 1.11.2
-
 RUN echo "deb http://overviewer.org/debian ./" >> /etc/apt/sources.list && \
     apt-get update && \
     apt-get install -y wget && \

+ 8 - 0
entrypoint.sh

@@ -1,3 +1,11 @@
+#!/bin/bash
+set -o errexit
+
+if [ -z "$MINECRAFT_VERSION" ]; then
+  echo "Expecting environment variable MINECRAFT_VERSION to be set to non-empty string. Exiting."
+  exit 1
+fi
+
 wget https://s3.amazonaws.com/Minecraft.Download/versions/${MINECRAFT_VERSION}/${MINECRAFT_VERSION}.jar -P /home/minecraft/.minecraft/versions/${MINECRAFT_VERSION}/
 
 overviewer.py --config /home/minecraft/config.py