Skip to content

[5.0] Fix build by changing test to only run on macOS. #14275

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

Conversation

gottesmm
Copy link
Contributor

The reason why this snuck through before is that we are not building asan on the
public bots for simulators. I am going to look into tightening this up, but in
the short term, lets get some green builds.

rdar://36916944

The reason why this snuck through before is that we are not building asan on the
public bots for simulators. I am going to look into tightening this up, but in
the short term, lets get some green builds.

rdar://36916944
@gottesmm gottesmm force-pushed the swift-5.0-branch_rdar36916944 branch from a151dd0 to a78fed1 Compare January 30, 2018 17:44
@gottesmm
Copy link
Contributor Author

@swift-ci test

@gottesmm
Copy link
Contributor Author

I just confirmed with Bob that Jordan's review from 4.1 is sufficient for 5.0 as well. I am going to let this finish testing and then send this PR to the CCC list.

@gottesmm
Copy link
Contributor Author

The 4.1 variant of this is #14238.

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - a78fed1

@gottesmm
Copy link
Contributor Author

The playground logger failure is not me. I just emailed @cwakamo about this.

@gottesmm
Copy link
Contributor Author

@swift-ci test os x platform

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - a78fed1

@gottesmm
Copy link
Contributor Author

@swift-ci test os x platform

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - a78fed1

@gottesmm
Copy link
Contributor Author

Same playground failure. Maybe I tried too soon before the fix was merged. Trying again...

@gottesmm
Copy link
Contributor Author

@swift-ci test

@gottesmm gottesmm merged commit f43883e into swiftlang:swift-5.0-branch Feb 1, 2018
@gottesmm gottesmm deleted the swift-5.0-branch_rdar36916944 branch February 1, 2018 02:31
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.

2 participants