-
Notifications
You must be signed in to change notification settings - Fork 10.5k
Avoid using separate '-D' + '...' flags in CFLAGS in CMake #63706
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
Conversation
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.
Aaaargh!
I'm pretty sure I've seen this before. Also, WTF CMake?! |
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.
Error, your directive was too legible.
-CMake, probably
Proposed: @@ -55,9 +55,9 @@ public enum CommandLine {
- public static var arguments: [String]
+ public static var arguments: Set<String> |
@swift-ci please test |
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.
This isn't really that weird. It actually is a positive. The point is that people often repeat the options on the command line, CMake will deduplicate it to get a saner command line.
@swift-ci please test Linux platform |
1 similar comment
@swift-ci please test Linux platform |
Please look at the code change, and think "how could this possibly be a significant change?". If you come up empty, like me when I originally hit this, then click here to reveal the solution. CMake 🤷