Skip to content

build: Update protobuf#6180

Merged
mattklein123 merged 1 commit intoenvoyproxy:masterfrom
keith:ks/update-protobuf
Mar 6, 2019
Merged

build: Update protobuf#6180
mattklein123 merged 1 commit intoenvoyproxy:masterfrom
keith:ks/update-protobuf

Conversation

@keith
Copy link
Copy Markdown
Member

@keith keith commented Mar 5, 2019

In order to compile envoy for iOS we need this commit protocolbuffers/protobuf@0894e07 from protobuf. This also includes the previous commits that required us to use a non-release version.

Risk Level: Low

In order to compile envoy for iOS we need this commit
protocolbuffers/protobuf@0894e07
from protobuf. This also includes the previous commits that required us
to use a non-release version.

Signed-off-by: Keith Smiley <keithbsmiley@gmail.com>
@keith keith force-pushed the ks/update-protobuf branch from ee1ac40 to 0264fd7 Compare March 5, 2019 23:31
@lizan lizan requested a review from htuch March 5, 2019 23:40
@lizan lizan assigned lizan and htuch Mar 5, 2019
Copy link
Copy Markdown
Member

@lizan lizan left a comment

Choose a reason for hiding this comment

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

Nice clean up!

Copy link
Copy Markdown
Member

@htuch htuch left a comment

Choose a reason for hiding this comment

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

Thanks!

@mattklein123 mattklein123 merged commit be066bc into envoyproxy:master Mar 6, 2019
@keith keith deleted the ks/update-protobuf branch March 6, 2019 04:54
fredlas pushed a commit to fredlas/envoy that referenced this pull request Mar 6, 2019
In order to compile envoy for iOS we need this commit
protocolbuffers/protobuf@0894e07
from protobuf. This also includes the previous commits that required us
to use a non-release version.

Signed-off-by: Keith Smiley <keithbsmiley@gmail.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