[libvirt PATCH 00/15] various meson fixes and improvements

Pavel Hrdina posted 15 patches 3 years, 6 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1602165221.git.phrdina@redhat.com
libvirt.spec.in       |   2 +
meson.build           | 135 +++++++++++++++++++++++++-----------------
meson_options.txt     |  12 ++--
src/bhyve/meson.build |   4 +-
src/util/meson.build  |   4 +-
5 files changed, 96 insertions(+), 61 deletions(-)
[libvirt PATCH 00/15] various meson fixes and improvements
Posted by Pavel Hrdina 3 years, 6 months ago
Pavel Hrdina (15):
  meson_options: change VMware default from enabled to auto
  meson_options: change VirtualBox default from enabled to auto
  meson_options: move firewalld options to build feature options
  meson: no need to call meson.get_compiler() again
  meson: properly handle libpcap if it's explicitly disabled
  meson: properly handle readline if it's explicitly disabled
  meson: build nodedev driver only if libvirtd is compiled
  meson: add libnl build option
  meson: add rbd build option
  meson: prefix kvm_dep, m_dep and util_dep with lib
  meson: add missing libraries to summary
  meson: remove required libraries from summary
  meson: move build feature options to miscellaneous summary
  meson: add docs option to enable/disable generating documentation
  meson: add tests build option to enable/disable unit tests

 libvirt.spec.in       |   2 +
 meson.build           | 135 +++++++++++++++++++++++++-----------------
 meson_options.txt     |  12 ++--
 src/bhyve/meson.build |   4 +-
 src/util/meson.build  |   4 +-
 5 files changed, 96 insertions(+), 61 deletions(-)

-- 
2.26.2

Re: [libvirt PATCH 00/15] various meson fixes and improvements
Posted by Andrea Bolognani 3 years, 6 months ago
On Thu, 2020-10-08 at 15:58 +0200, Pavel Hrdina wrote:
> Pavel Hrdina (15):
>   meson_options: change VMware default from enabled to auto
>   meson_options: change VirtualBox default from enabled to auto
>   meson_options: move firewalld options to build feature options
>   meson: no need to call meson.get_compiler() again
>   meson: properly handle libpcap if it's explicitly disabled
>   meson: properly handle readline if it's explicitly disabled
>   meson: build nodedev driver only if libvirtd is compiled
>   meson: add libnl build option
>   meson: add rbd build option
>   meson: prefix kvm_dep, m_dep and util_dep with lib
>   meson: add missing libraries to summary
>   meson: remove required libraries from summary
>   meson: move build feature options to miscellaneous summary
>   meson: add docs option to enable/disable generating documentation
>   meson: add tests build option to enable/disable unit tests
> 
>  libvirt.spec.in       |   2 +
>  meson.build           | 135 +++++++++++++++++++++++++-----------------
>  meson_options.txt     |  12 ++--
>  src/bhyve/meson.build |   4 +-
>  src/util/meson.build  |   4 +-
>  5 files changed, 96 insertions(+), 61 deletions(-)

I've added a few minor comments to individual patches, but regardless
of how you decide to address those the series gets a

  Reviewed-by: Andrea Bolognani <abologna@redhat.com>

-- 
Andrea Bolognani / Red Hat / Virtualization

Re: [libvirt PATCH 00/15] various meson fixes and improvements
Posted by Pavel Hrdina 3 years, 6 months ago
On Thu, Oct 08, 2020 at 08:18:51PM +0200, Andrea Bolognani wrote:
> On Thu, 2020-10-08 at 15:58 +0200, Pavel Hrdina wrote:
> > Pavel Hrdina (15):
> >   meson_options: change VMware default from enabled to auto
> >   meson_options: change VirtualBox default from enabled to auto
> >   meson_options: move firewalld options to build feature options
> >   meson: no need to call meson.get_compiler() again
> >   meson: properly handle libpcap if it's explicitly disabled
> >   meson: properly handle readline if it's explicitly disabled
> >   meson: build nodedev driver only if libvirtd is compiled
> >   meson: add libnl build option
> >   meson: add rbd build option
> >   meson: prefix kvm_dep, m_dep and util_dep with lib
> >   meson: add missing libraries to summary
> >   meson: remove required libraries from summary
> >   meson: move build feature options to miscellaneous summary
> >   meson: add docs option to enable/disable generating documentation
> >   meson: add tests build option to enable/disable unit tests
> > 
> >  libvirt.spec.in       |   2 +
> >  meson.build           | 135 +++++++++++++++++++++++++-----------------
> >  meson_options.txt     |  12 ++--
> >  src/bhyve/meson.build |   4 +-
> >  src/util/meson.build  |   4 +-
> >  5 files changed, 96 insertions(+), 61 deletions(-)
> 
> I've added a few minor comments to individual patches, but regardless
> of how you decide to address those the series gets a
> 
>   Reviewed-by: Andrea Bolognani <abologna@redhat.com>

Thanks, I'll fix the issues, drop rbd build option patch and push it.

Pavel