Skip to content
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.

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

Already on GitHub? Sign in to your account

Fix build on macOS CI runners #1158

Merged
merged 1 commit into from
Nov 22, 2024
Merged

Fix build on macOS CI runners #1158

merged 1 commit into from
Nov 22, 2024

Conversation

passaro
Copy link
Contributor

@passaro passaro commented Nov 21, 2024

In order to work around a homebrew issue with pkg-config on github runners (see actions/runner-images#10984), temporarily run a command to uninstall [email protected].

Does this change impact existing behavior?

No. Workflow change only.

Does this change need a changelog entry?

No.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and I agree to the terms of the Developer Certificate of Origin (DCO).

Signed-off-by: Alessandro Passaro <[email protected]>
@passaro passaro temporarily deployed to PR integration tests November 21, 2024 22:05 — with GitHub Actions Inactive
@passaro passaro temporarily deployed to PR integration tests November 21, 2024 22:05 — with GitHub Actions Inactive
@passaro passaro temporarily deployed to PR integration tests November 21, 2024 22:05 — with GitHub Actions Inactive
@passaro passaro temporarily deployed to PR integration tests November 21, 2024 22:05 — with GitHub Actions Inactive
@passaro passaro temporarily deployed to PR integration tests November 21, 2024 22:05 — with GitHub Actions Inactive
@passaro passaro temporarily deployed to PR integration tests November 21, 2024 22:05 — with GitHub Actions Inactive
@passaro passaro temporarily deployed to PR integration tests November 21, 2024 22:05 — with GitHub Actions Inactive
@passaro passaro enabled auto-merge November 21, 2024 22:08
@passaro passaro added this pull request to the merge queue Nov 22, 2024
Merged via the queue into awslabs:main with commit ea5c637 Nov 22, 2024
23 checks passed
@passaro passaro deleted the macos-ci-fix branch November 22, 2024 09:36
github-merge-queue bot pushed a commit that referenced this pull request Dec 17, 2024
In November, macOS runners had an issue with the `pkg-config` package
and we introduced a workaround:
#1158

We see now that the fix has been released and we should no longer need
the workaround: actions/runner-images#10984.

### Does this change impact existing behavior?

No, CI change only.

### Does this change need a changelog entry?

No.

---

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license and I agree to the terms of
the [Developer Certificate of Origin
(DCO)](https://developercertificate.org/).

Signed-off-by: Daniel Carl Jones <[email protected]>
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