[Lldb-commits] [PATCH] D15318: Get rid of global variables in dotest.py

Zachary Turner via lldb-commits lldb-commits at lists.llvm.org
Mon Dec 7 21:59:13 PST 2015


To be clear, we explicitly set the LLDB_TEST environment variable as part
of initialization, and it's now trying to read the value of the environment
variable before it's been set.  At least I think anyway, I'm going off of
memory, don't have code in front of me.

On Mon, Dec 7, 2015 at 9:58 PM Zachary Turner <zturner at google.com> wrote:

> zturner added a subscriber: zturner.
> zturner added a comment.
>
> Hmm, yea that code path only gets run on OSX.  I think the problem is that
> I changed the order that happens in.  It now happens right when you import
> `configuration` for the first time, and it used to happen explicitly in
> `main()`.  So maybe just make that function public, and call in the same
> place it used to be called.
>
>
> http://reviews.llvm.org/D15318
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20151208/9b8c2ea1/attachment.html>


More information about the lldb-commits mailing list