On Tue, Nov 04, 2008 at 11:19:56AM -0500, David Golden wrote:
> 
> No.  It's actually not perl -V output, it's Config::myconfig().
> 
> However, it's added after a "--\n\n" which makes it look like an email
> signature.  So if something is chomping/suppressing long signatures,
> that might do it.  I would change that in Test::Reporter, but I'm
> afraid it might break Barbie's or others' parsing of the report.

My parsing code looks at the whole body and looks for specific strings
within, eg:

$$body =~ /Summary of my (?:perl\d+)? \((?:revision )?(\d+(?:\.\d+)?)
(?:version|patchlevel) (\d+) subversion\s+(\d+) ?(.*?)\) configuration/s

The double dash is ignored completely.

In fact, I would go so far as to suggest it ought to be a proper
section, as per the Tester Comments, Program Output, etc.

Incidentally, I've just noticed that a recent report of mine, could
potentially have been an issue, as a double dash appears at the top of
the report:

http://www.nntp.perl.org/group/perl.cpan.testers/2008/04/msg1303800.html

I'm not aware of anyone else parsing the reports directly on a regular
basis, so I'm inclined to suggest getting rid of the potentially
troublesome double dashes completely :)

Cheers,
Barbie.
-- 
Birmingham Perl Mongers <http://birmingham.pm.org>
Memoirs Of A Roadie <http://barbie.missbarbell.co.uk>


Reply via email to