Skip to content

Fix macOS consumer CI install permissions - #58

Closed
arokem with Copilot wants to merge 3 commits into
mainfrom
copilot/fix-consumer-macos-latest-job-again
Closed

Fix macOS consumer CI install permissions#58
arokem with Copilot wants to merge 3 commits into
mainfrom
copilot/fix-consumer-macos-latest-job-again

Conversation

Copilot AI commented Sep 3, 2026

Copy link
Copy Markdown

The consumer macos-latest job failed because CMake installed trx-cpp to unwritable /usr/local. The consumer build now uses a workspace-local install prefix.

  • Local installation

    • Configure trx-cpp with ${GITHUB_WORKSPACE}/install as CMAKE_INSTALL_PREFIX.
  • Consumer discovery

    • Point CMAKE_PREFIX_PATH at the same local prefix.
-DCMAKE_INSTALL_PREFIX="${GITHUB_WORKSPACE}/install"
-DCMAKE_PREFIX_PATH="${GITHUB_WORKSPACE}/install"

Co-authored-by: arokem <118582+arokem@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job consumer macos-latest Fix macOS consumer CI install permissions Sep 3, 2026
Copilot AI requested a review from arokem September 3, 2026 15:42
@codecov

codecov Bot commented Sep 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.66%. Comparing base (5cf07d1) to head (909dcf5).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #58   +/-   ##
=======================================
  Coverage   88.66%   88.66%           
=======================================
  Files          15       15           
  Lines        7765     7765           
  Branches     1044     1044           
=======================================
  Hits         6885     6885           
  Misses        880      880           
Flag Coverage Δ
linux 87.86% <ø> (ø)
macos 88.50% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Co-authored-by: arokem <118582+arokem@users.noreply.github.com>
@arokem

arokem commented Sep 3, 2026

Copy link
Copy Markdown
Member

@frheault : why did you close this? It does fix the mac issue, and maybe with some more work could also fix the Windows issue?

@arokem

arokem commented Sep 3, 2026

Copy link
Copy Markdown
Member

OIC, looks like you are mitigating this in #49

@frheault

frheault commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

@arokem I thought the copilot suggestion would be shown to me and then I could test it on my PR rather than a separate PR and I didn't like the idea of doing iterative PR, then merging with my PR and then realize it was not working so I closed it and transfert the solution.

(So this was more ''in case'' to avoid multiple PR if it did not work on first try)

@arokem

arokem commented Sep 8, 2026

Copy link
Copy Markdown
Member

Yes - no worries - this all makes sense and I think we are all good now!

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.

3 participants