[flang-commits] [flang] [flang] Fix exec.f90 test on LIT integrated shell (PR #93961)

[via flang-commits]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-flang-driver

Author: David Truby (DavidTruby)

<details>
<summary>Changes</summary>

The exec.f90 test sets an environment variable for a specific command directly
rather than using env, which doesn't work on shells that don't support this
syntax, most notably the LIT integrated shell. This patch simply adds env so
that this works on the integrated shell.


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


1 Files Affected:

- (modified) flang/test/Driver/exec.f90 (+1-1) 


``````````diff
diff --git a/flang/test/Driver/exec.f90 b/flang/test/Driver/exec.f90
index 4a2e66bd2aac4..fd174005ddf62 100644
--- a/flang/test/Driver/exec.f90
+++ b/flang/test/Driver/exec.f90
@@ -2,7 +2,7 @@
 ! Verify that flang can correctly build executables.
 
 ! RUN: %flang %s -o %t
-! RUN: LD_LIBRARY_PATH="$LD_LIBRARY_PATH:%llvmshlibdir" %t | FileCheck %s
+! RUN: env LD_LIBRARY_PATH="$LD_LIBRARY_PATH:%llvmshlibdir" %t | FileCheck %s
 ! RUN: rm -f %t
 
 ! CHECK: Hello, World!

``````````

</details>


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