Skip to content
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

platform: remove re-definition of mmsghdr for Android targets #10521

Merged
merged 2 commits into from
Mar 26, 2020
Merged

platform: remove re-definition of mmsghdr for Android targets #10521

merged 2 commits into from
Mar 26, 2020

Conversation

junr03
Copy link
Member

@junr03 junr03 commented Mar 25, 2020

Description: remove Android flag that caused re-definition issues with Android's ndk for mmsghdr.
Risk Level: low given Envoy Mobile is the only active, well-known Android target for Envoy.
Testing: built Envoy Mobile for Android successfully.

Signed-off-by: Jose Nino jnino@lyft.com

Jose Nino added 2 commits March 25, 2020 12:47
Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: Jose Nino <jnino@lyft.com>
@junr03
Copy link
Member Author

junr03 commented Mar 25, 2020

Chatted with @danzh2010 in #envoy-udp-quic-dev

@junr03
Copy link
Member Author

junr03 commented Mar 26, 2020

The aws_lambda_filter_integration_test seems to be flaky from inception per #10398 (comment). I am going to merge here in spite of the mac os failure.

@junr03 junr03 merged commit 15878f9 into envoyproxy:master Mar 26, 2020
junr03 added a commit to envoyproxy/envoy-mobile that referenced this pull request Mar 26, 2020
Description: updates envoy to f4931758d29b2aa6be89900650aeee54addb1827. This brings a few relevant changes for envoy mobile:
-  Additional observability for virtual clusters: envoyproxy/envoy#10474 envoyproxy/envoy#10499 envoyproxy/envoy#10501
-  32-bit compatibility fix envoyproxy/envoy#10510
- Android re-definition of `mmsghdr` fix envoyproxy/envoy#10521

Signed-off-by: Jose Nino <jnino@lyft.com>
jpsim pushed a commit that referenced this pull request Nov 28, 2022
Description: updates envoy to f493175. This brings a few relevant changes for envoy mobile:
-  Additional observability for virtual clusters: #10474 #10499 #10501
-  32-bit compatibility fix #10510
- Android re-definition of `mmsghdr` fix #10521

Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: JP Simard <jp@jpsim.com>
jpsim pushed a commit that referenced this pull request Nov 29, 2022
Description: updates envoy to f493175. This brings a few relevant changes for envoy mobile:
-  Additional observability for virtual clusters: #10474 #10499 #10501
-  32-bit compatibility fix #10510
- Android re-definition of `mmsghdr` fix #10521

Signed-off-by: Jose Nino <jnino@lyft.com>
Signed-off-by: JP Simard <jp@jpsim.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.

3 participants