Skip Menu |

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

Report information
The Basics
Id: 123766
Status: resolved
Priority: 0/
Queue: JSON-PP

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

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



Subject: Clarify usage of json options
Hi, Please see the attached patch provided by Dan Jacobson in the Debian BTS. Best, Dominic.
Subject: json.patch
X-Debbugs-Cc: makamaka@cpan.org Package: libjson-pp-perl Version: 2.94000-1 Severity: minor File: /usr/bin/json_pp --- /usr/bin/json_pp 2017-06-25 03:59:24.000000000 +0800 +++ /tmp/json_pp 2017-11-28 00:37:16.414770019 +0800 @@ -20,7 +20,7 @@ 't=s' => \( my $opt_to = 'json' ), 'json_opt=s' => \( my $json_opt = 'pretty' ), 'V' => \( my $version ), -) or die "Usage: $0 [-V] [-f from_format] [-t to_format] [-json_opt options]\n"; +) or die "Usage: $0 [-V] [-f from_format] [-t to_format] [-json_opt options_to_json1[,options_to_json2[,...]]]\n"; if ( $version ) { @@ -94,7 +94,7 @@ =head1 SYNOPSIS - json_pp [-v] [-f from_format] [-t to_format] [-json_opt options_to_json] + json_pp [-v] [-f from_format] [-t to_format] [-json_opt options_to_json1[,options_to_json2[,...]]] =head1 DESCRIPTION @@ -154,6 +154,12 @@ ascii latin1 utf8 pretty indent space_before space_after relaxed canonical allow_nonref allow_singlequote allow_barekey allow_bignum loose escape_slash +Multiple options must be separated by commas: + + Right: -json_opt pretty,canonical + + Wrong: -json_opt pretty -json_opt canonical + =head2 -v Verbose option, but currently no action in fact.
On Tue Nov 28 06:29:33 2017, DOM wrote: Show quoted text
> Hi, > > Please see the attached patch provided by Dan Jacobson in the Debian BTS. > > Best, > Dominic.
Thanks. Applied: https://github.com/makamaka/JSON-PP/commit/376b14840aee5fc04efd792158075dcebc1ef584
On Wed Nov 29 23:28:53 2017, ISHIGAKI wrote: Show quoted text
> On Tue Nov 28 06:29:33 2017, DOM wrote:
> > Hi, > > > > Please see the attached patch provided by Dan Jacobson in the Debian > > BTS. > > > > Best, > > Dominic.
> > Thanks. Applied: https://github.com/makamaka/JSON- > PP/commit/376b14840aee5fc04efd792158075dcebc1ef584
It seems that I have forgotten to close this. JSON::PP 2.97001 should already have this. Closed. Thanks.