Skip to content

[gardening] align platform checks when enabling bitcode inclusion #40768

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

Merged
merged 1 commit into from
Jan 10, 2022

Conversation

edymtt
Copy link
Contributor

@edymtt edymtt commented Jan 7, 2022

  • use quoted arguments when referencing "SWIFTLIB_SINGLE_SDK"
  • break the first condition on multiple lines

This will make changes to these conditions easier to review.

Addresses rdar://87339543.

* use quoted arguments when referencing "SWIFTLIB_SINGLE_SDK"
* break the first condition on multiple lines

This will make changes to these conditions easier to review.
@edymtt
Copy link
Contributor Author

edymtt commented Jan 7, 2022

@swift-ci please test

Copy link
Member

@compnerd compnerd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume that the reason for the change is more consistency rather than a build issue? If so, LGTM

@swift-ci
Copy link
Contributor

swift-ci commented Jan 7, 2022

Build failed
Swift Test OS X Platform
Git Sha - 3b82dc0

@edymtt
Copy link
Contributor Author

edymtt commented Jan 7, 2022

@swift-ci please smoke test macOS

@edymtt
Copy link
Contributor Author

edymtt commented Jan 10, 2022

@swift-ci please test macOS

@edymtt
Copy link
Contributor Author

edymtt commented Jan 10, 2022

@compnerd yes, that's my intention.

@edymtt edymtt merged commit b0ed490 into swiftlang:main Jan 10, 2022
@edymtt edymtt deleted the polish-bitcode-conditions branch January 10, 2022 20:22
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.

4 participants