Skip Menu |

This queue is for tickets about the ExtUtils-MakeMaker CPAN distribution.

Report information
The Basics
Id: 46744
Status: resolved
Priority: 0/
Queue: ExtUtils-MakeMaker

People
Owner: Nobody in particular
Requestors: jdhedden [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in:
  • 6.53_01
  • 6.53_02
Fixed in: (no value)



Subject: [PATCH] Cygwin tests failing due to missing ()
The new path tests in MM_Cygwin.t are failing with: Can't locate object method "ok" via package "MM" at ../lib/ExtUtils/t/MM_Cygwin.t line 110. It is cause by a lack of parenthesis with the 'ok' call. The attached patch corrects this.
Subject: cygwin.patch
--- ExtUtils-MakeMaker-6.53_02/t/MM_Cygwin.t.orig 2009-06-08 13:42:50.229135400 -0400 +++ ExtUtils-MakeMaker-6.53_02/t/MM_Cygwin.t 2009-06-08 13:45:12.924751000 -0400 @@ -107,18 +107,18 @@ $comspec = Cygwin::win_to_posix_path($comspec); - ok MM->maybe_command($comspec), qq{'$comspec' should be executable"}; + ok(MM->maybe_command($comspec), qq{'$comspec' should be executable"}); } # 'C:/' should *never* be executable, it's a directory. { my $cdrive = Cygwin::win_to_posix_path("C:/"); - ok !MM->maybe_command($cdrive), qq{'$cdrive' should never be executable}; + ok(!MM->maybe_command($cdrive), qq{'$cdrive' should never be executable}); } # Our copy of Perl (with a unix-path) should always be executable. -ok MM->maybe_command($Config{perlpath}), qq{'$Config{perlpath}' should be executable}; +ok(MM->maybe_command($Config{perlpath}), qq{'$Config{perlpath}' should be executable}); package FakeOut;
Thanks, applied as e6b3236