Re: [PATCH] Remove redundant usage of WITH_YAJL

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

 



On 12/12/19 6:22 PM, Ján Tomko wrote:
As of commit 2a00ef6e71f30241f9ca6288da984d75f3cef957 which
was released in v5.2.0, we require YAJL to build the QEMU driver.

Remove the checks from code that requires the QEMU driver
or checks that also check for WITH_QEMU.

Signed-off-by: Ján Tomko <jtomko@xxxxxxxxxx>
---
  src/qemu/qemu_driver.c      |  5 -----
  tests/cputest.c             | 16 +++++++-------
  tests/qemuagenttest.c       |  5 -----
  tests/qemublocktest.c       | 42 ++++++++++++++-----------------------
  tests/qemumigparamstest.c   |  5 -----
  tests/qemumonitorjsontest.c |  5 -----
  6 files changed, 24 insertions(+), 54 deletions(-)

You can also skip checks from:

tests/qemucapabilitiestest.c
tests/qemucaps2xmltest.c
tests/qemucommandutiltest.c
tests/qemuhotplugtest.c

since these are ale guarded by WITH_QEMU check which can be true only if WITH_YAJL is true. With that:

Reviewed-by: Michal Privoznik <mprivozn@xxxxxxxxxx>

Michal

--
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