guix-patches
[Top][All Lists]
Advanced

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

[bug#57809] [PATCH] gnome-boxes: Patch 'qemu-img' path


From: Maxime Devos
Subject: [bug#57809] [PATCH] gnome-boxes: Patch 'qemu-img' path
Date: Mon, 19 Sep 2022 15:41:24 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.12.0



On 19-09-2022 06:09, Jacob Hrbek wrote:
+                          (setenv "DESTDIR" "/")))
+                      (add-before 'configure 'substitute-qemu-dependencies
+                        (lambda* (#:key inputs #:allow-other-keys)
+
                 (substitute* "src/installed-media.vala"
+                              (("qemu-img")
+                               (search-input-file inputs 
"/bin/qemu-img"))))))))

Aside from the indentation (which would IIUC be corrected by "guix style", see point (5) of the checklist in "(guix)Submitting Patches")) ...

      (native-inputs
       (list desktop-file-utils           ;for update-desktop-database
             gettext-minimal
@@ -12162,6 +12167,7 @@ (define-public gnome-boxes
             sparql-query
             spice-gtk
             tracker


+           qemu-minimal                 ;for qemu-img

... and the non-alphabetical sorting, ...

             vte
             webkitgtk-with-libsoup2))    ;for webkit2gtk-4.0

... looks good to me (though to be clear, I did not test the changes), though those problems can be addressed by the committer (just a note for future submissions).

Greetings,
Maxime.

Attachment: OpenPGP_0x49E3EE22191725EE.asc
Description: OpenPGP public key

Attachment: OpenPGP_signature
Description: OpenPGP digital signature


reply via email to

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