[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

From: Ludovic Courtès
Subject: [bug#28853] [PATCH 2/8] gnu: qemu: Enable spice usb redirection support.
Date: Thu, 19 Oct 2017 11:52:02 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux)


Andy Patterson <address@hidden> skribis:

> On Mon, 16 Oct 2017 21:15:39 +0800
> address@hidden (宋文武) wrote:
>> Andy Patterson <address@hidden> 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.


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.")
      (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.
     ;; Remove dependencies on optional libraries, notably GUI libraries.
     (inputs (fold alist-delete (package-inputs qemu)

reply via email to

[Prev in Thread] Current Thread [Next in Thread]