Bug #50109 for Audio-MPD: Problem with newer Class::Accessor

Tue Sep 29 16:41:02 2009 gregoa [...] debian.org - Ticket created

Subject:	Problem with newer Class::Accessor
Date:	Tue, 29 Sep 2009 22:40:26 +0200
To:	bug-Audio-MPD [...] rt.cpan.org
From:	gregor herrmann <gregoa [...] debian.org>

Hi, this is Debian bug http://bugs.debian.org/548305, the problem can also be seen on the CPAN Testers Reports page at http://www.cpantesters.org/distro/A/Audio-MPD.html#Audio-MPD-0.19.6 Audio-MPD doesn't work with Class::Accessor >= 0.34. The problem is that Class::Accessor added an import method, which Audio::MPD is now inheriting instead of the one from Exporter. A simple fix is to change the order: --- a/lib/Audio/MPD.pm +++ b/lib/Audio/MPD.pm @@ -22,7 +22,7 @@ use Readonly; -use base qw[ Class::Accessor::Fast Exporter ]; +use base qw[ Exporter Class::Accessor::Fast ]; __PACKAGE__->mk_accessors( qw[ _conntype _host _password _port _socket collection playlist version ] ); Cheers, gregor, Debian Perl Group -- .''`. http://info.comodo.priv.at/ -- GPG Key IDs: 0x00F3CFE4, 0x8649AA06 : :' : Debian GNU/Linux user, admin, & developer - http://www.debian.org/ `. `' Member of VIBE!AT, SPI Inc., fellow of FSFE | http://got.to/quote/ `- NP: Paul Simon: Train In The Distance

Download signature.asc
application/pgp-signature 198b

Message body not shown because it is not plain text.

Wed Sep 30 11:00:34 2009 jquelin [...] cpan.org - Taken

Wed Sep 30 11:01:45 2009 jquelin [...] cpan.org - Correspondence added

indeed, i already spotted it for poe::component::client::mpd but not for audio::mpd. audio::mpd 0.19.7 fixes the problem, on its way to cpan... thanks for the report!

Wed Sep 30 11:01:46 2009 The RT System itself - Status changed from 'new' to 'open'

Wed Sep 30 11:01:47 2009 jquelin [...] cpan.org - Status changed from 'open' to 'resolved'

Wed Sep 30 13:42:29 2009 gregoa [...] debian.org - Correspondence added

Subject:	Re: [rt.cpan.org #50109] Problem with newer Class::Accessor
Date:	Wed, 30 Sep 2009 19:42:09 +0200
To:	Jerome Quelin via RT <bug-Audio-MPD [...] rt.cpan.org>
From:	gregor herrmann <gregoa [...] debian.org>

On Wed, 30 Sep 2009 11:01:46 -0400, Jerome Quelin via RT wrote: Show quoted text

> audio::mpd 0.19.7 fixes the problem, on its way to cpan... > thanks for the report!

Thanks to you for the quick fix! Cheers, gregor, hoping that this mail won't reopen the bug ... -- .''`. http://info.comodo.priv.at/ -- GPG Key IDs: 0x00F3CFE4, 0x8649AA06 : :' : Debian GNU/Linux user, admin, & developer - http://www.debian.org/ `. `' Member of VIBE!AT, SPI Inc., fellow of FSFE | http://got.to/quote/ `- NP: Beach Boys: Sloop John B

Download signature.asc
application/pgp-signature 198b

Message body not shown because it is not plain text.

Wed Sep 30 13:42:30 2009 The RT System itself - Status changed from 'resolved' to 'open'

Wed Sep 30 13:45:52 2009 jquelin [...] cpan.org - Status changed from 'open' to 'resolved'