Skip to content

Commit 26b23bf

Browse files
committed
fix(install.sh): replace Unicode ellipsis with ASCII dots
U+2026 … in $variable… causes bash to treat the UTF-8 high-bit bytes as part of the variable name. With set -u this triggers 'unbound variable' when the script is piped through curl (locale defaults to C).
1 parent 1ac7485 commit 26b23bf

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

install.sh

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@ resolve_install_dir() {
107107
install_native() {
108108
local version=$1 asset=$2 dest=$3 os=$4
109109
local url="https://github.com/$REPO/releases/download/$version/$asset"
110-
info "Downloading $asset $version"
110+
info "Downloading $asset $version..."
111111
curl -fsSL "$url" -o "$dest"
112112
chmod +x "$dest"
113113
if [[ "$os" == "macos" ]]; then
@@ -125,7 +125,7 @@ install_jar() {
125125
local wrapper="$bin_dir/$BINARY"
126126

127127
mkdir -p "$lib_dir"
128-
info "Downloading fat JAR $version"
128+
info "Downloading fat JAR $version..."
129129
curl -fsSL "$url" -o "$jar_dest"
130130
success "JAR saved → $jar_dest"
131131

@@ -143,13 +143,13 @@ install_java_sdkman() {
143143
local sdkman_init="$HOME/.sdkman/bin/sdkman-init.sh"
144144

145145
if [[ ! -f "$sdkman_init" ]]; then
146-
info "Installing SDKMAN"
146+
info "Installing SDKMAN..."
147147
curl -fsSL "https://get.sdkman.io" | bash
148148
fi
149149

150150
# shellcheck source=/dev/null
151151
source "$sdkman_init"
152-
info "Installing Java 21 (Temurin) via SDKMAN"
152+
info "Installing Java 21 (Temurin) via SDKMAN..."
153153
sdk install java 21-tem </dev/tty
154154
success "Java 21 installed via SDKMAN."
155155
# Re-source so find_java picks up the new PATH

0 commit comments

Comments
 (0)