Skip to content

Fix install script test timeout for macOS compatibility - #5689

Merged
pelikhan merged 6 commits into
mainfrom
copilot/fix-install-script-failure
Dec 6, 2025
Merged

Fix install script test timeout for macOS compatibility#5689
pelikhan merged 6 commits into
mainfrom
copilot/fix-install-script-failure

Conversation

Copilot AI commented Dec 6, 2025

Copy link
Copy Markdown
Contributor

The install script CI workflow was failing on macOS runners because timeout command doesn't exist on macOS by default.

Changes

  • Added cross-platform timeout detection in .github/workflows/install.yml
  • Linux: uses native timeout command
  • macOS: falls back to perl -e 'alarm shift; exec @ARGV' (Perl is available by default)
- 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
+   timeout 10s ./install-gh-aw.sh v999.999.999 2>&1 | tee install-output.log || true
+ else
+   perl -e 'alarm shift; exec @ARGV' 10 ./install-gh-aw.sh v999.999.999 2>&1 | tee install-output.log || true
+ fi

The Perl-based approach avoids race conditions inherent to background process + kill patterns.

Original prompt

Review install script failure and fix the detection.

Reference: https://github.com/githubnext/gh-aw/actions/runs/19993149551/job/57336794162


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI and others added 5 commits December 6, 2025 19:28
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix install script failure detection Fix install script test timeout for macOS compatibility Dec 6, 2025
Copilot AI requested a review from pelikhan December 6, 2025 19:42
@pelikhan
pelikhan marked this pull request as ready for review December 6, 2025 19:51
@pelikhan
pelikhan merged commit b5b3c28 into main Dec 6, 2025
11 checks passed
@pelikhan
pelikhan deleted the copilot/fix-install-script-failure branch December 6, 2025 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants