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

Revert "bazel: update rules_foreign_cc" #17875

Conversation

keith
Copy link
Member

@keith keith commented Aug 26, 2021

Reverts #17799

Seems like this wasn't enough yet again envoyproxy/envoy-mobile#1754

Signed-off-by: Keith Smiley [email protected]

@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Aug 26, 2021
@repokitteh-read-only
Copy link

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

🐱

Caused by: #17875 was opened by keith.

see: more, trace.

@phlax
Copy link
Member

phlax commented Aug 27, 2021

@keith you need to fix DCO, not sure if the other errors are genuine

@keith keith force-pushed the revert-17799-ks/revert-revert-bazel-update-rules_foreign_cc-17445-17780 branch from c38dc5f to fd1503c Compare August 27, 2021 17:59
@junr03
Copy link
Member

junr03 commented Aug 27, 2021

/retest

@repokitteh-read-only
Copy link

Retrying Azure Pipelines:
Check envoy-presubmit isn't fully completed, but will still attempt retrying.
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #17875 (comment) was created by @junr03.

see: more, trace.

@junr03
Copy link
Member

junr03 commented Aug 27, 2021

/retest

@repokitteh-read-only
Copy link

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #17875 (comment) was created by @junr03.

see: more, trace.

@mattklein123 mattklein123 merged commit 7760bc0 into envoyproxy:main Aug 27, 2021
@keith keith deleted the revert-17799-ks/revert-revert-bazel-update-rules_foreign_cc-17445-17780 branch August 27, 2021 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deps Approval required for changes to Envoy's external dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants