Skip to content

Consolidate on @rules_cc and update it to the latest version#6580

Merged
josh11b merged 1 commit intocarbon-language:trunkfrom
chandlerc:push-mryllvmnonow
Jan 13, 2026
Merged

Consolidate on @rules_cc and update it to the latest version#6580
josh11b merged 1 commit intocarbon-language:trunkfrom
chandlerc:push-mryllvmnonow

Conversation

@chandlerc
Copy link
Copy Markdown
Contributor

@chandlerc chandlerc commented Jan 10, 2026

Also consolidate on using //bazel/cc_rules:defs.bzl where appropriate.

Also update a couple of Bazel modules deps of @rules_cc to the latest
versions.

@chandlerc chandlerc requested a review from a team as a code owner January 10, 2026 10:20
@chandlerc chandlerc requested review from josh11b and removed request for a team January 10, 2026 10:20
Copy link
Copy Markdown
Contributor

@josh11b josh11b left a comment

Choose a reason for hiding this comment

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

I don't see the "Switch install to be based on the busybox root" change, only the "//bazel/cc_rules:defs.bzl" changes.

@chandlerc chandlerc changed the title Switch install to be based on the busybox root Consolidate on @rules_cc and update it to the latest version Jan 11, 2026
@chandlerc
Copy link
Copy Markdown
Contributor Author

Sorry, just ended up pasting the wrong PR title. This PR is the one that consolidates on @rules_cc.

@josh11b josh11b enabled auto-merge January 12, 2026 16:29
Also consolidate on using `//bazel/cc_rules:defs.bzl` where appropriate.
@chandlerc
Copy link
Copy Markdown
Contributor Author

PTAL -- I needed to bump a couple of other modules to fix Bazel warnings.

@josh11b josh11b added this pull request to the merge queue Jan 13, 2026
Merged via the queue into carbon-language:trunk with commit 93c7c9a Jan 13, 2026
8 checks passed
@chandlerc chandlerc deleted the push-mryllvmnonow branch January 13, 2026 03:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants