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
cmake: Setup default macOS flags #37015
Conversation
Success on aarch64-linux (full log) Attempted: cmake Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: cmake Partial log (click to expand)
|
Does anyone have issues with this? Will merge soon if no issues. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, I can push this to my wip job if this should be tested more extensively before merging to staging.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes a Hydra job would be helpful.
@matthewbauer Any thoughts on a list of packages to test for this, building all of nixpkgs on the official hydra is maybe not the best idea given the limited build capacity. |
Okay I'll just merge to staging and watch what happens. It shouldn't break anything that wasn't already broken. |
I would be surprised if this causes any major issues, if everything looks ok we should probably backport this to 18.03. I've seen a number of build failures that look related to this. |
Motivation for this change
This is a collection of new defaults to setup for CMake related to macOS. Many packages are already setting these, but this will just make them the default. If I have time, I can make a treewide commit that removes references to these flags.
This could potentially break many packages, so it should be extensively tested. If someone could setup a Hydra jobset for this, that would be great!
Also related to #36834