[Lldb-commits] [PATCH] add tests for lldb-mi

Zachary Turner zturner at google.com
Mon Nov 24 07:15:12 PST 2014


+1, unless there is a specific reason -- which should be documented -- new
tests should be on everywhere by default
On Mon, Nov 24, 2014 at 7:12 AM Ed Maste <emaste at freebsd.org> wrote:

> On 24 November 2014 at 06:18, Abid, Hafiz <Hafiz_Abid at mentor.com> wrote:
> >
> > 1. I am wondering why have you restricted tests to darwin only? I have
> tried them
> > on Linux and they seem to work. Although 'interrupt' test gave a timeout
> error.
> > I will try to figure out why that test fails on Linux.
>
> I think we're generally too conservative in enabling new functionality
> or tests only on the platform where initially developed. Unless it's
> known not to work (or build), we should just enable new functionality
> everywhere. It's easy enough to change afterwards if we see failures
> on the bots.
> _______________________________________________
> lldb-commits mailing list
> lldb-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20141124/ff538d27/attachment.html>


More information about the lldb-commits mailing list