[PATCH v3 0/7] virdnsmasq: Lookup DNSMASQ in PATH

Michal Privoznik posted 7 patches 2 years, 3 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1641976729.git.mprivozn@redhat.com
There is a newer version of this series
src/libvirt_private.syms    |  1 -
src/util/virdnsmasq.c       | 50 ++++++++-----------------------------
src/util/virdnsmasq.h       |  1 -
tests/meson.build           |  1 +
tests/networkmock.c         | 46 ++++++++++++++++++++++++++++++++++
tests/networkxml2conftest.c | 46 ++++++++++++++++++++++++++++++----
6 files changed, 98 insertions(+), 47 deletions(-)
create mode 100644 tests/networkmock.c
[PATCH v3 0/7] virdnsmasq: Lookup DNSMASQ in PATH
Posted by Michal Privoznik 2 years, 3 months ago
v3 of:

https://listman.redhat.com/archives/libvir-list/2022-January/msg00386.html

diff to v2:
- Make dnsmasqCapsGetBinaryPath() fail if dnsmasq is not found
- Change the way caps are constructed in networkxml2conftest
- More code cleanup

However, as I was going through the code I realized that what we really
do is check dnsmasq's version against the minimal required one (2.67).
No actual capabilities are parsed. This is due to cleanup merged earlier
this release (v8.0.0-rc1~138 and commits around). While we could drop
more code I figured it may be worth keeping it for future use. I mean,
one day we might want to have an capability for give feature.

Michal Prívozník (7):
  virdnsmasq: Drop @binaryPath argument from dnsmasqCapsNewEmpty()
  lib: Prefer g_autoptr(dnsmasqCaps) instead of explicit unref
  virdnsmasq: Lookup DNSMASQ in PATH
  virdnsmasq: Require non NULL @caps in dnsmasqCapsGetBinaryPath()
  networkxml2conftest: Use dnsmasqCapsNewFromBinary() to construct caps
  virdnsmasq: Drop dnsmasqCapsNewFromBuffer()
  virdnsmasq: Join dnsmasqCapsNewEmpty() and dnsmasqCapsNewFromBinary()

 src/libvirt_private.syms    |  1 -
 src/util/virdnsmasq.c       | 50 ++++++++-----------------------------
 src/util/virdnsmasq.h       |  1 -
 tests/meson.build           |  1 +
 tests/networkmock.c         | 46 ++++++++++++++++++++++++++++++++++
 tests/networkxml2conftest.c | 46 ++++++++++++++++++++++++++++++----
 6 files changed, 98 insertions(+), 47 deletions(-)
 create mode 100644 tests/networkmock.c

-- 
2.34.1