Skip Menu |
 

This queue is for tickets about the Data-Dump-Streamer CPAN distribution.

Report information
The Basics
Id: 99026
Status: open
Priority: 0/
Queue: Data-Dump-Streamer

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

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



Subject: usage of JSON::XS
Download (untitled) / with headers
text/plain 244b
two requests: - please consider declaring JSON::XS as test recommends in metadata, rather than runtime recommends - expand t/madness_json.t to also test with Cpanel::JSON::XS (it papers over some of the insanity in the JSON::*::Boolean values)
Subject: Re: [rt.cpan.org #99026] usage of JSON::XS
Date: Mon, 22 Sep 2014 10:26:15 +0200
To: "bug-Data-Dump-Streamer [...] rt.cpan.org" <bug-Data-Dump-Streamer [...] rt.cpan.org>
From: demerphq <demerphq [...] gmail.com>
Download (untitled) / with headers
text/plain 1003b
On 21 September 2014 23:47, Karen Etheridge via RT < bug-Data-Dump-Streamer@rt.cpan.org> wrote: Show quoted text
> Sun Sep 21 17:47:38 2014: Request 99026 was acted upon. > Transaction: Ticket created by ETHER > Queue: Data-Dump-Streamer > Subject: usage of JSON::XS > Broken in: (no value) > Severity: Wishlist > Owner: Nobody > Requestors: ether@cpan.org > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=99026 > > > > two requests: > > - please consider declaring JSON::XS as test recommends in metadata, > rather than runtime recommends >
Yeah, this would be nice. It would also be nice if one didnt have to jump through huge hoops to do it as well. Which one unfortunately does. Show quoted text
> - expand t/madness_json.t to also test with Cpanel::JSON::XS (it papers > over some of the insanity in the JSON::*::Boolean values) >
Ill have a look at that. What exactly fails currently if you use that instead? Yves -- perl -Mre=debug -e "/just|another|perl|hacker/"
Subject: Re: [rt.cpan.org #99026] usage of JSON::XS
Date: Mon, 22 Sep 2014 09:23:29 -0700
To: demerphq via RT <bug-Data-Dump-Streamer [...] rt.cpan.org>
From: Karen Etheridge <ether [...] cpan.org>
Download (untitled) / with headers
text/plain 689b
On Mon, Sep 22, 2014 at 04:26:29AM -0400, demerphq via RT wrote: Show quoted text
> > - please consider declaring JSON::XS as test recommends in metadata, > > rather than runtime recommends
> > Yeah, this would be nice. It would also be nice if one didnt have to jump > through huge hoops to do it as well. Which one unfortunately does.
Is switching to ExtUtils::MakeMaker an option? Show quoted text
> > - expand t/madness_json.t to also test with Cpanel::JSON::XS (it papers > > over some of the insanity in the JSON::*::Boolean values)
> > Ill have a look at that. What exactly fails currently if you use that > instead?
I haven't seen any problems so far, but I can't be sure I'm exercising all the edge cases :)
Subject: Re: [rt.cpan.org #99026] usage of JSON::XS
Date: Mon, 22 Sep 2014 19:27:36 +0200
To: "bug-Data-Dump-Streamer [...] rt.cpan.org" <bug-Data-Dump-Streamer [...] rt.cpan.org>
From: demerphq <demerphq [...] gmail.com>
On 22 September 2014 18:23, Karen Etheridge via RT < bug-Data-Dump-Streamer@rt.cpan.org> wrote: Show quoted text
> Queue: Data-Dump-Streamer > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=99026 > > > On Mon, Sep 22, 2014 at 04:26:29AM -0400, demerphq via RT wrote:
> > > - please consider declaring JSON::XS as test recommends in metadata, > > > rather than runtime recommends
> > > > Yeah, this would be nice. It would also be nice if one didnt have to jump > > through huge hoops to do it as well. Which one unfortunately does.
> > Is switching to ExtUtils::MakeMaker an option? > >
Yes it is. DDS was originally EUMM based, but Josh (who was doing maint for a while there) migrated to MB. Show quoted text
> > > - expand t/madness_json.t to also test with Cpanel::JSON::XS (it papers > > > over some of the insanity in the JSON::*::Boolean values)
> > > > Ill have a look at that. What exactly fails currently if you use that > > instead?
> > I haven't seen any problems so far, but I can't be sure I'm exercising all > the edge cases :) > > >
Ah, I see. I get you now. :-) Yves -- perl -Mre=debug -e "/just|another|perl|hacker/"


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.