[PATCH] D57533: lit: support long paths on Windows

[Zachary Turner via Phabricator via llvm-commits]

zturner added a comment.

I prefer if things "just work", so I'm not sure I understand where the pushback on this one is coming from.  This isn't just an issue for buildbots, it's also an issue for developers who might have their build trees nested very deeply.  Even on my local machine, I have some directories whose names are quite long because of the way I structure my build folder.

It's already a hassle for people on Windows to make sure they have all the right tools necessary to get things working properly, and then telling them on top of that that they may have to find some nonstandard version of a tool or build something from source just seems like much more of a headache then writing a couple of lines of code ourselves that solve the problem once and for all.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57533