[PATCH] D23476: Allow manual specification of the binary directory in check_clang_tidy.py

[Alexander Kornienko via cfe-commits]

The script was never intended to be run manually. Its interface is rather
inconvenient for this. If you need to run a single test, it's easier to do
using llvm-lit (`path/to/lit test/file.cpp`). But if you for some reason
want to run the script directly, it should be possible to just modify PATH
or switch to the bin directory of the build. I don't think this flag is
needed.

On Aug 13, 2016 21:14, "Zachary Turner" <zturner at google.com> wrote:

> I was trying to rerun the lit commands manually to diagnose a failing
> test. This all works fine from lit, but if you want to run
> check_clang_tidy.py from the command line, this is convenient
> On Sat, Aug 13, 2016 at 10:27 AM Alexander Kornienko <alexfh at google.com>
> wrote:
>
>> alexfh added a comment.
>>
>> Why is this change needed? What exactly doesn't work without it? I think,
>> lit should take care of setting paths or changing directories appropriately.
>>
>>
>> https://reviews.llvm.org/D23476
>>
>>
>>
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160814/6eb47df5/attachment-0001.html>