Skip to content

Remove osx_cc_wrapper.sh.tpl#25681

Closed
keith wants to merge 1 commit intobazelbuild:masterfrom
keith:ks/remove-osx_cc_wrapper.sh.tpl
Closed

Remove osx_cc_wrapper.sh.tpl#25681
keith wants to merge 1 commit intobazelbuild:masterfrom
keith:ks/remove-osx_cc_wrapper.sh.tpl

Conversation

@keith
Copy link
Copy Markdown
Member

@keith keith commented Mar 24, 2025

apple_support and rules_cc have their own copies of this. Anyone still
relying on this should update those rules.

@github-actions github-actions bot added team-Rules-CPP Issues for C++ rules awaiting-review PR is awaiting review from an assigned reviewer labels Mar 24, 2025
@keith
Copy link
Copy Markdown
Member Author

keith commented Mar 24, 2025

cc @meteorcloudy re e531e7b

@meteorcloudy
Copy link
Copy Markdown
Member

@keith Do you have time to take a look at fixing the tests?

@keith keith force-pushed the ks/remove-osx_cc_wrapper.sh.tpl branch from 6a32d32 to 7a8d59d Compare March 25, 2025 17:42
@keith keith requested a review from a team as a code owner March 25, 2025 17:42
@keith keith force-pushed the ks/remove-osx_cc_wrapper.sh.tpl branch 5 times, most recently from 5116db4 to ae0d727 Compare March 31, 2025 19:15
@keith keith force-pushed the ks/remove-osx_cc_wrapper.sh.tpl branch from ae0d727 to 2ad86c5 Compare March 31, 2025 19:25
apple_support and rules_cc have their own copies of this. Anyone still
relying on this should update those rules.
@keith keith force-pushed the ks/remove-osx_cc_wrapper.sh.tpl branch from 2ad86c5 to ffd1a93 Compare March 31, 2025 19:49
@keith
Copy link
Copy Markdown
Member Author

keith commented Mar 31, 2025

thanks, that lead to quite the rabbit hole. green now!

Copy link
Copy Markdown
Member

@meteorcloudy meteorcloudy left a comment

Choose a reason for hiding this comment

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

Thank you so much for digging into this!

@meteorcloudy meteorcloudy added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels Apr 1, 2025
@copybara-service copybara-service bot closed this in ccd3f68 Apr 2, 2025
@github-actions github-actions bot removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Apr 2, 2025
fmeum pushed a commit to fmeum/bazel that referenced this pull request Apr 25, 2025
apple_support and rules_cc have their own copies of this. Anyone still
relying on this should update those rules.

Closes bazelbuild#25681.

PiperOrigin-RevId: 743050086
Change-Id: Ie375152cd57f9f1f50b50552f2f12230c0135179
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.

2 participants