Skip Menu |

This queue is for tickets about the Catalyst-Plugin-ErrorCatcher-ActiveMQ-Stomp CPAN distribution.

Report information
The Basics
Id: 97247
Status: open
Priority: 0/
Queue: Catalyst-Plugin-ErrorCatcher-ActiveMQ-Stomp

People
Owner: Nobody in particular
Requestors: DAKKAR [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 0.001004
Fixed in: (no value)



Subject: Data::Serializer::JSON is used, but JSON is not a dependency
t/00-load.t ................. ok Data::Serializer error: Please make sure Data/Serializer/JSON.pm is a properly installed package. Data::Serializer does not require all the libraries needed for its backends; so ::ErrorCatcher::..::Stomp should require JSON (or at least test_require it).
On 2014-07-15 04:57:50, DAKKAR wrote: Show quoted text
> t/00-load.t ................. ok > Data::Serializer error: Please make sure Data/Serializer/JSON.pm is a > properly installed package. > > Data::Serializer does not require all the libraries needed for its > backends; so ::ErrorCatcher::..::Stomp should require JSON (or at > least test_require it).
Tests that use this package should be optional (e.g. use Test::Requires to skip it if the module is not installed); and the prereq can be recommended instead of required.