Merge pull request #92 from carlossg/issue-46
Last line of plugins.txt ignored if a new line is not present
diff --git a/Dockerfile b/Dockerfile
index 6fb06b5..71c3430 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -1,4 +1,4 @@
-FROM java:openjdk-7u65-jdk
+FROM java:8u45-jdk
RUN apt-get update && apt-get install -y wget git curl zip && rm -rf /var/lib/apt/lists/*
@@ -26,7 +26,7 @@
# could use ADD but this one does not check Last-Modified header
# see https://github.com/docker/docker/issues/8331
-RUN curl -L http://mirrors.jenkins-ci.org/war-stable/$JENKINS_VERSION/jenkins.war -o /usr/share/jenkins/jenkins.war \
+RUN curl -fL http://mirrors.jenkins-ci.org/war-stable/$JENKINS_VERSION/jenkins.war -o /usr/share/jenkins/jenkins.war \
&& echo "$JENKINS_SHA /usr/share/jenkins/jenkins.war" | sha1sum -c -
ENV JENKINS_UC https://updates.jenkins-ci.org
diff --git a/README.md b/README.md
index 9846eab..76d486b 100644
--- a/README.md
+++ b/README.md
@@ -88,7 +88,7 @@
FROM jenkins
# if we want to install via apt
USER root
-RUN apt-get install -y ruby make more-thing-here
+RUN apt-get update && apt-get install -y ruby make more-thing-here
USER jenkins # drop back to the regular jenkins user - good practice
```
diff --git a/jenkins.sh b/jenkins.sh
index da0dd65..96db063 100755
--- a/jenkins.sh
+++ b/jenkins.sh
@@ -1,5 +1,7 @@
#! /bin/bash
+set -e
+
# Copy files from /usr/share/jenkins/ref into /var/jenkins_home
# So the initial JENKINS-HOME is set with expected content.
# Don't override, as this is just a reference setup, and use from UI