Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Catalyst-Authentication-Credential-Twitter CPAN distribution.

Report information
The Basics
Id: 76678
Status: resolved
Priority: 0/
Queue: Catalyst-Authentication-Credential-Twitter
People
Owner: Nobody in particular
Requestors: 'spro^^*%*^6ut# [...] &$%*c
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Wed Apr 18 18:59:06 2012 $_ = 'spro^^*%*^6ut# [...] &$%*c>#!^!#&!pan.org'; y/a-z.@//cd; print - Ticket created
Subject: Missing dependency on Catalyst::Plugin::Session::Store::FastMmap

Message body is not shown because it is too large.

  Tue Mar 11 20:20:40 2014 yanick+cpan [...] babyl.dyndns.org - Status changed from 'new' to 'resolved'
  Thu May 04 18:57:18 2017 ether [...] cpan.org - Correspondence added
I believe this bug is invalid and the change made to fix it should not have been done -- from looking at the tests, Catalyst::Plugin::Session::Store::FastMmap is intended to be an optional dependency and should not be required for tests to run. Its use is wrapped in an eval {}. Please could this dependency be removed? It is being dragged into production installations that do not need it. thank you!
  Thu May 04 18:57:28 2017 ether [...] cpan.org - Status changed from 'resolved' to 'open'
  Thu May 04 20:09:22 2017 $_ = 'spro^^*%*^6ut# [...] &$%*c>#!^!#&!pan.org'; y/a-z.@//cd; print - Correspondence added
On Thu May 04 18:57:18 2017, ETHER wrote: Show quoted text
> I believe this bug is invalid and the change made to fix it should not > have been done -- from looking at the tests, > Catalyst::Plugin::Session::Store::FastMmap is intended to be an > optional dependency and should not be required for tests to run. Its > use is wrapped in an eval {}. > > Please could this dependency be removed? It is being dragged into > production installations that do not need it. thank you!
Maybe the tests have since changed, but the tests were failing for me at the time I filed this report.
  Fri May 05 20:12:35 2017 yanick+cpan [...] babyl.dyndns.org - Correspondence added
On Thu May 04 18:57:18 2017, ETHER wrote: Show quoted text
> I believe this bug is invalid and the change made to fix it should not > have been done -- from looking at the tests, > Catalyst::Plugin::Session::Store::FastMmap is intended to be an > optional dependency and should not be required for tests to run. Its > use is wrapped in an eval {}. > > Please could this dependency be removed? It is being dragged into > production installations that do not need it. thank you!
I'm moving the dependency to TestRecommends, mostly because without it all tests are just useless passthroughs.
  Fri May 05 20:14:21 2017 yanick+cpan [...] babyl.dyndns.org - Status changed from 'open' to 'resolved'