<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, May 14, 2013 at 4:59 PM, Michael Sartain <span dir="ltr"><<a href="mailto:mikesart@valvesoftware.com" target="_blank">mikesart@valvesoftware.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="im"><div class="gmail_extra"><div class="gmail_quote">On Tue, May 14, 2013 at 4:42 PM, Michael Sartain <span dir="ltr"><<a href="mailto:mikesart@valvesoftware.com" target="_blank">mikesart@valvesoftware.com</a>></span> wrote:<br>



<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div></div><div>I haven't been able to run the python test script mentioned in the bug report yet, but "platform process list" in lldb looks correct on my Ubuntu 12.04 64-bit machine.<br>




</div></div></div></blockquote></div><br></div></div><div class="gmail_extra">We got "./dotest.py -C clang -v -t -f PlatformCommandTestCase.test_process_list" to run and it all appears to pass now except this line in TestPlatformCommand.py:<br>



<br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">    @expectedFailureLinux # due to bugzilla 14806 -- "platform status" prints more information on Mac OS X than on Linux<br>


    def test_status(self):<br>        self.expect("platform status",<br>            substrs = ['Platform', 'Triple', 'OS Version', 'Kernel', 'Hostname'])<br></blockquote>


<br></div><div class="gmail_extra">platform status returns this on my machine:<br><br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">(lldb) platform status<br>


Linux 3.5.0-28-generic #48~precise1-Ubuntu SMP Wed Apr 24 21:42:24 UTC 2013<br></blockquote><br></div><div class="gmail_extra">Not sure how this test should be modified to expect something along these lines and also handle all the other Linux distros? I'll leave this to someone else... :)<br>


</div></div>
</blockquote></div><br></div><div class="gmail_extra">I see - this looks like it has a separate bug filed for it:<br><br><a href="http://llvm.org/bugs/show_bug.cgi?id=14806">http://llvm.org/bugs/show_bug.cgi?id=14806</a><br>

<br></div><div class="gmail_extra">PlatformLinux::GetStatus() in PlatformLinux.cpp just needs a little bit of love. We'll take a look at that fixing that...<br></div></div>