Skip to content

Commit 74f7c76

Browse files
committed
tests: fix accidental not env usage
Manually expand out the few places where we had a `not ...` substitution where the substituted value was invoking `env` to alter the environment. This pattern is not portable and causes problems when using the integrated shell, such as on Windows. This was identified during the LLVM update. This additionally corrects the use of `not` to indicate that it is expecting a crashing failure. This was resulting in the Windows tests failing as `not` was not expecting a crashing failure.
1 parent 8f4c364 commit 74f7c76

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

test/ModuleInterface/swift_build_sdk_interfaces/compiler-crash.test-sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
RUN: not %swift_build_sdk_interfaces -sdk %S/Inputs/mock-sdk/ -o %t/output -debug-crash-compiler 2>&1 | %FileCheck %s
1+
RUN: env SWIFT_EXEC=%swiftc_driver_plain not --crash %{python} %utils/swift_build_sdk_interfaces.py %mcp_opt -sdk %S/Inputs/mock-sdk/ -o %t/output -debug-crash-compiler 2>&1 | %FileCheck %s
22

33
CHECK: Program arguments:
44
CHECK-SAME: -debug-crash-immediately

0 commit comments

Comments
 (0)