Skip Menu |

This queue is for tickets about the AnyMQ CPAN distribution.

Report information
The Basics
Id: 79126
Status: open
Priority: 0/
Queue: AnyMQ

People
Owner: Nobody in particular
Requestors: gavin [...] openfusion.com.au
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 0.34
Fixed in: (no value)



Subject: Tests fail with AnyEvent >= 7.0
AnyMQ tests are failing with AnyEvent >= 7.0 - looks like 'after' has become a required parameter to AnyEvent->timer where it wasn't before? Only seems to affect the timer call in AnyMQ::Topic. Trivial patch attached that fixes it here. Cheers, Gavin
Subject: perl-AnyMQ-0.34-topic_timer_after.patch
From e24ce218f62b255f25c7db1d197ad1a5496300e0 Mon Sep 17 00:00:00 2001 From: Gavin Carr <gavin@openfusion.com.au> Date: Wed, 22 Aug 2012 11:36:08 +1000 Subject: [PATCH] Update AnyMQ::Topic::install_reaper timer call to include after param. --- lib/AnyMQ/Topic.pm | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/lib/AnyMQ/Topic.pm b/lib/AnyMQ/Topic.pm index 3dd8ab2..c931e11 100644 --- a/lib/AnyMQ/Topic.pm +++ b/lib/AnyMQ/Topic.pm @@ -38,7 +38,8 @@ sub install_reaper { my $self = shift; $self->_listener_reaper( - AnyEvent->timer(interval => $self->reaper_interval, + AnyEvent->timer(after => $self->reaper_interval, + interval => $self->reaper_interval, cb => sub { $self->reap_destroyed_listeners }) ); } -- 1.7.1
Subject: Re: [rt.cpan.org #79126] Tests fail with AnyEvent >= 7.0
Date: Wed, 22 Aug 2012 20:34:16 +0800
To: bug-AnyMQ [...] rt.cpan.org
From: Chia-liang Kao <clkao [...] clkao.org>
Show quoted text
> AnyMQ tests are failing with AnyEvent >= 7.0 - looks like 'after' has > become a required parameter to AnyEvent->timer where it wasn't before? > > Only seems to affect the timer call in AnyMQ::Topic.
weird. that shouldn't be the case. since AnyEvent synopsis does have calls to timer without 'after'...
Subject: Re: [rt.cpan.org #79126] Tests fail with AnyEvent >= 7.0
Date: Thu, 23 Aug 2012 06:35:00 +1000
To: "clkao [...] clkao.org via RT" <bug-AnyMQ [...] rt.cpan.org>
From: Gavin Carr <gavin [...] openfusion.com.au>
On Wed, Aug 22, 2012 at 08:34:32AM -0400, clkao@clkao.org via RT wrote: Show quoted text
><URL: https://rt.cpan.org/Ticket/Display.html?id=79126 > >
>> AnyMQ tests are failing with AnyEvent >= 7.0 - looks like 'after' has >> become a required parameter to AnyEvent->timer where it wasn't before?
Show quoted text
>> Only seems to affect the timer call in AnyMQ::Topic.
> >weird. that shouldn't be the case. since AnyEvent synopsis does have >calls to timer without 'after'... >
Not in version 7.0+ I think?
Are you still seeing this issue? this travis-ci run with AE 7.04 seems fine: https://travis-ci.org/clkao/AnyMQ/jobs/4834793