[llvm-branch-commits] [ASan] Make duplicate_os_log_reports.cpp work with the internal shell (PR #168656)

[Aiden Grossman via llvm-branch-commits]

================
@@ -1,20 +1,16 @@
 // UNSUPPORTED: ios
 // REQUIRES: darwin_log_cmd
 // RUN: %clangxx_asan -fsanitize-recover=address %s -o %t
-// RUN: { %env_asan_opts=halt_on_error=0,log_to_syslog=1 %run %t > %t.process_output.txt 2>&1 & } \
-// RUN: ; export TEST_PID=$! ; wait ${TEST_PID}
+// RUN: bash -c "{ %env_asan_opts=halt_on_error=0,log_to_syslog=1 %run %t > %t.process_output.txt 2>&1 & } \
+// RUN: ; export TEST_PID=$! ; wait ${TEST_PID}; echo -n ${TEST_PID} > %t.test_pid"
----------------
boomanaiden154 wrote:

I've updated the test to use single quotes so the variables do not get substituted by the outer shell. Oddly, the test seems to pass either way with the external shell, but we probably lose some fidelity without the singular quotes.

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