[pkgsrc/trunk]: pkgsrc/emulators/qemu qemu: fix Linux build in most environments

[gutteridge <gutteridge%pkgsrc.org@localhost>]

details:   https://anonhg.NetBSD.org/pkgsrc/rev/21f443bd0a9a
branches:  trunk
changeset: 347545:21f443bd0a9a
user:      gutteridge <gutteridge%pkgsrc.org@localhost>
date:      Mon Jan 13 02:50:25 2020 +0000

description:
qemu: fix Linux build in most environments

Addresses PR pkg/54772, reported by and patch tested by Ottavio Caruso.
virtfs-proxy-helper is only built if two development libraries (for
libcap and libattr) are installed, which typically aren't included by
default in Linux distros. Make virtfs support a non-default option that
someone would enable when they expressly want it, and they have first
installed the necessary upstream header packages to support the
compilation of this option.

diffstat:

 emulators/qemu/Makefile   |   7 +++----
 emulators/qemu/options.mk |  17 +++++++++++++++--
 2 files changed, 18 insertions(+), 6 deletions(-)

diffs (69 lines):

diff -r 23aea3f67dd2 -r 21f443bd0a9a emulators/qemu/Makefile
--- a/emulators/qemu/Makefile   Mon Jan 13 02:26:55 2020 +0000
+++ b/emulators/qemu/Makefile   Mon Jan 13 02:50:25 2020 +0000
@@ -1,7 +1,7 @@
-# $NetBSD: Makefile,v 1.225 2020/01/12 20:20:13 ryoon Exp $
+# $NetBSD: Makefile,v 1.226 2020/01/13 02:50:25 gutteridge Exp $
 
 DISTNAME=      qemu-4.2.0
-PKGREVISION=   2
+PKGREVISION=   3
 CATEGORIES=    emulators
 MASTER_SITES=  https://download.qemu.org/
 EXTRACT_SUFX=  .tar.xz
@@ -125,14 +125,13 @@
 PLIST.nbd=                     yes
 PLIST.ivshmem=                 yes
 PLIST.pr-helper=               yes
-PLIST.virtfs-proxy-helper=     yes
 .elif !empty(MACHINE_PLATFORM:MSunOS-5.11-*)
 PLIST.ivshmem=                 yes
 PLIST.nbd=                     yes
 CONFIGURE_ARGS+=               --disable-coroutine-pool
 .endif
 
-PLIST_VARS+=           ${UE_ARCHS} bridge-helper ivshmem keymap nbd pr-helper virtfs-proxy-helper
+PLIST_VARS+=           ${UE_ARCHS} bridge-helper ivshmem keymap nbd pr-helper
 .for pvar in ${USER_EMUL}
 PLIST.${pvar}=         yes
 .endfor
diff -r 23aea3f67dd2 -r 21f443bd0a9a emulators/qemu/options.mk
--- a/emulators/qemu/options.mk Mon Jan 13 02:26:55 2020 +0000
+++ b/emulators/qemu/options.mk Mon Jan 13 02:50:25 2020 +0000
@@ -1,17 +1,21 @@
-# $NetBSD: options.mk,v 1.5 2018/08/14 06:57:26 adam Exp $
+# $NetBSD: options.mk,v 1.6 2020/01/13 02:50:25 gutteridge Exp $
 
 PKG_OPTIONS_VAR=       PKG_OPTIONS.qemu
 PKG_SUPPORTED_OPTIONS= gtk3 sdl
 
 .include "../../mk/bsd.fast.prefs.mk"
 
+.if ${OPSYS} == "Linux"
+PKG_SUPPORTED_OPTIONS+=        virtfs-proxy-helper
+.endif
+
 .if ${OPSYS} != "Darwin"
 PKG_SUGGESTED_OPTIONS+=        sdl
 .endif
 
 .include "../../mk/bsd.options.mk"
 
-PLIST_VARS+=           gtk
+PLIST_VARS+=           gtk virtfs-proxy-helper
 
 .if !empty(PKG_OPTIONS:Mgtk3)
 PLIST.gtk=             yes
@@ -27,3 +31,12 @@
 .else
 CONFIGURE_ARGS+=       --disable-sdl
 .endif
+
+# NB to successfully build virtfs-proxy-helper, the upstream Linux
+# header/development libraries for libcap and libattr must be installed.
+.if !empty(PKG_OPTIONS:Mvirtfs-proxy-helper)
+PLIST.virtfs-proxy-helper=     yes
+CONFIGURE_ARGS+=               --enable-virtfs
+.else
+CONFIGURE_ARGS+=               --disable-virtfs
+.endif