Skip Menu |

This queue is for tickets about the Data-Clean-JSON CPAN distribution.

Report information
The Basics
Id: 112801
Status: resolved
Priority: 0/
Queue: Data-Clean-JSON
People
Owner: Nobody in particular
Requestors: ether [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Tue Mar 08 14:11:24 2016 ether [...] cpan.org - Ticket created
Subject: JSON -> JSON::MaybeXS or JSON::PP?
JSON.pm is a discouraged module now, due to its somewhat broken backend handling and lack of support for Cpanel::JSON::XS. Would you consider switching to JSON::MaybeXS or perhaps just JSON::PP? You only seem to use JSON.pm directly in a few places, so JSON::PP (in core) should do just as well. Additionally, you probably should also check for Cpanel::JSON::XS::Boolean in your bool checks. (I noticed you use JSON.pm in a few other distributions as well, e.g. Data::Sah, Perinci::Access::HTTP::Client, etc.)
  Tue Mar 08 16:31:15 2016 PERLANCAR [...] cpan.org - Correspondence added
On Tue, 8 Mar 2016 19:11:24 GMT, ETHER wrote: Show quoted text
> JSON.pm is a discouraged module now, due to its somewhat broken > backend handling and lack of support for Cpanel::JSON::XS. Would you > consider switching to JSON::MaybeXS or perhaps just JSON::PP? > > You only seem to use JSON.pm directly in a few places, so JSON::PP (in > core) should do just as well. > > Additionally, you probably should also check for > Cpanel::JSON::XS::Boolean in your bool checks. > > > (I noticed you use JSON.pm in a few other distributions as well, e.g. > Data::Sah, Perinci::Access::HTTP::Client, etc.)
Sure, why not. I have a total of 42 distributions currently depending on JSON, so this will take a while.
  Tue Mar 08 16:31:15 2016 The RT System itself - Status changed from 'new' to 'open'
  Tue Mar 15 16:02:17 2016 PERLANCAR [...] cpan.org - Correspondence added
All dists done.
  Tue Mar 15 16:02:22 2016 PERLANCAR [...] cpan.org - Status changed from 'open' to 'resolved'