Skip to content

[cc] Add cc toolchain to starlark cc_proto_library#16585

Closed
Yannic wants to merge 5 commits intobazelbuild:masterfrom
EngFlow:yannic-cc-proto-library-toolchain
Closed

[cc] Add cc toolchain to starlark cc_proto_library#16585
Yannic wants to merge 5 commits intobazelbuild:masterfrom
EngFlow:yannic-cc-proto-library-toolchain

Conversation

@Yannic
Copy link
Copy Markdown
Contributor

@Yannic Yannic commented Oct 27, 2022

Fixes #16577

@Yannic
Copy link
Copy Markdown
Contributor Author

Yannic commented Oct 27, 2022

cc @buildbreaker2021 @meteorcloudy

@sgowroji sgowroji added the team-Rules-CPP Issues for C++ rules label Oct 27, 2022
@fmeum
Copy link
Copy Markdown
Collaborator

fmeum commented Nov 2, 2022

@buildbreaker2021 Could you review? This has the fix for the fix as well as an integration test.

@buildbreaker2021 buildbreaker2021 added the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Nov 2, 2022
@copybara-service copybara-service bot closed this in 6c1e4b7 Nov 2, 2022
@sgowroji sgowroji removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Nov 2, 2022
ShreeM01 added a commit that referenced this pull request Nov 4, 2022
Fixes #16577

Closes #16585.

PiperOrigin-RevId: 485600476
Change-Id: I3c2985731ff800a2c6136f401ed3c7e5e89518ad

Co-authored-by: Yannic <contact@yannic-bonenberger.com>
lripoche pushed a commit to lripoche/bazel that referenced this pull request Jun 13, 2023
Fixes bazelbuild#16577

Closes bazelbuild#16585.

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

Labels

team-Rules-CPP Issues for C++ rules

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[6.0.0rc1] cc_proto_library analysis failure

4 participants