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:[email protected]] Sent: Friday, January 04, 2013 6:01 PM To: Malea, Daniel Cc: [email protected] 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" <[email protected]> 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 <[email protected]<mailto:[email protected]>> > Intel Waterloo > Phone: 519-772-2566 > > <fix_lldb_minus_p.patch> > > Daniel Malea <[email protected]<mailto:[email protected]>> > Intel Waterloo > Phone: 519-772-2566 > > <fix_lldb_minus_p.patch>_______________________________________________ > lldb-dev mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev _______________________________________________ lldb-dev mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev
