Merge pull request #272 from aston-r/download-latest-version-if-regex-is-used

Download latest version if user uses regex and TFENV_AUTO_INSTALL is true

Author: Zordrak

libexec/tfenv-version-name

@@ -60,7 +60,7 @@ done;
 # Begin Script Body #
 #####################
 
-if [ -z "${TFENV_TERRAFORM_VERSION:-""}" ]; then
+if [[ -z "${TFENV_TERRAFORM_VERSION:-""}" ]]; then
   TFENV_VERSION_FILE="$(tfenv-version-file)" \
     && log 'debug' "TFENV_VERSION_FILE retrieved from tfenv-version-file: ${TFENV_VERSION_FILE}" \
     || log 'error' 'Failed to retrieve TFENV_VERSION_FILE from tfenv-version-file';
@@ -74,7 +74,7 @@ else
     && log 'debug' "TFENV_VERSION specified in TFENV_TERRAFORM_VERSION: ${TFENV_VERSION}";
 
   TFENV_VERSION_SOURCE='TFENV_TERRAFORM_VERSION';
-fi
+fi;
 
 if [[ "${TFENV_VERSION}" =~ ^latest.*$ ]]; then
   log 'debug' "TFENV_VERSION uses 'latest' keyword: ${TFENV_VERSION}";
@@ -86,23 +86,39 @@ if [[ "${TFENV_VERSION}" =~ ^latest.*$ ]]; then
     regex="${TFENV_VERSION##*\:}";
     log 'debug' "'latest' keyword uses regex: ${regex}";
   else
-    regex='.*'; # Just saves a seperate command below without the grep
-    log 'debug' "'latest' keyword does not use regex";
+    regex="^[0-9]\+\.[0-9]\+\.[0-9]\+$";
+    log 'debug' "Version uses latest keyword alone. Forcing regex to match stable versions only: ${regex}";
   fi;
 
-  version="$(\find "${TFENV_CONFIG_DIR}/versions/" -type d -exec basename {} \; \
-    | tail -n +2 \
-    | sort -t'.' -k 1nr,1 -k 2nr,2 -k 3nr,3 \
-    | grep -e "${regex}" \
-    | head -n 1)";
+  declare local_version='';
+  if [[ -d "${TFENV_CONFIG_DIR}/versions" ]]; then
+    local_version="$(\find "${TFENV_CONFIG_DIR}/versions/" -type d -exec basename {} \; \
+      | tail -n +2 \
+      | sort -t'.' -k 1nr,1 -k 2nr,2 -k 3nr,3 \
+      | grep -e "${regex}" \
+      | head -n 1)";
+  fi;
 
-  if [ -n "${version}" ]; then
-    log 'debug' "Version selected: ${version}";
-    TFENV_VERSION="${version}"
+  if [[ "${TFENV_AUTO_INSTALL:-true}" == "true" ]]; then
+    log 'debug' "Trying to find the remote version using the regex: ${regex}";
+    remote_version="$(tfenv-list-remote | grep -e "${regex}" | head -n 1)";
+    if [[ -n "${remote_version}" ]]; then
+        if [[ "${local_version}" != "${remote_version}" ]]; then
+          log 'debug' "The installed version '${local_version}' does not much the remote version '${remote_version}'";
+          TFENV_VERSION="${remote_version}";
+        else
+          TFENV_VERSION="${local_version}";
+        fi;
+    else
+      log 'error' "No versions matching '${requested}' found in remote";
+    fi;
   else
-    log 'error' "No installed versions of terraform matched '${TFENV_VERSION}'";
+    if [[ -n "${local_version}" ]]; then
+      TFENV_VERSION="${local_version}";
+    else
+      log 'error' "No installed versions of terraform matched '${TFENV_VERSION}'";
+    fi;
   fi;
-
 else
   log 'debug' 'TFENV_VERSION does not use "latest" keyword';
 
@@ -113,10 +129,11 @@ else
   fi;
 fi;
 
-[ -z "${TFENV_VERSION}" ] \
-  && log 'error' "Version could not be resolved (set by ${TFENV_VERSION_SOURCE} or tfenv use <version>)";
+if [[ -z "${TFENV_VERSION}" ]]; then
+  log 'error' "Version could not be resolved (set by ${TFENV_VERSION_SOURCE} or tfenv use <version>)";
+fi;
 
-if [ ! -d "${TFENV_CONFIG_DIR}/versions/${TFENV_VERSION}" ]; then
+if [[ ! -d "${TFENV_CONFIG_DIR}/versions/${TFENV_VERSION}" ]]; then
   log 'debug' "version '${TFENV_VERSION}' is not installed (set by ${TFENV_VERSION_SOURCE})";
 fi;