On Sun, Feb 19, 2006 at 02:58:10AM -0600, Joshua ben Jore wrote:
> eummu.diff: Using the current bleadperl I was getting the following
> warning during tests. I changed the test to use version->new( ... ) so
> the version object could handle the comparison instead of
> Test::Whatever which actively tries to remove the overloading which
> prevents a sane comparison.
>
> 1..110
> ok 1 - use ExtUtils::MM_Unix;
> ok 2 - There can be only one (or none)
> Argument "1.50_01" isn't numeric in addition (+) at
> lib/Test/Builder.pm line 505.
> ok 3 - Should be at least version 1.12606
> ok 4 - canonpath xx/
Alternatively, it could just test eval { use ExtUtils::MM_Unix 1.12606; 1 }
(including checking for lack of "isn't numeric" warnings.
MM_Unix.t is skipped on cygwin and many other platforms. Should it be?
Parts like:
my $os = ($ExtUtils::MM_Unix::Is_OS2 || 0)
+ ($ExtUtils::MM_Unix::Is_Win32 || 0)
+ ($ExtUtils::MM_Unix::Is_Dos || 0)
+ ($ExtUtils::MM_Unix::Is_VMS || 0);
ok ( $os <= 1, 'There can be only one (or none)');
make it look like it shouldn't be skipped. When forced to run, it passes
all tests.
