Re: [PATCH 00/16] Revert Jansson usage

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

 



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


Attachment: signature.asc
Description: Digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux