Skip to content

Re-enable asan_recover.swift test on Darwin because it should work correctly there. #29294

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 22, 2020

Conversation

danliew-apple
Copy link
Contributor

Re-enable asan_recover.swift test on Darwin because it should work correctly there.
correctly there.

To do this just disable on Linux where the test is currently broken.

rdar://problem/58640751

@danliew-apple
Copy link
Contributor Author

@swift-ci Please test and merge

@danliew-apple
Copy link
Contributor Author

The logs for the failures are missing. I'll just trigger again.

@danliew-apple
Copy link
Contributor Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 8b39404

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 8b39404

correctly there.

To do this just disable on Linux where the test is currently broken.

rdar://problem/58640751
@delcypher delcypher force-pushed the re_enable_58640751_darwin branch from 8b39404 to c2185c2 Compare January 21, 2020 22:51
@danliew-apple
Copy link
Contributor Author

Build failure is completely unrelated to this change. Rebasing and trying again.

@danliew-apple
Copy link
Contributor Author

@swift-ci please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 8b39404

@swift-ci
Copy link
Contributor

Build failed
Swift Test OS X Platform
Git Sha - 8b39404

@danliew-apple
Copy link
Contributor Author

The CI is very sad. The failures have nothing to do with this PR so I'm just going to land this.

@danliew-apple danliew-apple merged commit dbe7170 into master-rebranch Jan 22, 2020
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.

3 participants