Merge "Dockerfile cleanup"

8 files changed, 15 insertions, 9 deletions

diff --git a/docker/install_scripts/00_swift_needs.sh b/docker/install_scripts/00_swift_needs.sh
index 89e737f50..b83621d7b 100755
--- a/docker/install_scripts/00_swift_needs.sh
+++ b/docker/install_scripts/00_swift_needs.sh
@@ -1,9 +1,9 @@
 #!/bin/sh
+set -e
 
 # adduser -D -H syslog && \
 for user in "swift"; do
-  id -u $user > /dev/null 2>&1
-  if [ ! $? == '0' ]; then
+  if ! id -u $user > /dev/null 2>&1 ; then
     adduser -D $user
     printf "created user $user\n"
   fi
diff --git a/docker/install_scripts/10_apk_install_prereqs.sh b/docker/install_scripts/10_apk_install_prereqs.sh
index 879530bc5..eedff185a 100755
--- a/docker/install_scripts/10_apk_install_prereqs.sh
+++ b/docker/install_scripts/10_apk_install_prereqs.sh
@@ -1,4 +1,5 @@
 #!/bin/sh
+set -e
 
 echo "@testing http://dl-cdn.alpinelinux.org/alpine/edge/testing" >> /etc/apk/repositories
 apk add --update \
diff --git a/docker/install_scripts/15_install_python_and_pip_src.sh b/docker/install_scripts/15_install_python_and_pip_src.sh
index 286523db3..294ec92f1 100755
--- a/docker/install_scripts/15_install_python_and_pip_src.sh
+++ b/docker/install_scripts/15_install_python_and_pip_src.sh
@@ -1,4 +1,5 @@
 #!/bin/sh
+set -e
 
 echo
 echo
diff --git a/docker/install_scripts/20_pip_install_prereqs.sh b/docker/install_scripts/20_pip_install_prereqs.sh
index fe41b3b36..d9cc9e6b4 100755
--- a/docker/install_scripts/20_pip_install_prereqs.sh
+++ b/docker/install_scripts/20_pip_install_prereqs.sh
@@ -1,4 +1,5 @@
 #!/bin/sh
+set -e
 
 echo "+ + + + + + + + + + upgrading pip" && \
 pip install -U pip && \
diff --git a/docker/install_scripts/50_swift_install.sh b/docker/install_scripts/50_swift_install.sh
index 5a6ec2c6e..c189e4e44 100755
--- a/docker/install_scripts/50_swift_install.sh
+++ b/docker/install_scripts/50_swift_install.sh
@@ -1,4 +1,5 @@
 #!/bin/sh
+set -e
 
 pip install -U pip && \
 cd /opt/swift && \
diff --git a/docker/install_scripts/60_pip_uninstall_dev.sh b/docker/install_scripts/60_pip_uninstall_dev.sh
index 081b4292b..655aba5cc 100755
--- a/docker/install_scripts/60_pip_uninstall_dev.sh
+++ b/docker/install_scripts/60_pip_uninstall_dev.sh
@@ -1,4 +1,5 @@
 #!/bin/sh
+set -e
 
 echo "- - - - - - - - uninstalling simplejson"
 pip uninstall --yes simplejson
diff --git a/docker/install_scripts/99_apk_uninstall_dev.sh b/docker/install_scripts/99_apk_uninstall_dev.sh
index 469f12736..5109354e9 100755
--- a/docker/install_scripts/99_apk_uninstall_dev.sh
+++ b/docker/install_scripts/99_apk_uninstall_dev.sh
@@ -1,4 +1,5 @@
 #!/bin/sh
+set -e
 
 cd /
 rm -rf /build
diff --git a/docker/install_scripts/python_test_dirs b/docker/install_scripts/python_test_dirs
index 593815e8f..7441673ab 100644
--- a/docker/install_scripts/python_test_dirs
+++ b/docker/install_scripts/python_test_dirs
@@ -1,10 +1,10 @@
-/opt/python/usr/local/lib/python3.6//ctypes/test
-/opt/python/usr/local/lib/python3.6//distutils/tests
-/opt/python/usr/local/lib/python3.6//idlelib/idle_test
-/opt/python/usr/local/lib/python3.6//lib2to3/tests
-/opt/python/usr/local/lib/python3.6//sqlite3/test
-/opt/python/usr/local/lib/python3.6//test
-/opt/python/usr/local/lib/python3.6//tkinter/test
+/opt/python/usr/local/lib/python3.6/ctypes/test
+/opt/python/usr/local/lib/python3.6/distutils/tests
+/opt/python/usr/local/lib/python3.6/idlelib/idle_test
+/opt/python/usr/local/lib/python3.6/lib2to3/tests
+/opt/python/usr/local/lib/python3.6/sqlite3/test
+/opt/python/usr/local/lib/python3.6/test
+/opt/python/usr/local/lib/python3.6/tkinter/test
 /opt/python/usr/local/lib/python2.7/bsddb/test
 /opt/python/usr/local/lib/python2.7/ctypes/test
 /opt/python/usr/local/lib/python2.7/distutils/tests