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

[vcpkg-scripts] z_vcpkg_fixup_macho_rpath_in_dir: Call install_name_tool only once #39995

Conversation

autoantwort
Copy link
Contributor

@autoantwort autoantwort commented Jul 20, 2024

Should probably merged at the same time as #39889 to reduce would rebuilds

scripts/ports.cmake Outdated Show resolved Hide resolved
@autoantwort autoantwort force-pushed the feature/z_vcpkg_fixup_macho_rpath_in_dir branch from f37740d to 2ba5e2f Compare July 20, 2024 19:43
@MonicaLiu0311 MonicaLiu0311 added the category:tool-update The issue is with build tool or build script, which requires update or should be executed correctly label Jul 22, 2024
@data-queue
Copy link
Contributor

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@MonicaLiu0311
Copy link
Contributor

MonicaLiu0311 commented Jul 24, 2024

Should probably merged at the same time as #39889 to reduce would rebuilds

Waiting for #39889.

@data-queue data-queue added the info:reviewed Pull Request changes follow basic guidelines label Jul 24, 2024
@Cheney-W Cheney-W changed the title z_vcpkg_fixup_macho_rpath_in_dir: Call install_name_tool only once [vcpkg-scripts] z_vcpkg_fixup_macho_rpath_in_dir: Call install_name_tool only once Jul 26, 2024
@data-queue data-queue merged commit a68201b into microsoft:master Jul 27, 2024
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:tool-update The issue is with build tool or build script, which requires update or should be executed correctly info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants