Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Net-Twitter CPAN distribution.

Report information
The Basics
Id: 86092
Status: resolved
Priority: 0/
Queue: Net-Twitter
People
Owner: Nobody in particular
Requestors: gregoa [...] debian.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)
Attachments
signature.asc

History Show all quoted text
  Wed Jun 12 11:09:01 2013 gregoa [...] cpan.org - Ticket created
From: gregoa [...] cpan.org
Subject: libnet-twitter-perl: RateLimit can no longer be used with API::RESTv1_1
This bug has been forwarded from http://bugs.debian.org/712008 -->-->-->-->-->-->-->-->-->-->-->-->-->-->-->-->-->-->-->--> Package: libnet-twitter-perl Version: 4.00004-1 Severity: normal Tags: upstream If I use RateLimit together with API::RESTv1_1 as documented in the Net::Twitter::Role::RateLimit(3pm) man page, I get the following error: Conflict detected: Role Net::Twitter::Role::RateLimit excludes role 'Net::Twitter::Role::API::RESTv1_1' at /usr/lib/perl5/Moose/Meta/Role/Application/RoleSummation.pm line 88 But /usr/share/perl5/Net/Twitter/Role/RateLimit.pm says: # Rate limiting changed so dramatically with v1.1 this Role simply won't work with it excludes 'Net::Twitter::Role::API::RESTv1_1'; So, it seems to be a documentation bug, rather confusing for the user. -- System Information: Debian Release: jessie/sid APT prefers unstable APT policy: (500, 'unstable'), (500, 'testing'), (500, 'stable'), (1, 'experimental') Architecture: amd64 (x86_64) Kernel: Linux 3.9-1-amd64 (SMP w/2 CPU cores) Locale: LANG=POSIX, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8) Shell: /bin/sh linked to /bin/dash Versions of packages libnet-twitter-perl depends on: ii libcarp-clan-perl 6.04-1 ii libcrypt-ssleay-perl 0.58-1 ii libdata-visitor-perl 0.28-1 ii libdatetime-format-strptime-perl 1.5400-1 ii libdatetime-perl 2:1.03-1 ii libdevel-stacktrace-perl 1.3000-1 ii libdigest-hmac-perl 1.03+dfsg-1 ii libhtml-parser-perl 3.71-1 ii libjson-perl 2.58-1 ii libmoose-perl 2.0801-1 ii libmoosex-aliases-perl 0.10-1 ii libmoosex-role-parameterized-perl 1.00-1 ii libnamespace-autoclean-perl 0.13-1 ii libnet-http-perl 6.06-1 ii libnet-oauth-perl 0.28-1 ii libtry-tiny-perl 0.12-1 ii liburi-perl 1.60-1 ii libwww-perl 6.04-1 ii perl 5.14.2-21 libnet-twitter-perl recommends no packages. libnet-twitter-perl suggests no packages. -- no debconf information <--<--<--<--<--<--<--<--<--<--<--<--<--<--<--<--<--<--<--<-- Thanks in advance, gregor herrmann, Debian Perl Group
  Wed Jun 12 17:35:42 2013 MMIMS [...] cpan.org - Correspondence added
On Wed Jun 12 11:09:01 2013, GREGOA wrote: Show quoted text
> > If I use RateLimit together with API::RESTv1_1 as documented in the > Net::Twitter::Role::RateLimit(3pm) man page, I get the following > error: > > Conflict detected: Role Net::Twitter::Role::RateLimit excludes role > 'Net::Twitter::Role::API::RESTv1_1' at > /usr/lib/perl5/Moose/Meta/Role/Application/RoleSummation.pm line 88 > > But /usr/share/perl5/Net/Twitter/Role/RateLimit.pm says: > > # Rate limiting changed so dramatically with v1.1 this Role simply > won't work with it > excludes 'Net::Twitter::Role::API::RESTv1_1'; > > So, it seems to be a documentation bug, rather confusing for the user.
The RateLimit role is incompatible with Net::Twitter using role API::RESTv1_1. It is still perfectly compatible using API::REST (the v1 API). Although Twitter no longer supports API v1, but other services, like http://identi.ca may. I may provide an API v1.1 replacement for the RateLimit role, however, it will likely have a different API since the underlying rate limiting strategy has changed significantly. I'd be happy to accept a documentation patch that makes this clearer, as long as it preserves API v1 for users who mays still rely on it.
  Wed Jun 12 17:35:43 2013 The RT System itself - Status changed from 'new' to 'open'
  Thu Jun 13 03:45:09 2013 gregoa [...] debian.org - Correspondence added
Subject: Re: [rt.cpan.org #86092] libnet-twitter-perl: RateLimit can no longer be used with API::RESTv1_1
Date: Thu, 13 Jun 2013 09:44:50 +0200
To: Marc Mims via RT <bug-Net-Twitter [...] rt.cpan.org>
From: gregor herrmann <gregoa [...] debian.org>
On Wed, 12 Jun 2013 17:35:43 -0400, Marc Mims via RT wrote: Show quoted text
> > If I use RateLimit together with API::RESTv1_1 as documented in the > > Net::Twitter::Role::RateLimit(3pm) man page, I get the following > > error:
Show quoted text
> > So, it seems to be a documentation bug, rather confusing for the user.
> > The RateLimit role is incompatible with Net::Twitter using role > API::RESTv1_1. It is still perfectly compatible using API::REST > (the v1 API). Although Twitter no longer supports API v1, but other > services, like http://identi.ca may.
(BTW: identi.ca is being moved to pump.io ...) Show quoted text
> I'd be happy to accept a documentation patch that makes this > clearer, as long as it preserves API v1 for users who mays still > rely on it.
I guess the original submitter's confusion came from the example in the synopsis, which mentions the v1_1 APi which then doesn't work. So something like (pseudo-patch, I don't have the code here) my $nt = Net::Twitter->new( - traits => [qw/API::RESTv1_1 RateLimit/], + traits => [qw/API::REST RateLimit/], # does not work with API::RESTv1_1 %other_options, ); in the Net::Twitter::Role::RateLimit POD is probably enough. Cheers, gregor -- .''`. Homepage: http://info.comodo.priv.at/ - OpenPGP key 0xBB3A68018649AA06 : :' : Debian GNU/Linux user, admin, and developer - http://www.debian.org/ `. `' Member of VIBE!AT & SPI, fellow of the Free Software Foundation Europe `- BOFH excuse #288: Hard drive sleeping. Let it wake up on it's own...
  Thu Jun 13 13:36:05 2013 MMIMS [...] cpan.org - Correspondence added
On Thu Jun 13 03:45:09 2013, gregoa@debian.org wrote: Show quoted text
> I guess the original submitter's confusion came from the example in > the synopsis, which mentions the v1_1 APi which then doesn't work. > > So something like (pseudo-patch, I don't have the code here) > > my $nt = Net::Twitter->new( > - traits => [qw/API::RESTv1_1 RateLimit/], > + traits => [qw/API::REST RateLimit/], # does not work with API::RESTv1_1 > %other_options, > ); > > in the Net::Twitter::Role::RateLimit POD is probably enough.
Done. http://git.io/5ZQqyQ It'll go out in the next release. Thank you!
  Thu Jun 13 15:53:03 2013 gregoa [...] debian.org - Correspondence added
Subject: Re: [rt.cpan.org #86092] libnet-twitter-perl: RateLimit can no longer be used with API::RESTv1_1
Date: Thu, 13 Jun 2013 21:52:40 +0200
To: Marc Mims via RT <bug-Net-Twitter [...] rt.cpan.org>
From: gregor herrmann <gregoa [...] debian.org>
On Thu, 13 Jun 2013 13:36:06 -0400, Marc Mims via RT wrote: Show quoted text
> > I guess the original submitter's confusion came from the example in > > the synopsis, which mentions the v1_1 APi which then doesn't work. > > > > So something like (pseudo-patch, I don't have the code here) > > > > my $nt = Net::Twitter->new( > > - traits => [qw/API::RESTv1_1 RateLimit/], > > + traits => [qw/API::REST RateLimit/], # does not work with API::RESTv1_1 > > %other_options, > > ); > > > > in the Net::Twitter::Role::RateLimit POD is probably enough.
> > Done. http://git.io/5ZQqyQ > > It'll go out in the next release. Thank you!
Excellent, thanks! Cheers, gregor -- .''`. Homepage: http://info.comodo.priv.at/ - OpenPGP key 0xBB3A68018649AA06 : :' : Debian GNU/Linux user, admin, and developer - http://www.debian.org/ `. `' Member of VIBE!AT & SPI, fellow of the Free Software Foundation Europe `- NP: Joan Baez: No Woman No Cry
Download signature.asc
application/pgp-signature 836b

Message body not shown because it is not plain text.

  Tue Nov 19 19:30:48 2013 MMIMS [...] cpan.org - Status changed from 'open' to 'resolved'