Skip to content

[iOS] Disable posix polling on iOS#33094

Merged
drfloob merged 2 commits intogrpc:masterfrom
drfloob:ios-temp-fix
May 12, 2023
Merged

[iOS] Disable posix polling on iOS#33094
drfloob merged 2 commits intogrpc:masterfrom
drfloob:ios-temp-fix

Conversation

@drfloob
Copy link
Copy Markdown
Member

@drfloob drfloob commented May 12, 2023

@sampajano This should fix b/265779666. If the CFEngine ends up taking some time to land with rollbacks, bugs and whatnot, this can work in the meantime.

Copy link
Copy Markdown
Contributor

@sampajano sampajano left a comment

Choose a reason for hiding this comment

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

thank you!

@drfloob drfloob merged commit eecc897 into grpc:master May 12, 2023
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label May 15, 2023
eugeneo pushed a commit to eugeneo/grpc that referenced this pull request May 17, 2023
@sampajano This should fix b/265779666. If the CFEngine ends up taking
some time to land with rollbacks, bugs and whatnot, this can work in the
meantime.
wanlin31 pushed a commit that referenced this pull request May 18, 2023
@sampajano This should fix b/265779666. If the CFEngine ends up taking
some time to land with rollbacks, bugs and whatnot, this can work in the
meantime.
gnossen pushed a commit to gnossen/grpc that referenced this pull request Jun 2, 2023
This should fix b/265779666. If the CFEngine ends up taking
some time to land with rollbacks, bugs and whatnot, this can work in the
meantime.
gnossen added a commit that referenced this pull request Jun 2, 2023
Turns out this is needed in addition to
#33320 to resolve the MacOS fork issues
for Python.

CC @sampajano

Co-authored-by: AJ Heller <hork@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bloat/none imported Specifies if the PR has been imported to the internal repository lang/core per-call-memory/neutral per-channel-memory/neutral 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.

4 participants