On 8/28/20 8:48 AM, Michal Privoznik wrote:
On 8/28/20 4:19 PM, Jim Fehlig wrote:
On 8/28/20 8:02 AM, Peter Krempa wrote:
On Fri, Aug 28, 2020 at 07:52:23 -0600, Jim Fehlig wrote:
On 8/28/20 2:00 AM, Pavel Hrdina wrote:
On Thu, Aug 27, 2020 at 03:58:29PM -0600, Jim Fehlig wrote:
Running meson configure with '-Ddriver_hyperv=disabled' fails with
meson.build:1252:0: ERROR: Dependency "openwsman" not found, tried
pkgconfig and cmake
openwsman is only required if the hyperv driver is enabled. Don't
check for it if hyperv is disabled.
Signed-off-by: Jim Fehlig <jfeh...@suse.com>
---
There are likely other ways to fix the problem, this being a meson
noobish one :-).
You are probably using some other options as well because running:
meson build -Ddriver_hyperv=disabled
will not fail if openwsman is missing. The default for openwsman is
'auto'.
I stumbled across it while doing downstream packaging work on 6.7.0. hyperv
is the only thing that requires openwsman and I have it disabled. Here's the
full meson invocation from the build log
/usr/bin/meson --buildtype=plain --prefix=/usr --libdir=/usr/lib64
--libexecdir=/usr/libexec --bindir=/usr/bin --sbindir=/usr/sbin
--includedir=/usr/include --datadir=/usr/share --mandir=/usr/share/man
--infodir=/usr/share/info --localedir=/usr/share/locale --sysconfdir=/etc
--localstatedir=/var --sharedstatedir=/var/lib --wrap-mode=nodownload
--auto-features=enabled . build --libexecdir=/usr/lib64/libvirt
You've specified '--auto-features=enabled', so anything 'auto' is
considered enabled in your build situation.
Facepalm! Thanks for pointing out the obvious. I see that comes from %meson
macro in /usr/lib/rpm/macros.d/macros.meson.
Also, meson warns that -Dnetcontrol=enabled doesn't exist.
It's a message from an older log where I had the downstream netcontrol patch
commented but not the meson define.
BTW, what is the value in having an option for something like openwsman? Isn't
hyperv enough? If hyperv is enabled, then simply error if openwsman is not found.
Regards,
Jim
