From 0cafeee86296b9f19e6425055e0c4037eff41985 Mon Sep 17 00:00:00 2001
From: Josh Spicer <joshspicer@github.com>
Date: Tue, 23 Aug 2022 09:17:08 -0400
Subject: favor correctness by removing `apt update ...` short-circuiting (#98)
* favor correctness by removing apt update shortciruiting
* Update install.sh
* apt_get_upadate in node
* update test
---
src/aws-cli/install.sh | 13 ++++---------
1 file changed, 4 insertions(+), 9 deletions(-)
(limited to 'src/aws-cli')
diff --git a/src/aws-cli/install.sh b/src/aws-cli/install.sh
index b2566c9..4ab2234 100755
--- a/src/aws-cli/install.sh
+++ b/src/aws-cli/install.sh
@@ -62,21 +62,16 @@ get_common_setting() {
echo "$1=${!1}"
}
-# Function to run apt-get if needed
-apt_get_update_if_needed()
+apt_get_update()
{
- if [ ! -d "/var/lib/apt/lists" ] || [ "$(ls /var/lib/apt/lists/ | wc -l)" = "0" ]; then
- echo "Running apt-get update..."
- apt-get update
- else
- echo "Skipping apt-get update."
- fi
+ echo "Running apt-get update..."
+ apt-get update -y
}
# Checks if packages are installed and installs them if not
check_packages() {
if ! dpkg -s "$@" > /dev/null 2>&1; then
- apt_get_update_if_needed
+ apt_get_update
apt-get -y install --no-install-recommends "$@"
fi
}
--
cgit v1.2.3