diff --git a/first_time_install.sh b/first_time_install.sh index 44eed924..4b87b5c0 100644 --- a/first_time_install.sh +++ b/first_time_install.sh @@ -227,8 +227,8 @@ wait_for_apt_lock() { done } -apt_update() { wait_for_apt_lock; retry apt update; } -apt_install() { wait_for_apt_lock; retry apt install -y "$@"; } +apt_update() { wait_for_apt_lock; retry apt-get -o DPkg::Lock::Timeout=180 update; } +apt_install() { wait_for_apt_lock; retry apt-get -o DPkg::Lock::Timeout=180 install -y "$@"; } apt_remove() { apt-get remove -y "$@" || true; } check_network() { @@ -857,24 +857,30 @@ if [ "$_SKIP_BUILD" = "1" ]; then echo "rgbmatrix already installed${_skip_suffix}; skipping build (set RPI_RGB_FORCE_REBUILD=1 to force rebuild)." else # Ensure rpi-rgb-led-matrix submodule is initialized + # Wrapper used with retry(): removes any partial clone dir before each attempt + # so git clone doesn't fail with "destination path already exists". + _clone_rpi_rgb() { + rm -rf "$PROJECT_ROOT_DIR/rpi-rgb-led-matrix-master" + git clone https://github.com/hzeller/rpi-rgb-led-matrix.git rpi-rgb-led-matrix-master + } if [ ! -d "$PROJECT_ROOT_DIR/rpi-rgb-led-matrix-master" ]; then echo "rpi-rgb-led-matrix-master not found. Initializing git submodule..." cd "$PROJECT_ROOT_DIR" - + # Try to initialize submodule if .gitmodules exists if [ -f "$PROJECT_ROOT_DIR/.gitmodules" ] && grep -q "rpi-rgb-led-matrix" "$PROJECT_ROOT_DIR/.gitmodules"; then echo "Initializing rpi-rgb-led-matrix submodule..." if ! retry git submodule update --init --recursive rpi-rgb-led-matrix-master; then echo "⚠ Submodule init failed, cloning directly from GitHub..." - retry git clone https://github.com/hzeller/rpi-rgb-led-matrix.git rpi-rgb-led-matrix-master + retry _clone_rpi_rgb fi else # Fallback: clone directly if submodule not configured echo "Submodule not configured, cloning directly from GitHub..." - retry git clone https://github.com/hzeller/rpi-rgb-led-matrix.git rpi-rgb-led-matrix-master + retry _clone_rpi_rgb fi fi - + # Build and install rpi-rgb-led-matrix Python bindings if [ -d "$PROJECT_ROOT_DIR/rpi-rgb-led-matrix-master" ]; then # Check if submodule is properly initialized (not empty) @@ -885,7 +891,7 @@ else if [ -f "$PROJECT_ROOT_DIR/.gitmodules" ] && grep -q "rpi-rgb-led-matrix" "$PROJECT_ROOT_DIR/.gitmodules"; then retry git submodule update --init --recursive rpi-rgb-led-matrix-master else - retry git clone https://github.com/hzeller/rpi-rgb-led-matrix.git rpi-rgb-led-matrix-master + retry _clone_rpi_rgb fi fi diff --git a/scripts/install_dependencies_apt.py b/scripts/install_dependencies_apt.py index 4d9a977b..a03d0243 100644 --- a/scripts/install_dependencies_apt.py +++ b/scripts/install_dependencies_apt.py @@ -10,6 +10,7 @@ import tempfile import warnings from collections import deque from pathlib import Path +from typing import List, Tuple # How many trailing lines of a failed command's output to keep for the # end-of-run failure summary. Keeps the root cause near the end of the log, @@ -17,7 +18,7 @@ from pathlib import Path ERROR_TAIL_LINES = 15 -def _run(cmd): +def _run(cmd: List[str]) -> Tuple[bool, str]: """Run a command, streaming combined stdout/stderr to a temp file. Returns (success, output) instead of raising, so callers can report @@ -37,7 +38,7 @@ def _run(cmd): return result.returncode == 0, '\n'.join(tail) -def install_via_apt(package_name): +def install_via_apt(package_name: str) -> Tuple[bool, str]: """Try to install a package via apt. Returns (success, output).""" # Map pip package names to apt package names apt_package_map = { @@ -59,7 +60,7 @@ def install_via_apt(package_name): apt_package = apt_package_map.get(package_name, f'python3-{package_name}') print(f"Trying to install {apt_package} via apt...") - success, output = _run(['sudo', 'apt', 'install', '-y', apt_package]) + success, output = _run(['sudo', 'apt-get', '-o', 'DPkg::Lock::Timeout=180', 'install', '-y', apt_package]) if success: print(f"Successfully installed {apt_package} via apt") return True, "" @@ -68,7 +69,7 @@ def install_via_apt(package_name): return False, output -def install_via_pip(package_name): +def install_via_pip(package_name: str) -> Tuple[bool, str]: """Install a package via pip with --break-system-packages and --prefer-binary. --break-system-packages allows pip to install into the system Python on @@ -105,7 +106,7 @@ IMPORT_NAME_MAP = { } -def check_package_installed(package_name): +def check_package_installed(package_name: str) -> bool: """Check if a package is already installed.""" import_name = IMPORT_NAME_MAP.get(package_name, package_name) # Suppress deprecation warnings when checking if packages are installed @@ -119,7 +120,7 @@ def check_package_installed(package_name): return False -def print_failure_summary(failed_packages, failure_details): +def print_failure_summary(failed_packages: List[str], failure_details: dict) -> None: print("\n" + "=" * 60) print("DEPENDENCY INSTALLATION FAILURES - DETAILS") print("=" * 60) diff --git a/web_interface/templates/v3/partials/tools.html b/web_interface/templates/v3/partials/tools.html index bf4cf0a5..0fcbeeee 100644 --- a/web_interface/templates/v3/partials/tools.html +++ b/web_interface/templates/v3/partials/tools.html @@ -274,6 +274,7 @@ return; } + const dirtyBadge = d.dirty ? 'dirty' : 'clean';