better name for prereq check
authorJeff Atwood <jatwood@codinghorror.com>
Wed, 27 Apr 2016 00:36:58 +0000 (17:36 -0700)
committerJeff Atwood <jatwood@codinghorror.com>
Wed, 27 Apr 2016 00:36:58 +0000 (17:36 -0700)
launcher

index 44a333839c6d02d0b190894aa83c5414f4304b7f..5eb03c648ff063109ba708e823539c4cef716ad4 100755 (executable)
--- a/launcher
+++ b/launcher
@@ -9,10 +9,10 @@ usage () {
   echo "    destroy:    Stop and remove a container"
   echo "    enter:      Use nsenter to enter a container"
   echo "    logs:       Docker logs for container"
-  echo "    setup:      Create a configuration file"
   echo "    bootstrap:  Bootstrap a container for the config based on a template"
   echo "    rebuild:    Rebuild a container (destroy old, bootstrap, start new)"
   echo "    cleanup:    Remove all containers that have stopped for > 24 hours"
+  echo "    setup:      Create a new configuration file and bootstrap"
   echo
   echo "Options:"
   echo "    --skip-prereqs             Don't check launcher prerequisites"
@@ -21,17 +21,18 @@ usage () {
   exit 1
 }
 
-[ $# -lt 2 ] && {
-  usage
-}
 
 command=$1
 config=$2
 shift 2
 user_args=""
 
+[ $# -lt 2 ] && {
+  usage
+}
+
 while [ ${#} -gt 0 ]; do
-  case "${1}" in
+  case "$command" in
 
   --skip-prereqs)
     SKIP_PREREQ="1"
@@ -133,7 +134,7 @@ install_docker() {
   exit 1
 }
 
-prereqs() {
+check_prereqs() {
 
   if [ -z $docker_path ]; then
     install_docker
@@ -164,13 +165,13 @@ prereqs() {
   test=($($docker_path --version))  # Get docker version string
   test=${test[2]//,/}  # Get version alone and strip comma if exists
 
-  # At least minimum version
+  # At least minimum docker version
   if compare_version "${docker_min_version}" "${test}"; then
     echo "ERROR: Docker version ${test} not supported, please upgrade to at least ${docker_min_version}, or recommended ${docker_rec_version}"
     exit 1
   fi
 
-  # Recommend best version
+  # Recommend newer docker version
   if compare_version "${docker_rec_version}" "${test}"; then
     echo "WARNING: Docker version ${test} deprecated, recommend upgrade to ${docker_rec_version} or newer."
   fi
@@ -277,7 +278,7 @@ check_ports() {
 }
 
 if [ -z "$SKIP_PREREQS" ] ; then
-  prereqs
+  check_prereqs
 fi
 
 host_run() {