[PATCH] D82410: [llvm-install-name-tool] Add -id option

[Shoaib Meenai via Phabricator via llvm-commits]

smeenai added inline comments.


================
Comment at: llvm/test/tools/llvm-objcopy/MachO/install-name-tool-id.test:13
+# RUN: llvm-install-name-tool -id /usr/lib/B_long -id /usr/lib/K_long -id /usr/A_short  %t
+# RUN: llvm-objdump -p %t | FileCheck %s --check-prefix=ID-MULTIPLE --implicit-check-not=/usr
+
----------------
thakis wrote:
> This breaks check-llvm for anyone who has their LLVM checkout in a directory that has "/usr" anywhere on their checkout path, for example this bot: http://45.33.8.238/linux/21750/step_12.txt
Thanks for the report, fixed in rG5f56da3763ac


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82410/new/

https://reviews.llvm.org/D82410