On Sat, Jul 05, 2003 at 06:05:42PM +0300, Jarkko Hietaniemi wrote:
> > >       Comment out some debugging output in Math::BigInt, that
> > >       causes confusion in t/TEST when running Math/BigInt/t/bare_mbi.t.
> > 
> > But this really is just a bug in the TEST script. I'd recommend a
> > patch like this. The first chunk is just to make an overlong line
> > shorter, the second chunk is the real fix. All tests pass on my linux.
> > And even Tels' original test script passes all tests.
> 
> Hmmm, I dunno.  In general I would say tests should not produce extra
> output unless they mean to fail.  Or in other words, the test
> harnesses should not be too forgiving.  Or they may output using
> some known "out of band" channel, like lines matching ^/#/.

I think I agree with Jarkko. Hence I ought to be attaching a patch for
harness. But I am bad, because I am not. (I'm not sure where to do it,
and I don't have the time left today to find out)

Nicholas Clark

Reply via email to