Skip to content

Fix install script test timeout for macOS compatibility #5689

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
pelikhan merged 6 commits into from
Dec 6, 2025
Merged

Fix install script test timeout for macOS compatibility #5689

Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
6ee50f6
Initial plan
Copilot Dec 6, 2025
384c99d
Initial analysis - identify timeout command issue on macOS
Copilot Dec 6, 2025
eb2bb3b
Fix install script test timeout command for macOS compatibility
Copilot Dec 6, 2025
c0cbeb1
Use Perl-based timeout for better cross-platform compatibility
Copilot Dec 6, 2025
8457b87
Revert unintended changes to release.lock.yml
Copilot Dec 6, 2025
9b94142
Use POSIX-compliant redirection for better shell compatibility
Copilot Dec 6, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 10 additions & 1 deletion .github/workflows/install.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,16 @@ jobs:
# Run the script but capture the output before it tries to download
# We'll use a fake version to test URL construction
set +e
timeout 10s ./install-gh-aw.sh v999.999.999 2>&1 | tee install-output.log || true

# Cross-platform timeout implementation
if command -v timeout > /dev/null 2>&1; then
# Linux has timeout command
timeout 10s ./install-gh-aw.sh v999.999.999 2>&1 | tee install-output.log || true
else
# macOS and others - use Perl-based timeout (available by default on macOS)
perl -e 'alarm shift; exec @ARGV' 10 ./install-gh-aw.sh v999.999.999 2>&1 | tee install-output.log || true
fi

set -e

# Verify platform was detected
Expand Down
Loading