[Lldb-commits] [review reqest] zorg change to detect lldb test results

[Malea, Daniel]

Thanks all, committed in r176028.

Dan



On 2013-02-22, at 3:35 PM, "Malea, Daniel" <daniel.malea at intel.com<mailto:daniel.malea at intel.com>> wrote:

Here is the patch… it's a small one :)

<zorg_detect_lldb_test_results.patch>


On 2013-02-21, at 11:48 PM, David Blaikie <dblaikie at gmail.com<mailto:dblaikie at gmail.com>> wrote:




On Thu, Feb 21, 2013 at 7:09 AM, Dmitri Gribenko <gribozavr at gmail.com<mailto:gribozavr at gmail.com>> wrote:
On Wed, Feb 20, 2013 at 7:06 PM, Malea, Daniel <daniel.malea at intel.com<mailto:daniel.malea at intel.com>> wrote:
> Hi all,
>
> I recently made the lldb test output (from make) look like clang's for the purpose of detecting results with the same buildbot regex infrastructure that's used on Clang's lit tests.
>
> Could either of you review the attached patch; I'm just changing ShellCommand to LitTestCommand so we can plug into the same regex magic.
>
> Not quite sure this is the right list, but I'm posting here anyways since it's LLDB related.

This looks correct, but I am not an expert here.

David, do you know by chance how this output parsing code works?
Could you review, please?

Excuse the delay - though it seems you didn't forward the patch. The output parsing is fairly simple, though.

_______________________________________________
lldb-commits mailing list
lldb-commits at cs.uiuc.edu<mailto:lldb-commits at cs.uiuc.edu>
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits