Skip to content

Conversation

gottesmm
Copy link
Contributor

@gottesmm gottesmm commented Nov 6, 2018

…a wrapper to add_llvm_library"""

This reverts commit 121f5b6.

Sorry to revert this again. This commit makes some pretty big changes. After
messing with the merge-conflict created by this internally, I did not feel
comfortable landing this now. I talked with Saleem and he agreed with me that
this was the right thing to do.

…a wrapper to add_llvm_library"""

This reverts commit 121f5b6.

Sorry to revert this again. This commit makes some pretty big changes. After
messing with the merge-conflict created by this internally, I did not feel
comfortable landing this now. I talked with Saleem and he agreed with me that
this was the right thing to do.
@gottesmm gottesmm requested a review from compnerd November 6, 2018 21:24
@gottesmm
Copy link
Contributor Author

gottesmm commented Nov 6, 2018

@swift-ci smoke test and merge

@gottesmm
Copy link
Contributor Author

gottesmm commented Nov 6, 2018

the xctest failure = /.

@gottesmm
Copy link
Contributor Author

gottesmm commented Nov 6, 2018

@swift-ci smoke test linux platform

@milseman
Copy link
Member

milseman commented Nov 7, 2018

@compnerd, @gottesmm informs me that you are ok with reverting this for now until after we branch for 5.0, to work around some current instability.

Force merging now to unblock CI systems.

@milseman milseman merged commit c3785e2 into swiftlang:master Nov 7, 2018
@gottesmm gottesmm deleted the pr-657153590cac7970ccbc5c7e03ccf477d42a24fc branch November 7, 2018 00:13
@gottesmm
Copy link
Contributor Author

gottesmm commented Nov 7, 2018

Thanks @milseman!

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