Skip to content

Commit 3c6943b

Browse files
committedMar 19, 2024·
fix(src/common-utils): correct ADDITIONAL_PACKAGES declaration typo
1 parent ea302b5 commit 3c6943b

File tree

3 files changed

+7
-4
lines changed

3 files changed

+7
-4
lines changed
 

‎src/common-utils/install.sh

+2-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,8 @@ set -e
1111

1212
# shellcheck disable=SC2034
1313
INSTALL_ZSH="${INSTALLZSH:-"true"}"
14-
ADDITIONAL_PACKAGES="${ADDITIONAL_PACKAGES:-""}"
14+
# shellcheck disable=SC2034
15+
ADDITIONAL_PACKAGES="${ADDITIONALPACKAGES:-""}"
1516
# shellcheck disable=SC2034
1617
CONFIGURE_ZSH_AS_DEFAULT_SHELL="${CONFIGUREZSHASDEFAULTSHELL:-"false"}"
1718
# shellcheck disable=SC2034

‎src/common-utils/main.sh

+4-2
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
set -e
1111

1212
INSTALL_ZSH="${INSTALLZSH:-"true"}"
13-
ADDITIONAL_PACKAGES="${ADDITIONAL_PACKAGES:-""}"
13+
ADDITIONAL_PACKAGES="${ADDITIONALPACKAGES:-""}"
1414
CONFIGURE_ZSH_AS_DEFAULT_SHELL="${CONFIGUREZSHASDEFAULTSHELL:-"false"}"
1515
INSTALL_OH_MY_ZSH="${INSTALLOHMYZSH:-"true"}"
1616
INSTALL_OH_MY_ZSH_CONFIG="${INSTALLOHMYZSHCONFIG:-"true"}"
@@ -87,6 +87,7 @@ install_arch_packages() {
8787

8888
# Additonal packages (space separated string). Eg: "docker-compose kubectl"
8989
if [ -n "${ADDITIONAL_PACKAGES}" ]; then
90+
echo "Additional packages to install: ${ADDITIONAL_PACKAGES}..."
9091
IFS=' ' read -r -a additional_pkgs <<< "${ADDITIONAL_PACKAGES}"
9192
package_list+=("${additional_pkgs[@]}")
9293
fi
@@ -156,6 +157,7 @@ elif [ "${USERNAME}" = "none" ]; then
156157
USER_UID=0
157158
USER_GID=0
158159
fi
160+
159161
# Create or update a non-root user to match UID/GID.
160162
group_name="${USERNAME}"
161163
if id -u "${USERNAME}" > /dev/null 2>&1; then
@@ -338,4 +340,4 @@ echo -e "\
338340
RC_SNIPPET_ALREADY_ADDED=${RC_SNIPPET_ALREADY_ADDED}\n\
339341
ZSH_ALREADY_INSTALLED=${ZSH_ALREADY_INSTALLED}" > "${MARKER_FILE}"
340342

341-
echo "Done!"
343+
echo "Done! Common utilities devcontainer feature has been installed."

‎test/common-utils/install_common_utils_with_additional_pkgs.sh

+1-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ source dev-container-features-test-lib
77

88
# Additional packages
99
check "whois" whois --version
10-
check "dnsutils" nslookup --version
10+
check "dig" dig -v
1111
check "traceroute" traceroute --version
1212

1313
# Report result

0 commit comments

Comments
 (0)
Please sign in to comment.