Skip to content

Commit ba57e9d

Browse files
committed
fix(installer): resolve shellcheck warnings in install.sh
1 parent c968e6f commit ba57e9d

1 file changed

Lines changed: 8 additions & 5 deletions

File tree

install.sh

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,8 @@ verify_checksum() {
103103
fi
104104

105105
# Extract expected hash from checksum file (format: "hash filename")
106-
local expected_hash=$(awk '{print $1}' "$checksum_file")
106+
local expected_hash
107+
expected_hash=$(awk '{print $1}' "$checksum_file")
107108

108109
# Calculate actual hash
109110
local actual_hash
@@ -172,7 +173,7 @@ main() {
172173

173174
# Create temporary directory
174175
TMP_DIR=$(mktemp -d)
175-
trap "rm -rf $TMP_DIR" EXIT
176+
trap 'rm -rf $TMP_DIR' EXIT
176177

177178
# Download archive
178179
info "Downloading dtvem..."
@@ -265,9 +266,11 @@ main() {
265266
if [ -n "$SHELL_CONFIG" ]; then
266267
# Check if already in config
267268
if ! grep -q "$INSTALL_DIR" "$SHELL_CONFIG" 2>/dev/null; then
268-
echo "" >> "$SHELL_CONFIG"
269-
echo "# Added by dtvem installer" >> "$SHELL_CONFIG"
270-
echo "$EXPORT_CMD" >> "$SHELL_CONFIG"
269+
{
270+
echo ""
271+
echo "# Added by dtvem installer"
272+
echo "$EXPORT_CMD"
273+
} >> "$SHELL_CONFIG"
271274
success "Added to $SHELL_CONFIG"
272275
else
273276
info "Already in $SHELL_CONFIG"

0 commit comments

Comments
 (0)