[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