Subject: [bug#28853] [PATCH 2/8] gnu: qemu: Enable spice usb redirection support.

From: [email-masked] (Ludovic Courtès)
Date: Thu, 19 Oct 2017 11:52:02 +0200

Hello,

Andy Patterson <[email-masked]> skribis:

> On Mon, 16 Oct 2017 21:15:39 +0800
> [email-masked] (宋文武) wrote:
>
>> Andy Patterson <[email-masked]> writes:
>> 
>> > * gnu/packages/virtualization.scm
>> > (qemu)[arguments]<#:configure-flags>: New argument.
>> > [inputs]: Add usbredir.
>> > (qemu-minimal)[arguments]<#:configure-flags>: Restrict them even
>> > when they are set by qemu.
>> > [inputs]: Remove usbredir.  
>> 
>> Doesn't the 'configure' script auto-detect usb redirection support
>> when usbredir is available?
>
> To me, the advantage of specifying the enabled features is that the
> build will require the dependencies to be available for that feature
> even if they change. So, if more dependencies are required for the
> feature in future versions, the build will immediately inform us, rather
> than somewhat silently dropping support. Thoughts?

That makes sense to me.

I took the liberty to make the change below, to avoid a rebuild of
‘qemu-minimal’, and applied it.

Thanks,
Ludo’.

diff --git a/gnu/packages/virtualization.scm b/gnu/packages/virtualization.scm
index 4a711ee7f..49dee07f5 100644
--- a/gnu/packages/virtualization.scm
+++ b/gnu/packages/virtualization.scm
@@ -192,10 +192,8 @@ server and embedded PowerPC, and S390 guests.")
     (arguments
      (substitute-keyword-arguments (package-arguments qemu)
        ((#:configure-flags _ '(list))
-        `(list
-          ,(string-append
-            "--target-list=i386-softmmu,x86_64-softmmu,mips64el-softmmu"
-            ",arm-softmmu,aarch64-softmmu")))))
+        ;; Restrict to the targets supported by Guix.
+        ''("--target-list=i386-softmmu,x86_64-softmmu,mips64el-softmmu,arm-softmmu,aarch64-softmmu"))))
 
     ;; Remove dependencies on optional libraries, notably GUI libraries.
     (inputs (fold alist-delete (package-inputs qemu)