[PATCH] D86496: [not][test] Fix disable-symbolization.test when 'printenv' is not available

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 24 17:28:06 PDT 2020


MaskRay updated this revision to Diff 287531.
MaskRay added a comment.

Use -s to suppress 'cmp' output


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86496/new/

https://reviews.llvm.org/D86496

Files:
  llvm/test/tools/not/disable-symbolization.test


Index: llvm/test/tools/not/disable-symbolization.test
===================================================================
--- llvm/test/tools/not/disable-symbolization.test
+++ 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


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D86496.287531.patch
Type: text/x-patch
Size: 535 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200825/f37d7d5a/attachment.bin>


More information about the llvm-commits mailing list