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: Add GitHub Action for macOS building #501

Merged

Conversation

utzcoz
Copy link
Contributor

@utzcoz utzcoz commented Sep 15, 2024

  1. macos-14 for arm64 macOS.

Looks like humbletim/setup-vulkan-sdk's cache mechanism doesn't work for multiple jobs with the same OS but different architectures, like macOS 13 is x86_64 and macOS 14 is arm64. So this CL only adds macos-14 as runner OS and we can enable the macos-13 when we fixed the upstream cache supporting.

@utzcoz utzcoz force-pushed the add-github-action-for-macOS-building branch 3 times, most recently from a6c95e4 to 59994b2 Compare September 15, 2024 09:36
Copy link
Contributor

@rpavlik rpavlik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, this is awesome, thank you! I just have one little issue where I think it's not actually building, but otherwise this is great.

.github/workflows/macOS.yml Outdated Show resolved Hide resolved
@utzcoz utzcoz force-pushed the add-github-action-for-macOS-building branch 6 times, most recently from 5a4d5b6 to ba12c55 Compare September 17, 2024 08:20
@CLAassistant
Copy link

CLAassistant commented Sep 17, 2024

CLA assistant check
All committers have signed the CLA.

@utzcoz utzcoz force-pushed the add-github-action-for-macOS-building branch from ba12c55 to a3f71fc Compare September 17, 2024 08:23
@utzcoz
Copy link
Contributor Author

utzcoz commented Sep 21, 2024

@rpavlik PTAL.

@rpavlik rpavlik force-pushed the add-github-action-for-macOS-building branch from a3f71fc to 16ec447 Compare September 23, 2024 15:20
Copy link
Contributor

@rpavlik rpavlik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks! I made a few minor tweaks.

@rpavlik rpavlik force-pushed the add-github-action-for-macOS-building branch from 16ec447 to c71e1aa Compare September 23, 2024 15:21
humbletim/setup-vulkan-sdk's cache mechanism doesn't
work for the same OS but different architectures. So this
CL only tests with macOS 14 with arm64 architecture.

Signed-off-by: utzcoz <[email protected]>
Reviewed-by: Rylie Pavlik <[email protected]>
@rpavlik rpavlik merged commit d795b97 into KhronosGroup:main Sep 23, 2024
32 checks passed
@utzcoz utzcoz deleted the add-github-action-for-macOS-building branch September 23, 2024 15:50
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