Skip to content

Fix brew tap trust and Linux pip build failures on CI - #274

Open
PartTimeLegend with Copilot wants to merge 2 commits into
masterfrom
copilot/fix-copilot-issue
Open

Fix brew tap trust and Linux pip build failures on CI#274
PartTimeLegend with Copilot wants to merge 2 commits into
masterfrom
copilot/fix-copilot-issue

Conversation

Copilot AI commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Two independent CI failures on Ubuntu runners: Homebrew's new tap-trust enforcement silently blocks untrusted third-party taps during brew bundle, and sbapp[macos] pulls in kivy as a transitive dependency which fails to compile on Linux due to a missing GL/gl.h.

Changes

  • NewMachineSetup.sh — brew tap trust pre-flight
    install_brewfile() now parses the Brewfile for all tap directives and calls brew tap + brew trust on each before invoking brew bundle, satisfying Homebrew's tap-trust requirement without disabling the mechanism globally.

    while IFS= read -r line; do
      tap_name=$(echo "$line" | sed -n 's/^tap[[:space:]]*"\([^"]*\)".*/\1/p')
      if [ -n "$tap_name" ]; then
        brew tap "$tap_name" 2>/dev/null || true
        brew trust "$tap_name" 2>/dev/null || true
      fi
    done < Brewfile
  • requirements.txt — restrict sbapp to macOS
    Added PEP 508 platform marker so sbapp[macos] (and its kivy dependency) is skipped entirely on Linux:

    sbapp[macos]; sys_platform == 'darwin'
    

Co-authored-by: PartTimeLegend <791578+PartTimeLegend@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix issue in Copilot integration Fix brew tap trust and Linux pip build failures on CI Aug 13, 2026
Copilot AI requested a review from PartTimeLegend August 13, 2026 23:12
@PartTimeLegend
PartTimeLegend marked this pull request as ready for review August 13, 2026 23:18
@PartTimeLegend

Copy link
Copy Markdown
Owner

@copilot resolve the merge conflicts on this branch.

@PartTimeLegend

Copy link
Copy Markdown
Owner

@copilot resolve the merge conflicts on this branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants