[Lldb-commits] [PATCH] Skip lldb-mi tests on FreeBSD for now

[Zachary Turner]

Fwiw I don't feel strongly either way. Right now it seems there's issues
with lldb-mi such that even disabling them individually now, a new test
that gets added might have the same issue, because the issue is not in the
implementation of a particular test or feature, but of the larger lldb-mi
library. So this would still lead to the possibility that any future test
may also have to be disabled, creating more work than necessary.

Of course, the correct way to deal with this is to prioritize finding and
fixing the underlying race conditions. Presumably they affect not just
FreeBSD.
On Wed, Feb 18, 2015 at 10:05 AM Ilia K <ki.stfu at gmail.com> wrote:

> In http://reviews.llvm.org/D7727#125642, @emaste wrote:
>
> > In http://reviews.llvm.org/D7727#125628, @ki.stfu wrote:
> >
> > > If it works unstable on FreeBSD you should skip it and later we can
> re-enable test on FreeBSD when it will be fixed. I don't see a reason to
> modify dotest.py file.
> >
> >
> > I think it's better to make this change here than add 43 decorators
> across 12 lldb-mi test files
>
>
> It's 'sad' that lldb-mi has grown to such an extent, but the designed @
> skipIfFreeBSD is better than a hack in dotest.py.
>
>
> http://reviews.llvm.org/D7727
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20150218/1c244195/attachment.html>