Skip to content
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

Use xcrun to invoke install_name_tool #16951

Conversation

keith
Copy link
Member

@keith keith commented Dec 7, 2022

/usr/bin/install_name_tool is a shim; it invokes install_name_tool in the current active Xcode.

This allows using this template file to configure toolchain on non-macOS. This doesn't change the behavior as both will end up invoking the same tool.

replaces #13462

`/usr/bin/install_name_tool` is a shim; it invokes install_name_tool
in the current active Xcode.

This allows using this template file to configure toolchain on
non-macOS. This doesn't change the behavior as both will end up invoking
the same tool.
@thii
Copy link
Member

thii commented Dec 8, 2022

Thanks!

@sgowroji sgowroji added team-Rules-CPP Issues for C++ rules awaiting-review PR is awaiting review from an assigned reviewer labels Dec 8, 2022
@buildbreaker2021 buildbreaker2021 self-assigned this Jan 12, 2023
@buildbreaker2021 buildbreaker2021 self-requested a review January 12, 2023 14:00
@buildbreaker2021 buildbreaker2021 added the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Jan 12, 2023
@Wyverald Wyverald closed this in 8ec8743 Jan 12, 2023
@keith keith deleted the ks/use-xcrun-to-invoke-install_name_tool branch January 12, 2023 18:16
@brentleyjones
Copy link
Contributor

@bazel-io flag

@bazel-io bazel-io added the potential release blocker Flagged by community members using "@bazel-io flag". Should be added to a release blocker milestone label Jan 12, 2023
@sgowroji sgowroji removed awaiting-review PR is awaiting review from an assigned reviewer awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally labels Jan 13, 2023
@ShreeM01
Copy link
Contributor

@bazel-io fork 6.1.0

@bazel-io bazel-io removed the potential release blocker Flagged by community members using "@bazel-io flag". Should be added to a release blocker milestone label Jan 18, 2023
ShreeM01 added a commit that referenced this pull request Jan 19, 2023
`/usr/bin/install_name_tool` is a shim; it invokes install_name_tool in the current active Xcode.

This allows using this template file to configure toolchain on non-macOS. This doesn't change the behavior as both will end up invoking the same tool.

replaces #13462

Closes #16951.

PiperOrigin-RevId: 501557408
Change-Id: Ibf0b90aaf18bd703079296248cd66dc6f86359f1

Co-authored-by: Keith Smiley <[email protected]>
hvadehra pushed a commit that referenced this pull request Feb 14, 2023
`/usr/bin/install_name_tool` is a shim; it invokes install_name_tool in the current active Xcode.

This allows using this template file to configure toolchain on non-macOS. This doesn't change the behavior as both will end up invoking the same tool.

replaces #13462

Closes #16951.

PiperOrigin-RevId: 501557408
Change-Id: Ibf0b90aaf18bd703079296248cd66dc6f86359f1
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.

7 participants