r334494 - Re-apply r334418 "Enable crash recovery tests on Windows, globs work in the lit internal shell now"

Tue Jun 12 13:52:04 PDT 2018

The length of that path is about 257, or very near MAX_PATH, which suggests
that the version of rm you are using does not tolerate deep paths. This has
been a long standing problem, and our solution to it was to install a
special version of rm.exe on all of our buildbots that reliably handles
long paths and copes with virus scanners and things.

We may try to fix this in lit by reimplementing 'rm' in python, but that is
not a near term solution.

On Tue, Jun 12, 2018 at 1:30 PM Galina Kistanova <gkistanova at gmail.com>
wrote:

> Hello Hans,
>
> The test Driver/crash-report-modules.m still fails on one of windows
> builder:
>
>
> http://lab.llvm.org:8011/builders/llvm-clang-x86_64-expensive-checks-win/builds/10221
>
> . . .
> Failing Tests (1):
>     Clang :: Driver/crash-report-modules.m
>
> Please have a look?
>
> Thanks
>
> Galina
>
>
>
> On Tue, Jun 12, 2018 at 4:51 AM, Hans Wennborg via cfe-commits <
> cfe-commits at lists.llvm.org> wrote:
>
>> Author: hans
>> Date: Tue Jun 12 04:51:22 2018
>> New Revision: 334494
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=334494&view=rev
>> Log:
>> Re-apply r334418 "Enable crash recovery tests on Windows, globs work in
>> the lit internal shell now"
>>
>> Plus change run lines from
>>
>>   not env FOO=bar %clang
>>
>> to
>>
>>   env FOO=bar not %clang
>>
>> To not confuse the internal shell.
>>
>> Modified:
>>     cfe/trunk/test/Driver/crash-report-header.h
>>     cfe/trunk/test/Driver/crash-report-modules.m
>>     cfe/trunk/test/Driver/crash-report-spaces.c
>>     cfe/trunk/test/Driver/crash-report.c
>>
>> Modified: cfe/trunk/test/Driver/crash-report-header.h
>> URL:
>> http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/crash-report-header.h?rev=334494&r1=334493&r2=334494&view=diff
>>
>> ==============================================================================
>> --- cfe/trunk/test/Driver/crash-report-header.h (original)
>> +++ cfe/trunk/test/Driver/crash-report-header.h Tue Jun 12 04:51:22 2018
>> @@ -1,13 +1,10 @@
>>  // RUN: rm -rf %t
>>  // RUN: mkdir %t
>> -// RUN: not env TMPDIR="%t" TEMP="%t" TMP="%t" RC_DEBUG_OPTIONS=1 %clang
>> -fsyntax-only %s 2>&1 | FileCheck %s
>> +// RUN: env TMPDIR="%t" TEMP="%t" TMP="%t" RC_DEBUG_OPTIONS=1 not %clang
>> -fsyntax-only %s 2>&1 | FileCheck %s
>>  // RUN: cat %t/crash-report-header-*.h | FileCheck
>> --check-prefix=CHECKSRC "%s"
>>  // RUN: cat %t/crash-report-header-*.sh | FileCheck
>> --check-prefix=CHECKSH "%s"
>>  // REQUIRES: crash-recovery
>>
>> -// because of the glob (*.h, *.sh)
>> -// REQUIRES: shell
>> -
>>  #pragma clang __debug parser_crash
>>  // CHECK: Preprocessed source(s) and associated run script(s) are
>> located at:
>>  // CHECK-NEXT: note: diagnostic msg: {{.*}}.h
>>
>> Modified: cfe/trunk/test/Driver/crash-report-modules.m
>> URL:
>> http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/crash-report-modules.m?rev=334494&r1=334493&r2=334494&view=diff
>>
>> ==============================================================================
>> --- cfe/trunk/test/Driver/crash-report-modules.m (original)
>> +++ cfe/trunk/test/Driver/crash-report-modules.m Tue Jun 12 04:51:22 2018
>> @@ -1,20 +1,14 @@
>>  // RUN: rm -rf %t
>>  // RUN: mkdir -p %t/i %t/m %t
>>
>> -// RUN: not env FORCE_CLANG_DIAGNOSTICS_CRASH= TMPDIR=%t TEMP=%t TMP=%t \
>> -// RUN: %clang -fsyntax-only %s -I %S/Inputs/module -isysroot %/t/i/    \
>> +// RUN: env FORCE_CLANG_DIAGNOSTICS_CRASH= TMPDIR=%t TEMP=%t TMP=%t
>> \
>> +// RUN: not %clang -fsyntax-only %s -I %S/Inputs/module -isysroot %/t/i/
>> \
>>  // RUN: -fmodules -fmodules-cache-path=%t/m/ -DFOO=BAR 2>&1 | FileCheck
>> %s
>>
>>  // RUN: FileCheck --check-prefix=CHECKSRC %s -input-file
>> %t/crash-report-*.m
>>  // RUN: FileCheck --check-prefix=CHECKSH %s -input-file
>> %t/crash-report-*.sh
>>  // REQUIRES: crash-recovery
>>
>> -// because of the glob (*.m, *.sh)
>> -// REQUIRES: shell
>> -
>> -// FIXME: This XFAIL is cargo-culted from crash-report.c. Do we need it?
>> -// XFAIL: mingw32
>> -
>>  @import simple;
>>  const int x = MODULE_MACRO;
>>
>> @@ -34,4 +28,4 @@ const int x = MODULE_MACRO;
>>  // CHECKSH: "-D" "FOO=BAR"
>>  // CHECKSH-NOT: "-fmodules-cache-path="
>>  // CHECKSH: "crash-report-modules-{{[^ ]*}}.m"
>> -// CHECKSH: "-ivfsoverlay" "crash-report-modules-{{[^
>> ]*}}.cache/vfs/vfs.yaml"
>> +// CHECKSH: "-ivfsoverlay" "crash-report-modules-{{[^
>> ]*}}.cache{{(/|\\\\)}}vfs{{(/|\\\\)}}vfs.yaml"
>>
>> Modified: cfe/trunk/test/Driver/crash-report-spaces.c
>> URL:
>> http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/crash-report-spaces.c?rev=334494&r1=334493&r2=334494&view=diff
>>
>> ==============================================================================
>> --- cfe/trunk/test/Driver/crash-report-spaces.c (original)
>> +++ cfe/trunk/test/Driver/crash-report-spaces.c Tue Jun 12 04:51:22 2018
>> @@ -1,14 +1,11 @@
>>  // RUN: rm -rf "%t"
>>  // RUN: mkdir "%t"
>>  // RUN: cp "%s" "%t/crash report spaces.c"
>> -// RUN: not env TMPDIR="%t" TEMP="%t" TMP="%t" RC_DEBUG_OPTIONS=1 %clang
>> -fsyntax-only "%t/crash report spaces.c" 2>&1 | FileCheck "%s"
>> +// RUN: env TMPDIR="%t" TEMP="%t" TMP="%t" RC_DEBUG_OPTIONS=1 not %clang
>> -fsyntax-only "%t/crash report spaces.c" 2>&1 | FileCheck "%s"
>>  // RUN: cat "%t/crash report spaces"-*.c | FileCheck
>> --check-prefix=CHECKSRC "%s"
>>  // RUN: cat "%t/crash report spaces"-*.sh | FileCheck
>> --check-prefix=CHECKSH "%s"
>>  // REQUIRES: crash-recovery
>>
>> -// because of the glob (*.c, *.sh)
>> -// REQUIRES: shell
>> -
>>  #pragma clang __debug parser_crash
>>  // CHECK: Preprocessed source(s) and associated run script(s) are
>> located at:
>>  // CHECK-NEXT: note: diagnostic msg: {{.*}}.c
>>
>> Modified: cfe/trunk/test/Driver/crash-report.c
>> URL:
>> http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Driver/crash-report.c?rev=334494&r1=334493&r2=334494&view=diff
>>
>> ==============================================================================
>> --- cfe/trunk/test/Driver/crash-report.c (original)
>> +++ cfe/trunk/test/Driver/crash-report.c Tue Jun 12 04:51:22 2018
>> @@ -1,8 +1,8 @@
>>  // RUN: rm -rf %t
>>  // RUN: mkdir %t
>> -// RUN: not env TMPDIR=%t TEMP=%t TMP=%t RC_DEBUG_OPTIONS=1
>> \
>> +// RUN: env TMPDIR=%t TEMP=%t TMP=%t RC_DEBUG_OPTIONS=1
>> \
>>  // RUN:  CC_PRINT_HEADERS=1 CC_LOG_DIAGNOSTICS=1
>>  \
>> -// RUN:  %clang -fsyntax-only %s
>>  \
>> +// RUN:  not %clang -fsyntax-only %s
>>  \
>>  // RUN:  -F/tmp/ -I /tmp/ -idirafter /tmp/ -iquote /tmp/ -isystem /tmp/
>> \
>>  // RUN:  -iprefix /the/prefix -iwithprefix /tmp -iwithprefixbefore /tmp/
>> \
>>  // RUN:  -Xclang -internal-isystem -Xclang /tmp/
>>  \
>> @@ -13,9 +13,6 @@
>>  // RUN: cat %t/crash-report-*.sh | FileCheck --check-prefix=CHECKSH %s
>>  // REQUIRES: crash-recovery
>>
>> -// because of the glob (*.c, *.sh)
>> -// REQUIRES: shell
>> -
>>  #pragma clang __debug parser_crash
>>  // CHECK: Preprocessed source(s) and associated run script(s) are
>> located at:
>>  // CHECK-NEXT: note: diagnostic msg: {{.*}}crash-report-{{.*}}.c
>>
>>
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180612/f8083159/attachment.html>