[lld] [lld-macho][test] Require "shell" feature for usage of `ln -s` (PR #99355)

via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 17 09:51:10 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lld-macho

Author: Daniel Bertalan (BertalanD)

<details>
<summary>Changes</summary>

The use of `ln -s` is not guaranteed to be supported on Windows.

---
Full diff: https://github.com/llvm/llvm-project/pull/99355.diff


1 Files Affected:

- (modified) lld/test/MachO/implicit-and-allowable-clients.test (+1-1) 


``````````diff
diff --git a/lld/test/MachO/implicit-and-allowable-clients.test b/lld/test/MachO/implicit-and-allowable-clients.test
index 576db33af2ea0..f627d242a0075 100644
--- a/lld/test/MachO/implicit-and-allowable-clients.test
+++ b/lld/test/MachO/implicit-and-allowable-clients.test
@@ -1,4 +1,4 @@
-# REQUIRES: aarch64
+# REQUIRES: aarch64, shell
 # RUN: rm -rf %t; split-file %s %t
 # RUN: ln -s Versions/A/FrameworkPublic.tbd %t/System/Library/Frameworks/FrameworkPublic.framework/
 # RUN: ln -s Versions/A/FrameworkPrivate.tbd %t/System/Library/Frameworks/FrameworkPrivate.framework/

``````````

</details>


https://github.com/llvm/llvm-project/pull/99355


More information about the llvm-commits mailing list