Skip to content

Revert "Revert "Introduce CFRunLoop based iomgr""#22687

Merged
muxi merged 2 commits intomasterfrom
revert-22684-revert-22423-objc-poller
Apr 20, 2020
Merged

Revert "Revert "Introduce CFRunLoop based iomgr""#22687
muxi merged 2 commits intomasterfrom
revert-22684-revert-22423-objc-poller

Conversation

@muxi
Copy link
Copy Markdown
Contributor

@muxi muxi commented Apr 16, 2020

Reverts #22684

Breakage cause: grpc_cfstream tests need to be run with CFStream. This PR removes GRPC_CFSTREAM macro for Mac. We will need to enable it with --copt in bazel command.

@muxi muxi added lang/core release notes: no Indicates if PR should not be in release notes labels Apr 16, 2020
@muxi muxi requested a review from guantaol April 17, 2020 00:15
@muxi muxi marked this pull request as ready for review April 17, 2020 00:15
@muxi
Copy link
Copy Markdown
Contributor Author

muxi commented Apr 17, 2020

@guantaol - ready to review. Sorry for the trouble.

@muxi
Copy link
Copy Markdown
Contributor Author

muxi commented Apr 17, 2020

actually it's not... let me take a look

@muxi muxi force-pushed the revert-22684-revert-22423-objc-poller branch from 4b1d4bf to 8bff072 Compare April 18, 2020 00:00
@muxi
Copy link
Copy Markdown
Contributor Author

muxi commented Apr 20, 2020

Good now.

@muxi
Copy link
Copy Markdown
Contributor Author

muxi commented Apr 20, 2020

Known failure: #18892

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lang/core release notes: no Indicates if PR should not be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants