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

[ci] macos12 is no longer available, use macos14. #1034

Merged
merged 1 commit into from
Nov 6, 2024
Merged

Conversation

jpobst
Copy link
Contributor

@jpobst jpobst commented Nov 4, 2024

The Mac pool we use internal-macos12 is no longer available. Ideally we could just use the public Azure Pipelines pool, but those machines are underpowered and doubles the (already long!) time taken to build our packages. Switch to the internal VSEng-VSMac-Xamarin-Shared mac pool already in use by dotnet/android and MAUI.

JavaToolInstaller@0 does not seem to work on this image, even though there are many versions of Java SDK already installed on the image. Set JAVA_HOME to Java SDK 11 so we can build.

Additionally, the tools-executive-oreder-csv-and-markdown Cake task does not work on these images because it expects gradle and other tools to be available in the $PATH which they aren't. As we do not consume or ship these files from the build, add a try/catch block to the task so that it does not fail the build if it fails.

@jpobst jpobst force-pushed the update-macos branch 15 times, most recently from 80ab7b0 to 76e75c7 Compare November 5, 2024 23:35
@jpobst jpobst marked this pull request as ready for review November 6, 2024 18:31
@jpobst jpobst requested a review from moljac November 6, 2024 18:31
@moljac
Copy link
Contributor

moljac commented Nov 6, 2024

Additionally, the tools-executive-oreder-csv-and-markdown Cake task does not work on these images because it expects gradle and other tools to be available in the $PATH which they aren't. As we do not consume or ship these files from the build, add a try/catch block to the task so that it does not fail the build if it fails.

OK. I will fix those scripts in the future.

@jpobst jpobst merged commit 759fe88 into main Nov 6, 2024
6 checks passed
@jpobst jpobst deleted the update-macos branch November 6, 2024 18:53
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