[llvm] e40c4dd - [llvm-rc] Avoid which(1) dependency in windres-prefix.test

Michał Górny via llvm-commits llvm-commits at lists.llvm.org
Wed May 25 07:15:01 PDT 2022


Author: Michał Górny
Date: 2022-05-25T16:14:47+02:00
New Revision: e40c4dd66218393214ed7a817ebb82979adad01a

URL: https://github.com/llvm/llvm-project/commit/e40c4dd66218393214ed7a817ebb82979adad01a
DIFF: https://github.com/llvm/llvm-project/commit/e40c4dd66218393214ed7a817ebb82979adad01a.diff

LOG: [llvm-rc] Avoid which(1) dependency in windres-prefix.test

Rely on lit substitution to provide the path to llvm-windres instead
of redundantly calling which(1) with the subtituted absolute path.
This fixes test failure on the happy systems without which(1) installed.

Differential Revision: https://reviews.llvm.org/D126366

Added: 
    

Modified: 
    llvm/test/tools/llvm-rc/windres-prefix.test

Removed: 
    


################################################################################
diff  --git a/llvm/test/tools/llvm-rc/windres-prefix.test b/llvm/test/tools/llvm-rc/windres-prefix.test
index ef85f681febd1..9d24bf9ee7b2a 100644
--- a/llvm/test/tools/llvm-rc/windres-prefix.test
+++ b/llvm/test/tools/llvm-rc/windres-prefix.test
@@ -4,7 +4,7 @@
 
 ; Check that a triple prefix on the executable gets picked up as target triple.
 
-; RUN: ln -fs $(which llvm-windres) %t/aarch64-w64-mingw32-windres
+; RUN: ln -fs llvm-windres %t/aarch64-w64-mingw32-windres
 ; RUN: %t/aarch64-w64-mingw32-windres -### %p/Inputs/empty.rc %t.res | FileCheck %s --check-prefix=CHECK-PREPROC
 ; CHECK-PREPROC: "clang" "--driver-mode=gcc" "-target" "aarch64-w64-mingw32"
 


        


More information about the llvm-commits mailing list