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

Switch finite to std::isfinite for ICC and GCC #69

Merged
merged 1 commit into from
Sep 16, 2024

Conversation

Chekov2k
Copy link

This supports latest compilers on Mac OS X, see Homebrew/homebrew-core#181246

@Chekov2k
Copy link
Author

Would be nice to get some traction on this one

@ebertin
Copy link
Member

ebertin commented Sep 16, 2024

Thanks for the PR and sorry for the delay. I will take care of this tonight.

@Chekov2k
Copy link
Author

Thank you! The home-brew CI shows it works fine on Mac OS X for the last few major releases so should not break anything for anyone

@ebertin ebertin merged commit f273ca7 into astromatic:master Sep 16, 2024
1 check passed
@ebertin
Copy link
Member

ebertin commented Sep 16, 2024

Thank you @Chekov2k for the PR. It might worth a release.
Would you be interested in adding a basic CI workflow for MacOS on GitHub?

@Chekov2k
Copy link
Author

I haven't configured any GitHub CI yet but can give it a go ;) Should be fairly similar to GitLab...

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