-----Original Message-----
From: Ed
Sent: Friday, May 08, 2015 5:33 PM
To: sisyph...@optusnet.com.au
Cc: pdl-devel
Subject: Re: [Pdl-devel] [Pdl-general] CHM/PDL-2.007_17.tar.gz
(a.k.a.PDL-2.008 RC4)released to CPAN
> Glad it's all working on Windows!
>
> t/pnm.t used to not use File::Temp, which meant it could not be tested in
> parallel. The noise is unfortunate, but harmless.
Not *exactly* harmless - having 33 empty files being added to my $ENV{TEMP}
directory whenever I run the pnm.t test script is a bit annoying.
Is there something being gained by using File::Temp ?
> Test::Warn checks that certain warnings are emitted, but when I looked
> into this I couldn't find a way to silence these ones.
Yeah, Test::Warn has a challenging user interface. (*I* find it challenging,
at least.)
The docs state that warnings_exist() "will warn() all warnings that do not
match the supplied regex/category".
And it seems to me that the warnings we're seeing are ones that "do not
match the supplied regex/category".
The docs also mention that you should use warnings_exist() only if you
"couldn't care less if other warnings happened".
I think we'll be able to work around this using warnings_like() instead.
> Again, it is a misfortune, but harmless. Please let us not delay 2.008 any
> more for non-breakage?
Yes, I'm not proposing that any of this noise should block the release of
2.008. Plenty of time to look further into these issues (if deemed to be of
sufficient importance) after 2.008 is released.
For mine, the main drawback to having test scripts that issue warnings is
that it can raise doubts in the mind of the user. One might wonder "Well ...
the test suite passed. But, given the warnings that were emitted, perhaps it
was a *false* pass ?"
Whereas a test suite that passes && does not emit any warnings leaves one
feeling confident that everything is working as intended.
There is (or at least once was) a number of perl folk who hold the view that
(owing largely to this "confidence" aspect) a successful test suite should
emit no warnings.
Cheers,
Rob
------------------------------------------------------------------------------
One dashboard for servers and applications across Physical-Virtual-Cloud
Widest out-of-the-box monitoring support with 50+ applications
Performance metrics, stats and reports that give you Actionable Insights
Deep dive visibility with transaction tracing using APM Insight.
http://ad.doubleclick.net/ddm/clk/290420510;117567292;y
_______________________________________________
pdl-devel mailing list
pdl-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/pdl-devel
