Wireshark · Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 23095: /trunk/test/ /trunk/test/: suite-capture.sh

[Richard van der Hoff <richardv@xxxxxxxxxxxxx>]

Ulf Lamping wrote:
> morriss@xxxxxxxxxxxxx schrieb:
> > http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=23095
> >
> > User: morriss
> > Date: 2007/10/08 05:56 PM
> >
> > Log:
> >  Temporarily disable 'Capture 10 packets using stdout' test for tshark until we can get that working
> >   
> What's the motivation in doing so?!?

I think this was in response to the recent "buildbot tests failing" thread.

> tshark is currently broken - my fault - but pretending all is well while 
> we know that it's boken is a *very bad way* of handling things.

I'd agree with this wholeheartedly.

+1 for backing out this change and let the tests show the breakage.

It would be nice if the tests didn't bail out as soon as there was a 
problem, but that's a minor point.

Regards,

Richard