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

Subprojects LLDB
1

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@intel.com>
Intel Waterloo
Phone: 519-772-2566

fix_lldb_minus_p.patch (2.03 KB)

2

Apologies for the double-post; this is the same patch with the implemented TODO removed.

fix_lldb_minus_p.patch (2.69 KB)

3

Looks good as far as I can tell.

4

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