On Tue Feb 26 19:02:04 2008, [EMAIL PROTECTED] wrote:
> Patch applied tonight in r26082.
There's a bit of mop-up work needed as pointed out by chromatic on #parrot.
1. Should 'lib/Test' be removed from the:
no_index:
directory:
element in META.yml. I felt it should, because that directory has
ceased to be part of the Parrot distribution. So I patched META.yml in
r26083.
2. The versions of Test::(Simple|More|Builder) previously contained in
lib/Test/ were, for all practical purposes, from the v0.60 CPAN
distribution. By deleting them, Parrot::Test defaulted to using
whatever version of Test::Builder was default on my various boxes. In
both cases, this was 0.72. Between 0.60 and 0.70, the output from
failed tests changed. When I prepared my patch, I changed the expected
output in 6 tests contained in t/perl/Parrot_Test.t to match the output
I was actually getting.
chromatic noted that this implies that we should mark a minimum version
of Test::Builder needed to get the output expected in Parrot_Test.t.
Examination of the Test-Simple distribution's Changes file
(http://search.cpan.org/src/MSCHWERN/Test-Simple-0.76_02/Changes)
suggests that this change occurred in 0.64_01 Mon Sep 4 04:40:42 EDT
2006. Since I would not want to specify a 4-decimal-places version
number as the minimum required, that would suggest going to the next
higher 2-decimal-places version number that was a meaningful change.
That would be 0.66.
Should that be the minimum version? Or, perhaps we should say, 0.72
which (I think) was the version that came with 5.10?
kid51
