-
Notifications
You must be signed in to change notification settings - Fork 10.5k
Add SWIFT_PLATFORMS_USING_ISYSROOT to list platforms that should be building with -isysroot #33469
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
Add SWIFT_PLATFORMS_USING_ISYSROOT to list platforms that should be building with -isysroot #33469
Conversation
…uilding with -isysroot
Hmm, I'm not sure I understand this. If I'm working with freestanding, I could be on a Linux or windows environment, which don't use |
You're right. (this is not a well-thought-out PR, I just wanted to see your take on this.) We should make the freestanding config be usable from any environment. My immediate problem is when building freestanding on a macOS host, I need to use -isysroot (that's what all Apple SDKs expect). Let me try adding this as an option into the SDK configuration itself. |
@compnerd Updated. Notice that currently the FREESTANDING SDK is set up as |
@swift-ci please test |
Build failed |
Build failed |
@@ -145,6 +145,8 @@ endfunction() | |||
# SWIFT_SDK_${prefix}_LIB_SUBDIR Library subdir for this SDK | |||
# SWIFT_SDK_${prefix}_VERSION_MIN_NAME Version min name for this SDK | |||
# SWIFT_SDK_${prefix}_TRIPLE_NAME Triple name for this SDK | |||
# SWIFT_SDK_${prefix}_OBJECT_FORMAT The object file format (e.g. MACHO) |
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.
Hah! I think you put this in the wrong commit 🤷♂️
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.
No, this was an intentional drive-by addition :) It seemed to be missing...
No description provided.