Skip Menu |

This queue is for tickets about the Text-Todo CPAN distribution.

Report information
The Basics
Id: 101154
Status: resolved
Priority: 0/
Queue: Text-Todo

People
Owner: andrew [...] cpan.org
Requestors: svetoslav.chingov [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: Critical
Broken in: v0.2.0
Fixed in: v0.2.1



Subject: Todo.pm fails on newer versions of Perl
This patch makes Todo.pm work on Perl version 5.18 and newer.
Subject: Todo.patch
--- Todo.pm 2014-12-27 18:53:01.053589408 +0200 +++ TodoNew.pm 2014-12-27 14:25:27.401541671 +0200 @@ -110,7 +110,7 @@ if ( my ( $pre, $post ) = $path_of{$ident}{$type} =~ /^(.*)$1(.*)\.txt$/ixms ) { - foreach my $f qw( todo done report ) { + foreach my $f (qw( todo done report )) { if ( !defined $path_of{$ident}{ $f . '_file' } ) { $path_of{$ident}{ $f . '_file' } = $pre . $f . $post . '.txt';
Subject: Re: [rt.cpan.org #101154] Todo.pm fails on newer versions of Perl
Date: Sat, 27 Dec 2014 20:34:13 -0700
To: "svetoslav.chingov [...] gmail.com via RT" <bug-Text-Todo [...] rt.cpan.org>
From: Andrew Fresh <andrew [...] afresh1.com>
On Sat, Dec 27, 2014 at 12:05:32PM -0500, svetoslav.chingov@gmail.com via RT wrote: Show quoted text
> This patch makes Todo.pm work on Perl version 5.18 and newer.
Thank you for your report. I did have some other changes under way, long ago, so will have to figure out how to merge them and get a new release out. This patch looks good. Show quoted text
> --- Todo.pm 2014-12-27 18:53:01.053589408 +0200 > +++ TodoNew.pm 2014-12-27 14:25:27.401541671 +0200 > @@ -110,7 +110,7 @@ > if ( my ( $pre, $post ) > = $path_of{$ident}{$type} =~ /^(.*)$1(.*)\.txt$/ixms ) > { > - foreach my $f qw( todo done report ) { > + foreach my $f (qw( todo done report )) { > if ( !defined $path_of{$ident}{ $f . '_file' } ) { > $path_of{$ident}{ $f . '_file' } > = $pre . $f . $post . '.txt';
-- andrew - http://afresh1.com A printer consists of three main parts: the case, the jammed paper tray and the blinking red light.
I decided to create a branch and release 0.2.1 to resolve this. Happy to hear someone is using this.