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

Macos workflow #303

Merged
merged 5 commits into from
Jan 30, 2025
Merged

Macos workflow #303

merged 5 commits into from
Jan 30, 2025

Conversation

lucbv
Copy link
Contributor

@lucbv lucbv commented Jan 24, 2025

Adding an OSX CI workflow so that the work to get things working with AppleClang does not regress over time.

lucbv added 3 commits January 23, 2025 14:01
This could be revised later if a more modern version actually
implements the feature correctly by checking the version value.

Signed-off-by: Luc Berger-Vergiat <[email protected]>
Since the test does not compile on MacOS and it is not really
needed we are removing it. See discussion in issue kokkos#300.

Signed-off-by: Luc Berger-Vergiat <[email protected]>
@lucbv lucbv added the enhancement New feature or request label Jan 24, 2025
@lucbv lucbv requested review from dalg24, mhoemmen and crtrott January 24, 2025 20:56
@lucbv lucbv self-assigned this Jan 24, 2025
@lucbv
Copy link
Contributor Author

lucbv commented Jan 24, 2025

After a few trial and errors, this seems to do the job pretty well : )

@masterleinad
Copy link
Contributor

Can't uou just do something like https://stackoverflow.com/a/78173242? or do you actually need change more than just the underlying runner OS?

@lucbv lucbv linked an issue Jan 24, 2025 that may be closed by this pull request
@lucbv
Copy link
Contributor Author

lucbv commented Jan 27, 2025

@masterleinad there are a few additional improvements made as you can see but we can revert to a more complicated and slower build option if you want?

@lucbv lucbv merged commit 8b62a13 into kokkos:main Jan 30, 2025
4 checks passed
@lucbv lucbv deleted the macos_workflow branch January 30, 2025 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Build failure on MacOS 14.7.1
3 participants