Skip Menu |

This queue is for tickets about the MooseX-Storage CPAN distribution.

Report information
The Basics
Id: 62067
Status: resolved
Priority: 0/
Queue: MooseX-Storage

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

Bug Information
Severity: Normal
Broken in: 0.28
Fixed in: (no value)



Subject: MooseX::Storage::Format::JSON should require JSON::Any
Apologies if this is my mistake, but I when I look at the packages it appears that MooseX::Storage and specifically MooseX::Storage::Format::JSON use JSON::Any but do not declare it as a requirement.

I think it should be added as a dependency.

Thank you,

Tom
On Tue Oct 12 03:53:24 2010, TOMELIAZ wrote: Show quoted text
> Apologies if this is my mistake, but I when I look at the packages it > appears > that MooseX::Storage and specifically MooseX::Storage::Format::JSON > use > JSON::Any but do not declare it as a requirement. > > I think it should be added as a dependency. > > Thank you, > > Tom
The Makefile.PL in the current version of MooseX::Storage has this in it: # you should have at least one # serialization format feature 'JSON', -default => 1, 'JSON::Any' => '1.15', 'Test::JSON' => '0.06'; This makes the JSON feature "Optional" and asks at installation time (if your installer is configured to ask) with the default being "Yes". Since using MooseX::Storage doesn't require using JSON (ie YAML backends, etc), I don't know how we could make this any more required without forcing unwanted features on people. -Chris
I agree with Chris; closing.