<div dir="ltr">Just to resurrect this thread I gave this patch a run on Linux (required a small fix to work with ToT) and it applied fine there and all currently-passing unit tests still pass.<br></div><div class="gmail_extra">

<br><br><div class="gmail_quote">On Mon, Mar 10, 2014 at 7:49 PM, Ed Maste <span dir="ltr"><<a href="mailto:emaste@freebsd.org" target="_blank">emaste@freebsd.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
<br>
================<br>
Comment at: scripts/Python/buildSwigPython.py:1<br>
@@ +1,2 @@<br>
+""" Python SWIG wrapper creation script Windows/LINUX/OSX platform<br>
+<br>
----------------<br>
Missing FreeBSD here; probably best to just not list platforms at all.<br>
(Same comment applies to several files)<br>
<br>
================<br>
Comment at: scripts/Python/buildSwigPython.py:9<br>
@@ +8,3 @@<br>
+<br>
+       Environment:    OS:                     Windows Vista or newer, LINUX, OSX.<br>
+                                       IDE:        Visual Studio 2013 Plugin Python Tools (PTVS)<br>
----------------<br>
Same comment as above, and "IDE" on the next line seems odd -- presumably PTVS isn't needed on non-Windows platforms :)<br>
<br>
(Same comment applies to several files)<br>
<br>
================<br>
Comment at: scripts/Python/finish-swig-Python-LLDB.sh:159<br>
@@ -158,3 +158,3 @@<br>
         ln -s "../../../LLDB" _lldb.so<br>
-    else<br>
+       else<br>
         ln -s "../../../liblldb${SOEXT}" _lldb.so<br>
----------------<br>
accidental whitespace change?<br>
<br>
================<br>
Comment at: scripts/Python/finishSwigPythonLLDB.py:38<br>
@@ +37,3 @@<br>
+<br>
+       Copyright:              None.<br>
+       --------------------------------------------------------------------------<br>
----------------<br>
Recommend putting in the standard copyright & license notice instead<br>
<br>
================<br>
Comment at: scripts/utilsDebug.py:134<br>
@@ +133,1 @@<br>
+<br>
\ No newline at end of file<br>
<br>
----------------<br>
whitespace issue<br>
<br>
================<br>
Comment at: source/Interpreter/ScriptInterpreterPython.cpp:704<br>
@@ -701,1 +703,3 @@<br>
+namespace lldb_private<br>
 {<br>
+    class IOHandlerPythonInterpreter :<br>
----------------<br>
it's too bad that phabricator is confused by the indentation; is the only change the addition of the namespace?<br>
<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D2980" target="_blank">http://llvm-reviews.chandlerc.com/D2980</a><br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
lldb-commits mailing list<br>
<a href="mailto:lldb-commits@cs.uiuc.edu">lldb-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
</div></div></blockquote></div><br></div>