[libvirt] [PATCH 00/16] Revert Jansson usage

Ján Tomko jtomko at redhat.com
Mon Aug 13 12:54:00 UTC 2018


On Mon, Aug 13, 2018 at 01:55:10PM +0200, Ján Tomko wrote:
>For QEMU, we need a JSON parser that is able to handle its non-compliant
>JSON usage:
>https://bugzilla.redhat.com/show_bug.cgi?id=1614569
>
>Unfortunately, this does not seem to be possible with Jansson.
>
>Revert back to using yajl, which also lets us get rid of the 'dlopen'
>hacks and their bugfixes.
>
>(The QEMU default changes are not strictly required, but the patches
> specifically use JANSSON)
>
>Also available on:
>git://repo.or.cz/libvirt/jtomko.git revert
>
>Ján Tomko (16):
>  Revert "src: Move DLOPEN_LIBS to libraries introducing the dependency"

Note that this patch broke the build with clang by adding DLOPEN_LIBS to
libvirt_setuid_rpc_client_la_CFLAGS instead of LDFLAGS:
https://www.redhat.com/archives/libvir-list/2018-August/msg00604.html
So technically this series is a build breaker fix ;)

>  Revert "Fix link errors in tools/nss and tests"
>  Revert "remote: daemon: Make sure that JSON symbols are properly
>    loaded at startup"
>  Revert "util: jsoncompat: Stub out virJSONInitialize when compiling
>    without jansson"
>  Revert "tests: qemucapsprobe: Fix output after switching to jansson"
>  Revert "util: avoid symbol clash between json libraries"
>  Revert "tests: also skip qemuagenttest with old jansson"
>  Revert "m4: Introduce STABLE_ORDERING_JANSSON"
>  Revert "build: require Jansson if QEMU driver is enabled"
>  Revert "build: switch --with-qemu default from yes to check"
>  Revert "Remove virJSONValueNewStringLen"
>  Revert "build: remove references to WITH_YAJL for SETUID_RPC_CLIENT"
>  Revert "Remove functions using yajl"
>  Revert "Switch from yajl to Jansson"
>  Revert "build: undef WITH_JANSSON for SETUID_RPC_CLIENT"

>  Revert "build: add --with-jansson"

And this patch was the only one that did not revert cleanly.

Jano

>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20180813/538c9f12/attachment-0001.sig>


More information about the libvir-list mailing list