[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [Libguestfs] [PATCH v3 1/4] mltools: Rename Yajl module as JSON_parser and move to common/mltools.

On Fri, Aug 17, 2018 at 05:44:58PM +0200, Pino Toscano wrote:
> On Friday, 17 August 2018 17:30:35 CEST Eric Blake wrote:
> > On 08/17/2018 10:16 AM, Richard W.M. Jones wrote:
> > > Commit bd1c5c9f4dcf38458099db8a0bf4659a07ef055d changed all the code
> > > to use Jansson instead of yajl.  However it didn't change the OCaml
> > > API name (which was still Yajl).
> > > 
> > 
> > Are you aware that Jansson can't parse all JSON generated by qemu, and 
> > that the developers of Jansson did not seem sympathetic to patches that 
> > would make it possible?  Libvirt recently reverted their use of Jansson 
> > because of its inability to deal with unsigned 64-bit numbers (and 
> > sadly, RFC7159 does not define bounds for what forms valid JSON numbers, 
> > but merely leaves it up to implementations to decide for themselves).
> The problematic values that qemu outputs are, at least to my knowledge,
> only in the some of the messages in the QMP communication, which we do
> not do.  We parse JSON from:
> - output of ldmtool
> - output of qemu-img
> - QMP schema
> - SimpleStream files
> Furthermore, why not just fix qemu, instead of letting its
> "JSON implementation" rule out perfectly compliant JSON parsers?

I'm going to guess because this would change the qemu API
(if you mean printing out a string instead of an int).


Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
virt-df lists disk usage of guests without needing to install any
software inside the virtual machine.  Supports Linux and Windows.

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]