Fix all ((configured_count++)) instances causing script exit during configuration
- Replace ((configured_count++)) with configured_count=$((configured_count + 1)) - This fixes script exiting prematurely after configuring first application - Issue: ((var++)) returns old value (0) which causes exit with set -euo pipefail - Fixed 14 instances across the configure_applications_for_tmpfs function
This commit is contained in:
@@ -952,7 +952,7 @@ configure_applications_for_tmpfs() {
|
||||
chown -R $current_user:$current_user /tmp/tmpfs-cache/browser/firefox
|
||||
|
||||
success " ✅ Firefox configured to use tmpfs cache"
|
||||
((configured_count++))
|
||||
configured_count=$((configured_count + 1))
|
||||
fi
|
||||
fi
|
||||
done
|
||||
@@ -979,7 +979,7 @@ BRAVEEOF
|
||||
chown $current_user:$current_user "$brave_desktop" 2>/dev/null || true
|
||||
chmod +x "$brave_desktop" 2>/dev/null || true
|
||||
success " ✅ Brave Browser configured to use tmpfs cache"
|
||||
((configured_count++))
|
||||
configured_count=$((configured_count + 1))
|
||||
fi
|
||||
|
||||
# Google Chrome
|
||||
@@ -1004,7 +1004,7 @@ CHROMEOF
|
||||
chown $current_user:$current_user "$chrome_desktop" 2>/dev/null || true
|
||||
chmod +x "$chrome_desktop" 2>/dev/null || true
|
||||
success " ✅ Google Chrome configured to use tmpfs cache"
|
||||
((configured_count++))
|
||||
configured_count=$((configured_count + 1))
|
||||
fi
|
||||
|
||||
# Chromium
|
||||
@@ -1029,7 +1029,7 @@ CHROMIUMEOF
|
||||
chown $current_user:$current_user "$chromium_desktop" 2>/dev/null || true
|
||||
chmod +x "$chromium_desktop" 2>/dev/null || true
|
||||
success " ✅ Chromium configured to use tmpfs cache"
|
||||
((configured_count++))
|
||||
configured_count=$((configured_count + 1))
|
||||
fi
|
||||
|
||||
# Development Tools Configuration
|
||||
@@ -1039,7 +1039,7 @@ CHROMIUMEOF
|
||||
# NPM
|
||||
if command -v npm &>/dev/null; then
|
||||
sudo -u $current_user npm config set cache /tmp/tmpfs-cache/development/npm 2>/dev/null && \
|
||||
success " ✅ NPM cache configured" && ((configured_count++))
|
||||
success " ✅ NPM cache configured" && configured_count=$((configured_count + 1))
|
||||
fi
|
||||
|
||||
# Pip
|
||||
@@ -1050,7 +1050,7 @@ CHROMIUMEOF
|
||||
cache-dir = /tmp/tmpfs-cache/development/pip
|
||||
PIPEOF
|
||||
chown $current_user:$current_user "$user_home/.config/pip/pip.conf" 2>/dev/null || true
|
||||
success " ✅ Pip cache configured" && ((configured_count++))
|
||||
success " ✅ Pip cache configured" && configured_count=$((configured_count + 1))
|
||||
fi
|
||||
|
||||
# KDE/Plasma Configuration
|
||||
@@ -1066,7 +1066,7 @@ PIPEOF
|
||||
ln -sf /tmp/tmpfs-cache/kde/thumbnails "$user_home/.cache/thumbnails"
|
||||
chown -R $current_user:$current_user /tmp/tmpfs-cache/kde/thumbnails
|
||||
success " ✅ KDE thumbnail cache linked to tmpfs"
|
||||
((configured_count++))
|
||||
configured_count=$((configured_count + 1))
|
||||
fi
|
||||
|
||||
echo ""
|
||||
|
||||
Reference in New Issue
Block a user