Skip Menu |

This queue is for tickets about the DBIx-Class-InflateColumn-Serializer CPAN distribution.

Report information
The Basics
Id: 104276
Status: open
Priority: 0/
Queue: DBIx-Class-InflateColumn-Serializer
People
Owner: Nobody in particular
Requestors: steve [...] deefs.net
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 0.07
Fixed in: (no value)

History Show all quoted text
  Wed May 06 19:39:20 2015 steve [...] deefs.net - Ticket created
Subject: JSON::MaybeXS should be build_requires, not test_requires
Hi, JSON::MaybeXS a required module as of v0.07, but it's only marked as required for testing in the dist.ini file. If tests are skipped, it doesn't get installed, which leads to crashes when the JSON serializer is subsequently used. It looks like fixing this just involves moving JSON::MaybeXS from [Prereqs / TestRequires] to [Prereqs] in the dist.ini file. Hope this helps, Steve Simms
  Wed May 06 20:10:31 2015 ether [...] cpan.org - Correspondence added
Subject: JSON::MaybeXS should be runtime_requires, not test_requires
On 2015-05-06 16:39:20, SSIMMS wrote: Show quoted text
> Hi, > > JSON::MaybeXS a required module as of v0.07, but it's only marked as > required for testing in the dist.ini file. If tests are skipped, it > doesn't get installed, which leads to crashes when the JSON serializer > is subsequently used. > > It looks like fixing this just involves moving JSON::MaybeXS from > [Prereqs / TestRequires] to [Prereqs] in the dist.ini file.
I think that's being done so as to not force everyone to install JSON::MaybeXS if they're not going to use the JSON serialization bits.
  Wed May 06 20:10:32 2015 The RT System itself - Status changed from 'new' to 'open'
  Wed May 06 20:11:10 2015 ether [...] cpan.org - Subject changed from 'JSON::MaybeXS should be build_requires, not test_requires' to 'JSON::MaybeXS should be runtime_requires, not test_requires'