Bug #20488 for Template-Toolkit: fileline.t fails -- expects wrong slashes on Win32

Sun Jul 16 08:25:17 2006 dagolden [...] cpan.org - Ticket created

Subject:

fileline.t fails -- expects wrong slashes on Win32

FAILED 7: - template text 2 did not match expected MATCH FAILED input: [[% INCLUDE warning -%]\nfile: [% file.chunk(-16).last %]\nline: [% line %]\nwarn: [% warn %]\n] expect: [Hello\nWorld\nfile: test/lib/warning\nline: 2\nwarn: Argument "" isn't numeric in addition (+)] output: [Hello\nWorld\nfile: test\lib\warning\nline: 2\nwarn: Argument "" isn't numeric in addition (+)] not ok 7 - template text 2 did not match expected

Sun Sep 10 10:29:18 2006 smueller [...] cpan.org - Correspondence added

From:

SMUELLER [...] cpan.org

Hi David, hi Andy, the attached patch fixes things on win32 and still works on linux for me. No idea about Mac OS. Applying from within the unpacked distribution directory using the usual "patch -p0 < diff.txt" should work well. Hope this helps, Steffen

--- t/fileline.t 2006-01-30 16:28:55.000000000 +0100 +++ t/fileline.t 2006-09-10 16:21:10.000000000 +0200 @@ -37,7 +37,9 @@ warning => sub { return $warning }, file => sub { $warning =~ /at (.*?) line/; - return $1; + my $file = $1; + $file =~ s!\\!/!g if defined $file; + return $file; }, line => sub { $warning =~ /line (\d*)/;

Sun Sep 10 10:29:20 2006 The RT System itself - Status changed from 'new' to 'open'

Wed Mar 07 08:41:14 2007 CHORNY [...] cpan.org - Broken in 2.18 added

Wed Mar 07 08:41:15 2007 CHORNY [...] cpan.org - Severity Normal changed to Important

Wed Mar 07 08:48:05 2007 CHORNY [...] cpan.org - Correspondence added

2.17 has 'fileline.t' skipped on MSWin32. But 2.18 does not have skipping. I think patch is better. Replace should be done only on MSWin32. -- Alexandr Ciornii, http://chorny.net

Wed Aug 06 07:16:48 2008 cpan [...] wardley.org - Correspondence added

This was resolved in 2.19

Wed Aug 06 07:16:51 2008 cpan [...] wardley.org - Status changed from 'open' to 'resolved'