[lldb-dev] review request: fix lldb -P on Linux (and simplify running dotest.py)

[Malea, Daniel]

Thanks Greg! Committed in r171558. The instructions on the build page are now more complicated than they need to be; I'll update those shortly.

Cheers,
Dan

-----Original Message-----
From: Greg Clayton [mailto:gclayton at apple.com] 
Sent: Friday, January 04, 2013 6:01 PM
To: Malea, Daniel
Cc: lldb-dev at cs.uiuc.edu
Subject: Re: [lldb-dev] review request: fix lldb -P on Linux (and simplify running dotest.py)

Looks good as far as I can tell.

On Jan 4, 2013, at 2:25 PM, "Malea, Daniel" <daniel.malea at intel.com> wrote:

> Apologies for the double-post; this is the same patch with the implemented TODO removed.
> 
> 
> 
> 
> On 2013-01-04, at 5:22 PM, Malea, Daniel wrote:
> 
> Hi all,
> 
> When someone has a spare moment, could you please take a look at the attached patch? It implements a TODO in Host.cpp and fixes the PYTHONPATH printing feature of lldb (the -P flag). With fix applied, it is no longer necessary to set PYTHONPATH manually or to use the "-i" flag when invoking dotest.py on Linux (and I expect FreeBSD).
> 
> The couple lines added to dotest.py fix LLDB_BUILD_DIR environment variable on Linux, which was causing the TestPublicAPIHeaders testcase to fail.
> 
> 
> 
> 
> Daniel Malea <daniel.malea at intel.com<mailto:daniel.malea at intel.com>>
> Intel Waterloo
> Phone: 519-772-2566
> 
> <fix_lldb_minus_p.patch>
> 
> Daniel Malea <daniel.malea at intel.com<mailto:daniel.malea at intel.com>>
> Intel Waterloo
> Phone: 519-772-2566
> 
> <fix_lldb_minus_p.patch>_______________________________________________
> lldb-dev mailing list
> lldb-dev at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev