[PATCH] D83759: [clangd] Port lit tests to Windows
Kadir Cetinkaya via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jul 21 01:54:02 PDT 2020
kadircet added a comment.
hi, sorry for the late replay :/
We discussed this offline with Sam and are both concerned with the duplications it introduces, haven't really thought about a nice way to address this yet.
As you mentioned the background-index test in itself looks a lot better do, so we can land that one separately and think about the rest in the meantime.
================
Comment at: clang-tools-extra/clangd/test/background-index.test:18
# Test that the index is writing files in the expected location.
# RUN: ls %t/.cache/clangd/index/foo.cpp.*.idx
# RUN: ls %t/sub_dir/.cache/clangd/index/foo.h.*.idx
----------------
are you sure these don't need changes on windows bots ? AFAICT this contains both backslashes(coming from %t) and forwards slashes, and command is `ls` which might not be available on a default windows prompt.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83759/new/
https://reviews.llvm.org/D83759
More information about the cfe-commits
mailing list