[Lldb-commits] [PATCH] D19216: test infra cleanup: make test_runner/lib into the test_runner package

Todd Fiala via lldb-commits lldb-commits at lists.llvm.org
Mon Apr 18 11:10:35 PDT 2016


Yep, I agree with that now.  Wasn't the case when I originally wrote it,
but yeah that can move.  I'll adjust that.

On Mon, Apr 18, 2016 at 10:57 AM, Zachary Turner <zturner at google.com> wrote:

> zturner added inline comments.
>
> ================
> Comment at: packages/Python/lldbsuite/test/test_runner/lldb_utils.py:1
> @@ +1,2 @@
> +"""
> +The LLVM Compiler Infrastructure
> ----------------
> I feel like this entire file should go under `lldbsuite/support`.  If it's
> under `test/test_runner` we can't use it from anything in `lldb/scripts`,
> but this is generic enough that there's nothing actually test-specific
> about it.
>
>
> http://reviews.llvm.org/D19216
>
>
>
>


-- 
-Todd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20160418/676e895f/attachment.html>


More information about the lldb-commits mailing list