+thakis<br><div class="gmail_quote">On Fri, Mar 20, 2015 at 4:28 AM Pavel Labath <<a href="mailto:labath@google.com">labath@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: labath<br>
Date: Fri Mar 20 06:24:36 2015<br>
New Revision: 232805<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=232805&view=rev" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project?rev=232805&view=rev</a><br>
Log:<br>
Fix ninja check-lldb<br>
<br>
Ninja apparently has issues with commands writing nul characters '\0' to stdout. When it<br>
encounters a nul character, the rest of the output is skipped (I will file a bug with ninja about<br>
that). This breaks the linux buildbot since it parses the ninja check-lldb output to get a list<br>
of failures.<br>
<br>
Recently, we have started outputting nul characters in lldb-server tests. This is probably<br>
a bug in itself, but I am not addressing that now. This is just a workaround commit, which<br>
prevents the nul characters from appearing in the output, and gets the buildbot operational again.<br>
<br>
Modified:<br>
    lldb/trunk/test/tools/lldb-<u></u>server/socket_packet_pump.py<br>
<br>
Modified: lldb/trunk/test/tools/lldb-<u></u>server/socket_packet_pump.py<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/lldb/trunk/test/tools/lldb-server/socket_packet_pump.py?rev=232805&r1=232804&r2=232805&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/lldb/trunk/test/tools/<u></u>lldb-server/socket_packet_<u></u>pump.py?rev=232805&r1=232804&<u></u>r2=232805&view=diff</a><br>
==============================<u></u>==============================<u></u>==================<br>
--- lldb/trunk/test/tools/lldb-<u></u>server/socket_packet_pump.py (original)<br>
+++ lldb/trunk/test/tools/lldb-<u></u>server/socket_packet_pump.py Fri Mar 20 06:24:36 2015<br>
@@ -3,6 +3,7 @@ import re<br>
 import select<br>
 import threading<br>
 import traceback<br>
+import codecs<br>
<br>
 def _handle_output_packet_string(<u></u>packet_contents):<br>
     if (not packet_contents) or (len(packet_contents) < 1):<br>
@@ -16,7 +17,7 @@ def _handle_output_packet_string(<u></u>packet_<br>
<br>
 def _dump_queue(the_queue):<br>
     while not the_queue.empty():<br>
-        print the_queue.get(True)<br>
+        print codecs.encode(the_queue.get(<u></u>True), "string_escape")<br>
         print "\n"<br>
<br>
 class SocketPacketPump(object):<br>
<br>
<br>
______________________________<u></u>_________________<br>
lldb-commits mailing list<br>
<a href="mailto:lldb-commits@cs.uiuc.edu" target="_blank">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/<u></u>mailman/listinfo/lldb-commits</a><br>
</blockquote></div>