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

Mac build #18

Merged
merged 7 commits into from
Jul 10, 2024
Merged

Mac build #18

merged 7 commits into from
Jul 10, 2024

Conversation

K20shores
Copy link
Contributor

@K20shores K20shores commented Jun 26, 2024

Name and Institution (Required)

Name: Kyle Shores
Institution: National Center for Atmospheric Research

Describe the update

Adds a github action to build CloudJ on mac with gcc 11, 12, and 13. Some of the cmake files also had to be modified to make it work

Expected changes

There shouldn't be any since there were no code changes. However, when I run the standalone binary, I get a segfault on mac

Reference(s)

None

Related Github Issues and PRs

None

@lizziel lizziel changed the base branch from main to dev/7.7.2 June 27, 2024 14:15
@lizziel lizziel self-assigned this Jun 27, 2024
@lizziel lizziel added the bug Something isn't working label Jun 27, 2024
@lizziel lizziel added this to the 7.7.2 milestone Jun 27, 2024
@lizziel
Copy link
Contributor

lizziel commented Jun 27, 2024

@K20shores, making sure I am understanding correctly that your updates to the standalone CMakeLists.txt fix your seg fault issue. True?

@K20shores
Copy link
Contributor Author

@lizziel sorry, I was imprecise with my language. Even with these changes I was seeing the segfault. But that went away when I removed the stack frame limit with ulimit -s unlimited as suggested in the readme.

@lizziel
Copy link
Contributor

lizziel commented Jul 10, 2024

@K20shores, could you update the changelog?

@K20shores
Copy link
Contributor Author

@lizziel done! Sorry for missing that the first time around

@lizziel
Copy link
Contributor

lizziel commented Jul 10, 2024

Thanks!

@lizziel lizziel merged commit 03313bb into geoschem:dev/7.7.2 Jul 10, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants