paulkirth reopened this revision.
paulkirth added inline comments.
This revision is now accepted and ready to land.

================
Comment at: clang-tools-extra/test/clang-doc/single-file-public.cpp:9
+//   for a name with a 40-char USR name.
+// RUN: find %t -regex 
".*[0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z][0-9A-Z].*"
 -exec cat {} \; | FileCheck %s --check-prefix=CHECK
 // RUN: rm -rf %t
----------------
paulkirth wrote:
> 
since this failed, maybe:
```
// RUN: find %t/docs -regex ".*/[0-9A-F]*.yaml" -exec cat {} \; | FileCheck %s
```
That works locally for me on both mac and linux. you lose the nice check for 40 
chars, but at least it functions. if its very important, you can always have a 
single run line that does `ls` and `grep`.

FYI: the BSD `find` accepts the failing regex w/o `-regextype sed`. It's 
unfortunate that GNU `find` doesn't have the same default regex, and doesn't 
accept that version w/o the `-regextype` being set.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138073

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to