Skip to content

build: update zlib-ng to latest.#13642

Merged
htuch merged 3 commits intoenvoyproxy:masterfrom
PiotrSikora:update_zlib_ng
Oct 22, 2020
Merged

build: update zlib-ng to latest.#13642
htuch merged 3 commits intoenvoyproxy:masterfrom
PiotrSikora:update_zlib_ng

Conversation

@PiotrSikora
Copy link
Copy Markdown
Contributor

Signed-off-by: Piotr Sikora [email protected]

@repokitteh-read-only
Copy link
Copy Markdown

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt).

🐱

Caused by: #13642 was opened by PiotrSikora.

see: more, trace.

@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Oct 20, 2020
@moderation
Copy link
Copy Markdown
Contributor

LGTM

@PiotrSikora
Copy link
Copy Markdown
Contributor Author

It looks like my PR upstream might actually be merged soon, so let's wait for it.

@PiotrSikora
Copy link
Copy Markdown
Contributor Author

It looks like my PR upstream might actually be merged soon, so let's wait for it.

Actually, I might have underestimated the turnaround time on that PR. Let's merge this as-is, and in the worst case I'll include a 6-line patch in the PR that depends on it.

@PiotrSikora PiotrSikora requested a review from htuch October 22, 2020 19:26
@htuch
Copy link
Copy Markdown
Member

htuch commented Oct 22, 2020

/lgtm deps

@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Oct 22, 2020
@htuch htuch merged commit e647b51 into envoyproxy:master Oct 22, 2020
chradcliffe pushed a commit to chradcliffe/envoy that referenced this pull request Oct 23, 2020
Signed-off-by: Piotr Sikora <[email protected]>
Signed-off-by: Craig Radcliffe <[email protected]>
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