From b787c533bfdfd5fc1dad224a72ae63cc165dcde1 Mon Sep 17 00:00:00 2001 From: Florian Date: Fri, 24 Apr 2026 22:12:20 +0200 Subject: [PATCH] fix: make setup-dependencies portable on macOS (#1463) Two issues prevented the script from running on macOS without manual intervention: 1. `sed -i` requires a backup suffix on macOS BSD sed. Replace with a portable `grep -v | mv` idiom that works on both macOS and Linux. 2. `python` is not in PATH on macOS by default. Replace with `python3`. Co-authored-by: Florian Horner Co-authored-by: Bas Nijholt --- scripts/setup-dependencies | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/setup-dependencies b/scripts/setup-dependencies index d69bce3c..f3948dd5 100755 --- a/scripts/setup-dependencies +++ b/scripts/setup-dependencies @@ -5,7 +5,7 @@ cd "$(dirname "$0")/.." # Remove mypy-dev from requirements_test.txt since the maintainer deletes old versions from PyPI. # We'll install the latest version separately below. # See: https://github.com/cdce8p/mypy-dev/issues/62 -sed -i '/^mypy-dev/d' core/requirements_test.txt +grep -v '^mypy-dev' core/requirements_test.txt > core/requirements_test.txt.tmp && mv core/requirements_test.txt.tmp core/requirements_test.txt uv pip install -r core/requirements.txt uv pip install -r core/requirements_test.txt @@ -35,7 +35,7 @@ done uv pip install -e core/ uv pip install ulid-transform # this is in Adaptive-lighting's manifest.json -uv pip install $(python test_dependencies.py) +uv pip install $(python3 test_dependencies.py) # Install the latest mypy-dev (not pinned since old versions get deleted from PyPI) uv pip install --upgrade mypy-dev