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 the macOS CI build #2220

Merged
merged 10 commits into from
Mar 30, 2024
Merged

Fix the macOS CI build #2220

merged 10 commits into from
Mar 30, 2024

Conversation

lmoureaux
Copy link
Contributor

The macOS CI stopped working, seemingly because multiple Python versions are installed simultaneously. We forced the Python version in response to #2069. There is now a proper fix for #2069 in upstream vcpkg. Update the vcpkg version so we can use it.

The macOS CI stopped working, seemingly because multiple Python versions
are installed simultaneously. We forced the Python version in response
to longturn#2069. There is now a proper fix for longturn#2069 in upstream vcpkg. Update
the vcpkg version so we can use it.
@lmoureaux lmoureaux requested a review from blabber March 17, 2024 03:30
@blabber
Copy link
Collaborator

blabber commented Mar 17, 2024

Is this still ready for review? This did not fix the build, so I guess work on this PR is ongoing.

@lmoureaux lmoureaux marked this pull request as draft March 17, 2024 23:15
@lmoureaux
Copy link
Contributor Author

Is this still ready for review? This did not fix the build, so I guess work on this PR is ongoing.

Right, thanks. Switched to draft

@jwrober jwrober added the back-port back-port candidate label Mar 30, 2024
@lmoureaux lmoureaux marked this pull request as ready for review March 30, 2024 20:48
@lmoureaux
Copy link
Contributor Author

Let's squash and merge if the pipeline succeeds

@jwrober
Copy link
Collaborator

jwrober commented Mar 30, 2024

100%, so only one commit goes into master and stable branches.

@jwrober jwrober requested review from jwrober and removed request for blabber March 30, 2024 21:02
@jwrober jwrober merged commit 1e6a9a2 into longturn:master Mar 30, 2024
20 checks passed
lmoureaux added a commit that referenced this pull request Mar 31, 2024
* Fix the macOS CI build

The macOS CI stopped working, seemingly because multiple Python versions
are installed simultaneously. We forced the Python version in response
to #2069. There is now a proper fix for #2069 in upstream vcpkg. Update
the vcpkg version so we can use it.
@lmoureaux lmoureaux deleted the bugfix/macos-ci branch October 26, 2024 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
back-port back-port candidate
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants