Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the SVN-Notify CPAN distribution.

Report information
The Basics
Id: 35157
Status: resolved
Priority: 0/
Queue: SVN-Notify

People
Owner: Nobody in particular
Requestors: LARRYSH [...] cpan.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 2.70
Fixed in: 2.71



Subject: register_attributes not working in Filters
Consider the following sample filter: package SVN::Notify::Filter::Test; use strict; use warnings; use SVN::Notify; SVN::Notify->register_attributes( testattr => 'testattr=s' ); sub recipients { my ($notify, $to) = @_; $notify->register_attributes( testattr => 'testattr=s' ); print "attr => " . $notify->testattr . "\n"; return $to; } 1; When run from the command line, I get the following: svnnotify --repos-path /home/ubuntu/svnroot/ --revision 3 --to ubuntu --handler HTML::ColorDiff --filter Test --testattr blah Use of uninitialized value in concatenation (.) or string at /usr/local/share/perl/5.8.8/SVN/Notify/Filter/Test.pm line 12. attr =>
Subject: Re: [rt.cpan.org #35157] register_attributes not working in Filters
Date: Thu, 17 Apr 2008 11:33:50 -0700
To: bug-SVN-Notify [...] rt.cpan.org
From: "David E. Wheeler" <david [...] kineticode.com>
On Apr 17, 2008, at 09:55, LARRYSH via RT wrote: Show quoted text
> When run from the command line, I get the following: > > svnnotify --repos-path /home/ubuntu/svnroot/ --revision 3 --to ubuntu > --handler HTML::ColorDiff --filter Test --testattr blah > > Use of uninitialized value in concatenation (.) or string at > /usr/local/share/perl/5.8.8/SVN/Notify/Filter/Test.pm line 12. > attr =>
Bah! Replicated. There's a load-ordering problem. Fixing. Best, David
Subject: Re: [rt.cpan.org #35157] register_attributes not working in Filters
Date: Thu, 17 Apr 2008 11:56:38 -0700
To: bug-SVN-Notify [...] rt.cpan.org
From: "David E. Wheeler" <david [...] kineticode.com>
On Apr 17, 2008, at 11:34, David Wheeler via RT wrote: Show quoted text
> Bah! Replicated. There's a load-ordering problem. Fixing.
2.71 is no its way to CPAN now, along with a few other fixes. Thanks, David
It is fixed now, Thanks!