Skip to content

Commit

Permalink
[not][test] Fix disable-symbolization.test when 'printenv' is not ava…
Browse files Browse the repository at this point in the history
…ilable

On Windows, 'env' or 'printenv' may not exist.

Also switch back to 'env' which is specified by POSIX.1-2017. 'printenv' is not
standard (I picked it because 'printenv' exists on GnuWin32 but 'env' does not).

Reviewed By: zequanwu

Differential Revision: https://reviews.llvm.org/D86496
  • Loading branch information
MaskRay committed Aug 25, 2020
1 parent 4283320 commit a5e65ad
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions llvm/test/tools/not/disable-symbolization.test
@@ -1,5 +1,6 @@
# RUN: not --crash printenv > %t || true
# RUN: FileCheck %s < %t
## 'env' may not exist on Windows. %t will be empty.
# RUN: not --crash env > %t || true
# RUN: cmp -s %t /dev/null || FileCheck %s < %t

# CHECK-DAG: LLVM_DISABLE_CRASH_REPORT=1
# CHECK-DAG: LLVM_DISABLE_SYMBOLIZATION=1

0 comments on commit a5e65ad

Please sign in to comment.