Skip to content

[WIP] bazel: Resolve toolchains correctly #38938

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

phlax
Copy link
Member

@phlax phlax commented Mar 27, 2025

the up to date bazel way of resolving toolchains is currently disabled, we will need to fix this in order to update bazel 8+ (and to update some deps)

@phlax phlax changed the title bazel: Resolve toolchains correctly [WIP] bazel: Resolve toolchains correctly Mar 27, 2025
@phlax phlax marked this pull request as draft March 27, 2025 14:07
@phlax phlax force-pushed the icompatibly-resolve-toolchains branch 3 times, most recently from d68909d to d588c4d Compare March 27, 2025 17:01
@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Mar 27, 2025
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).
envoyproxy/dependency-shepherds assignee is @RyanTheOptimist

🐱

Caused by: #38938 was synchronize by phlax.

see: more, trace.

@phlax phlax force-pushed the icompatibly-resolve-toolchains branch from f14f8d5 to 03c3ef0 Compare March 28, 2025 08:00
Copy link

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

@github-actions github-actions bot added the stale stalebot believes this issue/PR has not been touched recently label Apr 27, 2025
@phlax phlax removed the stale stalebot believes this issue/PR has not been touched recently label Apr 27, 2025
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.

2 participants