On Tue, May 05, 2020 at 12:10:07PM +0200, Michal Privoznik wrote:
diff --git a/tools/virt-host-validate-bhyve.c b/tools/virt-host-validate-bhyve.c index 2f0ec1e36c..3e8cc59a38 100644 --- a/tools/virt-host-validate-bhyve.c +++ b/tools/virt-host-validate-bhyve.c @@ -47,7 +47,6 @@
int virHostValidateBhyve(void) { - int ret = 0; int fileid = 0; struct kld_file_stat stat; bool vmm_loaded = false, if_tap_loaded = false; @@ -73,5 +72,5 @@ int virHostValidateBhyve(void) MODULE_STATUS_WARN(if_bridge, "bridged networking will not work"); MODULE_STATUS_WARN(nmdm, "nmdm console will not work");
- return ret; + return 0; }
This change broke the build because MODULE_STATUS_WARN is a macro that does indeed use 'ret'. Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|