Skip to content

Commit

Permalink
Do not check the release list if a Terraform is already installed (fixes
Browse files Browse the repository at this point in the history
 #395)

Today, tfenv checks the Terraform release list when running
`tfenv install`, even if the requested Terraform is already
installed. This fixes an issue where a script containing
`tfenv install` becomes inoperable even if Terraform is installed,
but releases.hashicorp.com is down.

See #395 for more information.
  • Loading branch information
dluchini-ai committed May 30, 2023
1 parent 1ccfddb commit 0e7c19e
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions libexec/tfenv-install
Original file line number Diff line number Diff line change
Expand Up @@ -73,15 +73,15 @@ declare regex="${resolved##*\:}";

log 'debug' "Processing install for version ${version}, using regex ${regex}";

remote_version="$(tfenv-list-remote | grep -e "${regex}" | head -n 1)";
[ -n "${remote_version}" ] && version="${remote_version}" || log 'error' "No versions matching '${requested:-$version}' found in remote";

dst_path="${TFENV_CONFIG_DIR}/versions/${version}";
if [ -f "${dst_path}/terraform" ]; then
echo "Terraform v${version} is already installed";
exit 0;
fi;

remote_version="$(tfenv-list-remote | grep -e "${regex}" | head -n 1)";
[ -n "${remote_version}" ] && version="${remote_version}" || log 'error' "No versions matching '${requested:-$version}' found in remote";

case "$(uname -s)" in
Darwin*)
kernel="darwin";
Expand Down

0 comments on commit 0e7c19e

Please sign in to comment.