Skip to content

dependencies: avoid hardcoding version string for curl/ICU/c-ares.#12943

Merged
htuch merged 3 commits intoenvoyproxy:masterfrom
htuch:version-strings
Sep 3, 2020
Merged

dependencies: avoid hardcoding version string for curl/ICU/c-ares.#12943
htuch merged 3 commits intoenvoyproxy:masterfrom
htuch:version-strings

Conversation

@htuch
Copy link
Copy Markdown
Member

@htuch htuch commented Sep 2, 2020

@moderation suggested a better approach in baze://github.com//pull/12639#issuecomment-676606571.

Signed-off-by: Harvey Tuch htuch@google.com

@moderation suggested a better approach in baze://github.com/envoyproxy/pull/12639#issuecomment-676606571.

Signed-off-by: Harvey Tuch <htuch@google.com>
@moderation
Copy link
Copy Markdown
Contributor

moderation commented Sep 2, 2020

LGTM. I've tested this in my local environments (x86_64 Linux + MacOS, aarch64) without issue.

Edit - Looks like I spoke too soon as CI failed

snowp
snowp previously approved these changes Sep 2, 2020
Signed-off-by: Harvey Tuch <htuch@google.com>
@htuch htuch merged commit 126a766 into envoyproxy:master Sep 3, 2020
@htuch htuch deleted the version-strings branch September 3, 2020 23:47
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