Skip to content

Flip incompatible_enable_cc_toolchain_resolution#16252

Closed
comius wants to merge 16 commits intobazelbuild:masterfrom
comius:flip-cc_toolchain_resolution
Closed

Flip incompatible_enable_cc_toolchain_resolution#16252
comius wants to merge 16 commits intobazelbuild:masterfrom
comius:flip-cc_toolchain_resolution

Conversation

@comius
Copy link
Copy Markdown
Contributor

@comius comius commented Sep 12, 2022

RELNOTES[INC]: Flip incompatible_enable_cc_toolchain_resolution (#7260)

@comius comius requested a review from lberki as a code owner September 12, 2022 08:32
@comius comius removed the request for review from lberki September 12, 2022 08:32
@comius comius marked this pull request as draft September 12, 2022 08:32
Copy link
Copy Markdown
Contributor

@lberki lberki left a comment

Choose a reason for hiding this comment

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

This is great. I mean, it will be great when it sticks, but merging this pull request is a necessary part of the path for getting there :)

@comius comius force-pushed the flip-cc_toolchain_resolution branch from df5b46d to a4c82eb Compare September 19, 2022 05:33
@comius comius force-pushed the flip-cc_toolchain_resolution branch from 6a8c83b to 13f5ea6 Compare September 19, 2022 08:35
@comius
Copy link
Copy Markdown
Contributor Author

comius commented Sep 19, 2022

Downstream tests: https://buildkite.com/bazel/bazel-at-head-plus-downstream/builds/2641#01835443-ca23-4d89-b700-dd2ef89c8f7a

Failures:

@comius comius changed the title Flip cc toolchain resolution Flip incompatible_enable_cc_toolchain_resolution Sep 19, 2022
@comius comius requested a review from meteorcloudy September 19, 2022 11:24
@comius comius marked this pull request as ready for review September 19, 2022 11:24
@comius
Copy link
Copy Markdown
Contributor Author

comius commented Sep 19, 2022

Failing test needs rules_cc 0.0.2 on bzlmod repository.

@comius comius requested a review from oquenchil as a code owner September 19, 2022 14:29
@meteorcloudy
Copy link
Copy Markdown
Member

You can also view the broken downstream project here:
https://buildkite.com/bazel/bazelisk-plus-incompatible-flags/builds/1264

image

@ShreeM01 ShreeM01 added team-Rules-CPP Issues for C++ rules awaiting-review PR is awaiting review from an assigned reviewer labels Sep 20, 2022
@comius comius requested a review from lberki September 20, 2022 08:38
aiuto pushed a commit to aiuto/bazel that referenced this pull request Oct 12, 2022
RELNOTES[INC]: Flip incompatible_enable_cc_toolchain_resolution (bazelbuild#7260)

Closes bazelbuild#16252.

PiperOrigin-RevId: 476058407
Change-Id: I6ceb9cd306e0eeed61cb1ff8cd9804d26988d22d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-review PR is awaiting review from an assigned reviewer team-Rules-CPP Issues for C++ rules

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants