Skip to content

Conversation

@ktopley-apple
Copy link
Contributor

@ktopley-apple ktopley-apple commented Jan 25, 2017

The same change was already made in the swift repo (swiftlang/swift#6958)

rdar://problem/27915073

@ktopley-apple
Copy link
Contributor Author

@swift-ci Please test

Copy link
Contributor

@moiseev moiseev left a comment

Choose a reason for hiding this comment

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

:shipit:

@ktopley-apple
Copy link
Contributor Author

Test failure is unrelated to this change.

@ktopley-apple
Copy link
Contributor Author

@swift-ci Please test

@ktopley-apple
Copy link
Contributor Author

@dgrove-oss could you please take a look at the build output for Linux? It's failing with a timeout in the SwiftPM tests.

@dgrove-oss
Copy link
Contributor

I think there is something broken in how the libdispatch PR testing is being done. It has been habitually failing on this swiftPM subtest (and we have been ignoring the failure, merging the changes, and not seeing the failure in the main runs). I'll ask for help on swift-dev list to see if anyone has an idea of what could be wrong.

@ktopley-apple
Copy link
Contributor Author

@dgrove-oss thanks. The failure is clearly not related to the change, so I am going to go ahead and merge.

@ktopley-apple ktopley-apple merged commit b6e9cf4 into swiftlang:master Jan 27, 2017
das pushed a commit that referenced this pull request Feb 21, 2017
Correct the Xcode fixits for dispatch_async() and dispatch_group_async()

Signed-off-by: Daniel A. Steffen <dsteffen@apple.com>
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