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 OSX specific CI issues #548

Merged
merged 3 commits into from
Nov 30, 2024
Merged

Conversation

LaurentRDC
Copy link
Contributor

This PR fixes OSX-specific CI issues which don't happen on other platforms.

This PR will remain in draft state until I can figure out what's wrong -- I don't have Apple hardware to test locally

@tomsmeding
Copy link
Member

Thanks for the work! We'll have a look, some of us have Apple hardware to test on.

@LaurentRDC
Copy link
Contributor Author

@tomsmeding If you approve this workflow, I can work at it over the weekend

@tomsmeding
Copy link
Member

I'm seeing some capstone-related errors in the macos CI; I'm currently upgrading tracy, and the new tracy version downloads and builds its own capstone. Let's see if that works better.

It turns out that .init_array is a Linuxism, so we now use
__attribute__((constructor)) on Mac, section(".init_array") on Linux,
and still nothing on Windows because what even can we do.
@tomsmeding
Copy link
Member

It turns out that the +ACC flag parsing doesn't work on mac with your fix; see the commit description. I tested this on a mac and this does seem to work.

@tomsmeding
Copy link
Member

Ah meh this CI run would be pointless anyway because it's going to succeed for macos-release and fail for macos-debug for Tracy reasons.

@tomsmeding tomsmeding marked this pull request as ready for review November 30, 2024 08:46
@tomsmeding tomsmeding merged commit 73af56f into AccelerateHS:master Nov 30, 2024
1 of 33 checks passed
@LaurentRDC
Copy link
Contributor Author

Thanks for taking care of this!

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