Bug #21545 for Test-Warn: Directories with spaces cause test failures

Mon Sep 18 06:50:47 2006 link [...] redbrick.dcu.ie - Ticket created

Subject:

Directories with spaces cause test failures

Tests pass in the warning without the filename and the module matches against any filename/line number but the match fails if the file has spaces eg c:\document and setting\me\Something.pm. I think the problem is at line 318. my $cmp = $got_msg =~ /^\Q$exp_msg\E at \S+ line \d+\.?$/;

Wed May 20 23:03:29 2009 frioux [...] gmail.com - Correspondence added

Hi! This bug caused a test to fail (unsurprisingly) on my windows box today at work. Obviously this would be more common in windows as the /home location has a space in it (C:/Documents and Settings.) I am including a small patch that should fix the issue. Please consider accepting the patch. Thanks so much! -fREW

--- Warn.pm 2009-05-20 21:58:52.000000000 -0500 +++ NewWarn.pm 2009-05-20 21:58:47.000000000 -0500 @@ -315,7 +315,7 @@ my ($got_kind, $got_msg) = %{ shift() }; my ($exp_kind, $exp_msg) = %{ shift() }; return 0 if ($got_kind eq 'warn') && ($exp_kind eq 'carped'); - my $cmp = $got_msg =~ /^\Q$exp_msg\E at (.+) line \d+\.?$/; + my $cmp = $got_msg =~ /^\Q$exp_msg\E at \S+ line \d+\.?$/; return $cmp; }

Wed May 20 23:03:30 2009 The RT System itself - Status changed from 'new' to 'open'

Sun Jun 14 15:48:42 2009 CHORNY [...] cpan.org - Correspondence added

fixed in 11_01 -- Alexandr Ciornii, http://chorny.net

Sun Jun 14 16:44:13 2009 CHORNY [...] cpan.org - Status changed from 'open' to 'resolved'

Sun Jun 14 16:44:13 2009 CHORNY [...] cpan.org - Fixed in 0.11_01 added