unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#57809] [PATCH] gnome-boxes: Add 'qemu' as propagated-input
@ 2022-09-14 18:17 Jacob Hrbek
  2022-09-14 18:18 ` [bug#57810] " Jacob Hrbek
                   ` (6 more replies)
  0 siblings, 7 replies; 20+ messages in thread
From: Jacob Hrbek @ 2022-09-14 18:17 UTC (permalink / raw)
  To: 57809


[-- Attachment #1.1: Type: text/plain, Size: 455 bytes --]

Fixes:

(gnome-boxes:9304): Boxes-WARNING **: 19:54:29.345: review-page.vala:32: Box setup failed: Failed to create volume: internal error: creation of non-raw file images is not supported without qemu-img.

(gnome-boxes:9304): Boxes-CRITICAL **: 19:54:29.345: boxes_assistant_review_page_populate: assertion 'machine != NULL' failed

Steps to reproduce:
1. Open gnome-boxes on GNU Guix
2. Try to create a box with GNU Guix ISO and expect failure above



[-- Attachment #1.2: publickey - kreyren@rixotstudio.cz - 1677db82.asc --]
[-- Type: application/pgp-keys, Size: 713 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 249 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#57810] [PATCH] gnome-boxes: Add 'qemu' as propagated-input
  2022-09-14 18:17 [bug#57809] [PATCH] gnome-boxes: Add 'qemu' as propagated-input Jacob Hrbek
@ 2022-09-14 18:18 ` Jacob Hrbek
  2022-09-15 13:03 ` [bug#57809] " Maxime Devos
                   ` (5 subsequent siblings)
  6 siblings, 0 replies; 20+ messages in thread
From: Jacob Hrbek @ 2022-09-14 18:18 UTC (permalink / raw)
  To: 57810; +Cc: Jacob Hrbek


[-- Attachment #1.1: Type: text/plain, Size: 677 bytes --]

Results in a runtime failure to create the boxes without it
---
 gnu/packages/gnome.scm | 1 +
 1 file changed, 1 insertion(+)

diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
index d6d22f8eb1..a8fd284766 100644
--- a/gnu/packages/gnome.scm
+++ b/gnu/packages/gnome.scm
@@ -12164,6 +12164,7 @@ (define-public gnome-boxes
            tracker
            vte
            webkitgtk-with-libsoup2))    ;for webkit2gtk-4.0
+    (propagated-inputs (list qemu))
     (home-page "https://wiki.gnome.org/Apps/Boxes")
     (synopsis "View, access, and manage remote and virtual systems")
     (description "GNOME Boxes is a simple application to view, access, and
--
2.37.1


[-- Attachment #1.2: publickey - kreyren@rixotstudio.cz - 1677db82.asc --]
[-- Type: application/pgp-keys, Size: 713 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 249 bytes --]

^ permalink raw reply related	[flat|nested] 20+ messages in thread

* [bug#57809] [PATCH] gnome-boxes: Add 'qemu' as propagated-input
  2022-09-14 18:17 [bug#57809] [PATCH] gnome-boxes: Add 'qemu' as propagated-input Jacob Hrbek
  2022-09-14 18:18 ` [bug#57810] " Jacob Hrbek
@ 2022-09-15 13:03 ` Maxime Devos
  2022-09-18 20:43 ` [bug#57809] [PATCH] gnome-boxes: Patch 'qemu-img' path Jacob Hrbek
                   ` (4 subsequent siblings)
  6 siblings, 0 replies; 20+ messages in thread
From: Maxime Devos @ 2022-09-15 13:03 UTC (permalink / raw)
  To: Jacob Hrbek, 57809


[-- Attachment #1.1.1: Type: text/plain, Size: 1048 bytes --]

On 14-09-2022 20:17, Jacob Hrbek wrote:
> Fixes:
> 
> (gnome-boxes:9304): Boxes-WARNING **: 19:54:29.345: review-page.vala:32: Box setup failed: Failed to create volume: internal error: creation of non-raw file images is not supported without qemu-img.
> 
> (gnome-boxes:9304): Boxes-CRITICAL **: 19:54:29.345: boxes_assistant_review_page_populate: assertion 'machine != NULL' failed
> 
> Steps to reproduce:
> 1. Open gnome-boxes on GNU Guix
> 2. Try to create a box with GNU Guix ISO and expect failure above
> 
> 

Instead of propagation, I recommend doing a substitute* instead, to 
avoid the usual downsides of propagation.  Searching for 'qemu-img' in 
the source code of gnome-boxes (with tar, grep -rF and guix build 
--source), I found:

src/installed-media.vala:        string[] argv = { "qemu-img", 
"convert", "-O", "qcow2", device_file, destination_path };

Could you substitute* "qemu-img" -> (search-input-file inputs 
"bin/qemu-img") and move QEMU from 'propagated-inputs' to 'inputs'.

Greetings,
Maxime.

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 929 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 236 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#57809] [PATCH] gnome-boxes: Patch 'qemu-img' path
  2022-09-14 18:17 [bug#57809] [PATCH] gnome-boxes: Add 'qemu' as propagated-input Jacob Hrbek
  2022-09-14 18:18 ` [bug#57810] " Jacob Hrbek
  2022-09-15 13:03 ` [bug#57809] " Maxime Devos
@ 2022-09-18 20:43 ` Jacob Hrbek
  2022-09-18 21:05   ` Jacob Hrbek
  2022-09-18 20:46 ` Jacob Hrbek
                   ` (3 subsequent siblings)
  6 siblings, 1 reply; 20+ messages in thread
From: Jacob Hrbek @ 2022-09-18 20:43 UTC (permalink / raw)
  To: 57809; +Cc: Jacob Hrbek


[-- Attachment #1.1: Type: text/plain, Size: 1519 bytes --]

---
 gnu/packages/gnome.scm | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
index d6d22f8eb1..89f5999920 100644
--- a/gnu/packages/gnome.scm
+++ b/gnu/packages/gnome.scm
@@ -12111,6 +12111,7 @@ (define-public polari
     (home-page "https://wiki.gnome.org/Apps/Polari")
     (license license:gpl2+)))

+;; NOTE(Krey): src/util-app.vala might need patched paths
 (define-public gnome-boxes
   (package
     (name "gnome-boxes")
@@ -12133,7 +12134,12 @@ (define-public gnome-boxes
            #:phases #~(modify-phases %standard-phases
                         (add-after 'unpack 'disable-gtk-update-icon-cache
                           (lambda _
-                            (setenv "DESTDIR" "/"))))))
+                            (setenv "DESTDIR" "/")))
+                        (add-before 'configure 'substitute-qemu-dependencies
+                          (lambda _
+
    (substitute* "src/installed-media.vala"
+                                (("qemu-img")
+                                 (search-input-file %build-inputs "/bin/qemu-img"))))))))
     (native-inputs
      (list desktop-file-utils           ;for update-desktop-database
            gettext-minimal
@@ -12141,6 +12147,7 @@ (define-public gnome-boxes
            itstool
            pkg-config
            python
+           qemu-minimal                 ;for qemu-img
            vala))
     (inputs
      (list glib-networking              ;for TLS support
--
2.37.1


[-- Attachment #1.2: publickey - kreyren@rixotstudio.cz - 1677db82.asc --]
[-- Type: application/pgp-keys, Size: 713 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 249 bytes --]

^ permalink raw reply related	[flat|nested] 20+ messages in thread

* [bug#57809] [PATCH] gnome-boxes: Patch 'qemu-img' path
  2022-09-14 18:17 [bug#57809] [PATCH] gnome-boxes: Add 'qemu' as propagated-input Jacob Hrbek
                   ` (2 preceding siblings ...)
  2022-09-18 20:43 ` [bug#57809] [PATCH] gnome-boxes: Patch 'qemu-img' path Jacob Hrbek
@ 2022-09-18 20:46 ` Jacob Hrbek
  2022-09-18 20:47 ` [bug#57809] [PATCH] gnome-boxes: Add 'qemu' as propagated-input Jacob Hrbek
                   ` (2 subsequent siblings)
  6 siblings, 0 replies; 20+ messages in thread
From: Jacob Hrbek @ 2022-09-18 20:46 UTC (permalink / raw)
  To: 57809; +Cc: Jacob Hrbek


[-- Attachment #1.1: Type: text/plain, Size: 1287 bytes --]

---
 gnu/packages/gnome.scm | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
index d6d22f8eb1..d9cfa5ebed 100644
--- a/gnu/packages/gnome.scm
+++ b/gnu/packages/gnome.scm
@@ -12133,7 +12133,12 @@ (define-public gnome-boxes
            #:phases #~(modify-phases %standard-phases
                         (add-after 'unpack 'disable-gtk-update-icon-cache
                           (lambda _
-                            (setenv "DESTDIR" "/"))))))
+                            (setenv "DESTDIR" "/")))
+                        (add-before 'configure 'substitute-qemu-dependencies
+                          (lambda _
+                              (substitute* "src/installed-media.vala"
+                                (("qemu-img")
+                                 (search-input-file %build-inputs "/bin/qemu-img"))))))))
     (native-inputs
      (list desktop-file-utils           ;for update-desktop-
database
            gettext-minimal
@@ -12141,6 +12146,7 @@ (define-public gnome-boxes
            itstool
            pkg-config
            python
+           qemu-minimal                 ;for qemu-img
            vala))
     (inputs
      (list glib-networking              ;for TLS support
--
2.37.1


[-- Attachment #1.2: publickey - kreyren@rixotstudio.cz - 1677db82.asc --]
[-- Type: application/pgp-keys, Size: 713 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 249 bytes --]

^ permalink raw reply related	[flat|nested] 20+ messages in thread

* [bug#57809] [PATCH] gnome-boxes: Add 'qemu' as propagated-input
  2022-09-14 18:17 [bug#57809] [PATCH] gnome-boxes: Add 'qemu' as propagated-input Jacob Hrbek
                   ` (3 preceding siblings ...)
  2022-09-18 20:46 ` Jacob Hrbek
@ 2022-09-18 20:47 ` Jacob Hrbek
  2022-09-19  4:09 ` [bug#57809] [PATCH] gnome-boxes: Patch 'qemu-img' path Jacob Hrbek
  2022-09-19 21:36 ` [bug#57809] [PATCH] gnome-boxes: Fix path for 'qemu-img' Jacob Hrbek
  6 siblings, 0 replies; 20+ messages in thread
From: Jacob Hrbek @ 2022-09-18 20:47 UTC (permalink / raw)
  To: 57809


[-- Attachment #1.1.1: Type: text/plain, Size: 145 bytes --]

While looking through the code i've noticed a path using /etc/.. so i
added a note about it so two patches one with the note and one without
it


[-- Attachment #1.1.2: Type: text/html, Size: 229 bytes --]

[-- Attachment #1.2: publickey - kreyren@rixotstudio.cz - 1677db82.asc --]
[-- Type: application/pgp-keys, Size: 713 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 249 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#57809] [PATCH] gnome-boxes: Patch 'qemu-img' path
  2022-09-18 20:43 ` [bug#57809] [PATCH] gnome-boxes: Patch 'qemu-img' path Jacob Hrbek
@ 2022-09-18 21:05   ` Jacob Hrbek
  2022-09-18 21:14     ` Maxime Devos
  0 siblings, 1 reply; 20+ messages in thread
From: Jacob Hrbek @ 2022-09-18 21:05 UTC (permalink / raw)
  To: 57809


[-- Attachment #1.1.1: Type: text/plain, Size: 1840 bytes --]

isn't that going to break if the user invokes `guix cg -D` though?

On Ne, zář 18 2022 at 20:43:43 +0000, Jacob Hrbek 
<kreyren@rixotstudio.cz> wrote:
> ---
>  gnu/packages/gnome.scm | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
> index d6d22f8eb1..89f5999920 100644
> --- a/gnu/packages/gnome.scm
> +++ b/gnu/packages/gnome.scm
> @@ -12111,6 +12111,7 @@ (define-public polari
>      (home-page "<https://wiki.gnome.org/Apps/Polari>")
>      (license license:gpl2+)))
> 
> +;; NOTE(Krey): src/util-app.vala might need patched paths
>  (define-public gnome-boxes
>    (package
>      (name "gnome-boxes")
> @@ -12133,7 +12134,12 @@ (define-public gnome-boxes
>             #:phases #~(modify-phases %standard-phases
>                          (add-after 'unpack 
> 'disable-gtk-update-icon-cache
>                            (lambda _
> -                            (setenv "DESTDIR" "/"))))))
> +                            (setenv "DESTDIR" "/")))
> +                        (add-before 'configure 
> 'substitute-qemu-dependencies
> +                          (lambda _
> +                              (substitute* "src/installed-media.vala"
> +                                (("qemu-img")
> +                                 (search-input-file %build-inputs 
> "/bin/qemu-img"))))))))
>      (native-inputs
>       (list desktop-file-utils           ;for update-desktop-database
>             gettext-minimal
> @@ -12141,6 +12147,7 @@ (define-public gnome-boxes
>             itstool
>             pkg-config
>             python
> +           qemu-minimal                 ;for qemu-img
>             vala))
>      (inputs
>       (list glib-networking              ;for TLS support
> --
> 2.37.1
> 


[-- Attachment #1.1.2: Type: text/html, Size: 2078 bytes --]

[-- Attachment #1.2: publickey - kreyren@rixotstudio.cz - 1677db82.asc --]
[-- Type: application/pgp-keys, Size: 713 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 249 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#57809] [PATCH] gnome-boxes: Patch 'qemu-img' path
  2022-09-18 21:05   ` Jacob Hrbek
@ 2022-09-18 21:14     ` Maxime Devos
  2022-09-18 21:32       ` Jacob Hrbek
  0 siblings, 1 reply; 20+ messages in thread
From: Maxime Devos @ 2022-09-18 21:14 UTC (permalink / raw)
  To: Jacob Hrbek, 57809


[-- Attachment #1.1.1: Type: text/plain, Size: 1257 bytes --]



On 18-09-2022 23:05, Jacob Hrbek wrote:
> isn't that going to break if the user invokes `guix cg -D` though?

No, "guix gc" only deletes things that aren't (indirectly) referred to 
by a root (being a garbage collector), and by adding that substitution, 
the compiled gnome-boxes refers to qemu(-minimal).

(Unless the reference is obfuscated, but Vala has a C-like compilation 
model, so I wouldn't expect that.)

> +                              (substitute* "src/installed-media.vala"
> +                                (("qemu-img")
> +                                 (search-input-file %build-inputs "/bin/qemu-img"))))))))

I find (search-input-file inputs "/bin/qemu-img") clearer -- 'inputs' / 
'native-inputs' is sufficiently complicated on its own, no need to bring 
%build-inputs and friends in the equation when it can be avoided.

> +;; NOTE(Krey): src/util-app.vala might need patched paths

(1) the convention is TODO or FIXME, not NOTE
(2) who discovered the potential problem, does not need to be mentioned 
-- there is still "git blame" (which can be used for other things than 
blaming) and "git log" if it turns out to be somehow useful information 
after all (e.g. for attribution).

Greetings,
Maxime.

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 929 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 236 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#57809] [PATCH] gnome-boxes: Patch 'qemu-img' path
  2022-09-18 21:14     ` Maxime Devos
@ 2022-09-18 21:32       ` Jacob Hrbek
  2022-09-18 21:50         ` Maxime Devos
  0 siblings, 1 reply; 20+ messages in thread
From: Jacob Hrbek @ 2022-09-18 21:32 UTC (permalink / raw)
  To: Maxime Devos; +Cc: 57809


[-- Attachment #1.1.1: Type: text/plain, Size: 1058 bytes --]

On Ne, zář 18 2022 at 23:14:12 +0200, Maxime Devos 
<maximedevos@telenet.be> wrote:
> I find (search-input-file inputs "/bin/qemu-img") clearer -- 'inputs' 
> /
> 'native-inputs' is sufficiently complicated on its own, no need to 
> bring
> %build-inputs and friends in the equation when it can be avoided.

That ain't working for me: <https://i.imgur.com/Co4Xsjl.png>

    unbound-variable #f "Unbound variable: ~S" (native-inputs) #f

 > (1) the convention is TODO or FIXME, not NOTE

It's not:
- TODO as in "To Do something" or "for something to be done"
- FIXME as in "Low priority thing or quality assurance something that 
needs to be fixed"

More like handing over note to the next guy about possible issue in the 
future and leaving the signature here so that they can just go on chat 
and ask instead of wasting ~3 min on average searching git blame/log 
which gets exponentially more unsustainable to track

At least that's how it works where i work O_o .. personally i find it 
better way than the generic TODO/FIXME


[-- Attachment #1.1.2: Type: text/html, Size: 1462 bytes --]

[-- Attachment #1.2: publickey - kreyren@rixotstudio.cz - 1677db82.asc --]
[-- Type: application/pgp-keys, Size: 713 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 249 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#57809] [PATCH] gnome-boxes: Patch 'qemu-img' path
  2022-09-18 21:32       ` Jacob Hrbek
@ 2022-09-18 21:50         ` Maxime Devos
  2022-09-19  4:28           ` Jacob Hrbek
  0 siblings, 1 reply; 20+ messages in thread
From: Maxime Devos @ 2022-09-18 21:50 UTC (permalink / raw)
  To: Jacob Hrbek; +Cc: 57809


[-- Attachment #1.1.1: Type: text/plain, Size: 2314 bytes --]



On 18-09-2022 23:32, Jacob Hrbek wrote:
> On Ne, zář 18 2022 at 23:14:12 +0200, Maxime Devos 
> <maximedevos@telenet.be> wrote:
>> I find (search-input-file inputs "/bin/qemu-img") clearer -- 'inputs' 
>> / 'native-inputs' is sufficiently complicated on its own, no need to 
>> bring %build-inputs and friends in the equation when it can be avoided. 
> 
> That ain't working for me: https://i.imgur.com/Co4Xsjl.png 
> <https://i.imgur.com/Co4Xsjl.png>

Quoting the relevant code would be sufficient, no need for image links. 
  Besides, text is more accessible than images of text, text would reach 
a wider audience.

>      unbound-variable #f "Unbound variable: ~S" (native-inputs) #f

(1) 'qemu-minimal' is in the wrong place, it needs to be in 'inputs', 
not 'native-inputs', otherwise it is compiled for the wrong 
architecture. (In both the phase and the inputs/native-inputs field)
(2) 'native-inputs' (or 'inputs'), is unbound.  For information on how 
to use search-input-file, inputs and phases, see the manual, in 
particular (guix)Build Utilities (index entry: search-input-file) and 
its example.

>> (1) the convention is TODO or FIXME, not NOTE  It's not: - TODO as in "To Do something" or "for something to be done" 
> - FIXME as in "Low priority thing or quality assurance something that 
> needs to be fixed"

A missing hardcoded file name can easily be a (usually low-priority) 
bug, so FIXME applies, it's also something to investigate and solve (if 
appropriate), so TODO applies.

> More like handing over note to the next guy about possible issue in the 
> future and leaving the signature here so that they can just go on chat

The required information is already in the "TODO: [stuff about file 
names]", they do not need to go 'on chat', and if they do anyway, 
hardcoding file names is a relatively well-known solution, so I don't 
expect a contact person to be needed.

If that's not sufficient, you can insert an (impersonal) link to the 
Guix issue <https://issues.guix.gnu.org/issue/57809>, which has all the 
relevant information (which is available even when you aren't available 
for chatting, and also when your IRC nick is unknown or if you're gone).

Also, there are no gender restrictions in Guix.

Greetings,
Maxime.

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 929 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 236 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#57809] [PATCH] gnome-boxes: Patch 'qemu-img' path
  2022-09-14 18:17 [bug#57809] [PATCH] gnome-boxes: Add 'qemu' as propagated-input Jacob Hrbek
                   ` (4 preceding siblings ...)
  2022-09-18 20:47 ` [bug#57809] [PATCH] gnome-boxes: Add 'qemu' as propagated-input Jacob Hrbek
@ 2022-09-19  4:09 ` Jacob Hrbek
  2022-09-19 13:41   ` Maxime Devos
  2022-09-26 21:03   ` bug#57809: [PATCH] gnome-boxes: Add 'qemu' as propagated-input Ludovic Courtès
  2022-09-19 21:36 ` [bug#57809] [PATCH] gnome-boxes: Fix path for 'qemu-img' Jacob Hrbek
  6 siblings, 2 replies; 20+ messages in thread
From: Jacob Hrbek @ 2022-09-19  4:09 UTC (permalink / raw)
  To: 57809; +Cc: Jacob Hrbek


[-- Attachment #1.1: Type: text/plain, Size: 1584 bytes --]

---
 gnu/packages/gnome.scm | 12 +++++++++---
 1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
index d6d22f8eb1..ff5cb92907 100644
--- a/gnu/packages/gnome.scm
+++ b/gnu/packages/gnome.scm
@@ -12131,9 +12131,14 @@ (define-public gnome-boxes
                                                     #$output
                                                     "/lib/gnome-boxes"))
            #:phases #~(modify-phases %standard-phases
-                        (add-after 'unpack 'disable-gtk-update-icon-cache
-                          (lambda _
-                            (setenv "DESTDIR" "/"))))))
+                      (add-after 'unpack 'disable-gtk-update-icon-cache
+                        (lambda _
+                          (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"))))))))
     (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
            vte
            webkitgtk-with-libsoup2))    ;for webkit2gtk-4.0
     (home-page "https://wiki.gnome.org/Apps/Boxes")
--
2.37.3


[-- Attachment #1.2: publickey - kreyren@rixotstudio.cz - 1677db82.asc --]
[-- Type: application/pgp-keys, Size: 713 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 249 bytes --]

^ permalink raw reply related	[flat|nested] 20+ messages in thread

* [bug#57809] [PATCH] gnome-boxes: Patch 'qemu-img' path
  2022-09-18 21:50         ` Maxime Devos
@ 2022-09-19  4:28           ` Jacob Hrbek
  2022-09-19 13:35             ` Maxime Devos
  0 siblings, 1 reply; 20+ messages in thread
From: Jacob Hrbek @ 2022-09-19  4:28 UTC (permalink / raw)
  To: Maxime Devos; +Cc: 57809


[-- Attachment #1.1: Type: text/plain, Size: 3636 bytes --]

>> (2) 'native-inputs' (or 'inputs'), is unbound.  For information on how 

>> to use search-input-file, inputs and phases, see the manual, in particular (guix)Build Utilities (index entry: search-input-file) >> and 

>> its example.

I've been searching through that thing endlessly for almost 5 hours >_> it doesn't help much for that.. I can grep the code to look for usage though which is what I did here ^-^

>> If that's not sufficient, you can insert an (impersonal) link to the 

>> Guix issue

meh, imho takes too much time especially when it's from a newbie like me who needs guidance which adds bunch of useless info to the issue.

>> which has all the 

>> relevant information (which is available even when you aren't available 

>> for chatting, and also when your IRC nick is unknown or if you're gone).

The idea is that people are using chat/client that has chat history and sends notifications so that people can respond quickly or when they are available worst case sending an email..

-- Jacob "Kreyren" Hrbek

------- Original Message -------
On Sunday, September 18th, 2022 at 11:50 PM, Maxime Devos <maximedevos@telenet.be> wrote:


> 

> 

> On 18-09-2022 23:32, Jacob Hrbek wrote:
> 

> > On Ne, zář 18 2022 at 23:14:12 +0200, Maxime Devos
> > maximedevos@telenet.be wrote:
> > 

> > > I find (search-input-file inputs "/bin/qemu-img") clearer -- 'inputs'
> > > / 'native-inputs' is sufficiently complicated on its own, no need to
> > > bring %build-inputs and friends in the equation when it can be avoided.
> > 

> > That ain't working for me: https://i.imgur.com/Co4Xsjl.png
> > https://i.imgur.com/Co4Xsjl.png
> 

> 

> Quoting the relevant code would be sufficient, no need for image links.
> Besides, text is more accessible than images of text, text would reach
> a wider audience.
> 

> > unbound-variable #f "Unbound variable: ~S" (native-inputs) #f
> 

> 

> (1) 'qemu-minimal' is in the wrong place, it needs to be in 'inputs',
> not 'native-inputs', otherwise it is compiled for the wrong
> architecture. (In both the phase and the inputs/native-inputs field)
> (2) 'native-inputs' (or 'inputs'), is unbound. For information on how
> to use search-input-file, inputs and phases, see the manual, in
> particular (guix)Build Utilities (index entry: search-input-file) and
> its example.
> 

> > > (1) the convention is TODO or FIXME, not NOTE It's not: - TODO as in "To Do something" or "for something to be done"
> > > - FIXME as in "Low priority thing or quality assurance something that
> > > needs to be fixed"
> 

> 

> A missing hardcoded file name can easily be a (usually low-priority)
> bug, so FIXME applies, it's also something to investigate and solve (if
> appropriate), so TODO applies.
> 

> > More like handing over note to the next guy about possible issue in the
> > future and leaving the signature here so that they can just go on chat
> 

> 

> The required information is already in the "TODO: [stuff about file
> names]", they do not need to go 'on chat', and if they do anyway,
> hardcoding file names is a relatively well-known solution, so I don't
> expect a contact person to be needed.
> 

> If that's not sufficient, you can insert an (impersonal) link to the
> Guix issue https://issues.guix.gnu.org/issue/57809, which has all the
> 

> relevant information (which is available even when you aren't available
> for chatting, and also when your IRC nick is unknown or if you're gone).
> 

> Also, there are no gender restrictions in Guix.
> 

> Greetings,
> Maxime.

[-- Attachment #1.2: publickey - kreyren@rixotstudio.cz - 0x1677DB82.asc --]
[-- Type: application/pgp-keys, Size: 661 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 249 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#57809] [PATCH] gnome-boxes: Patch 'qemu-img' path
  2022-09-19  4:28           ` Jacob Hrbek
@ 2022-09-19 13:35             ` Maxime Devos
  0 siblings, 0 replies; 20+ messages in thread
From: Maxime Devos @ 2022-09-19 13:35 UTC (permalink / raw)
  To: Jacob Hrbek; +Cc: 57809


[-- Attachment #1.1.1: Type: text/plain, Size: 1259 bytes --]



On 19-09-2022 06:28, Jacob Hrbek wrote:
>>> (2) 'native-inputs' (or 'inputs'), is unbound.  For information on how
> 
>>> to use search-input-file, inputs and phases, see the manual, in particular (guix)Build Utilities (index entry: search-input-file) >> and
> 
>>> its example.
> 
> I've been searching through that thing endlessly for almost 5 hours >_> it doesn't help much for that.. I can grep the code to look for usage though which is what I did here ^-^

Here are some proposed methods for finding it, which can be adapted for 
other searches as well.

  * "info guix" followed by 's' 'search-input-file'
    (free-form text search, good for finding examples)
  * "info guix" followed by 'i' 'search-input-file'
    (index entry, works well for procedure names)
  * "info guix" > "Programming Interface" > "Build Utilities", followed
     by some scrolling, then find 'search-input-file' there.
    (using the TOC)

The first two options only take a few seconds.  The latter can be quick 
or long, depending on whether your guess on where it's located is good 
or bad.

And a fourth option, since you seem to like grepping:

  * git grep -F search-input-file doc/
    (in a git checkout of Guix)

Greetings,
Maxime.

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 929 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 236 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#57809] [PATCH] gnome-boxes: Patch 'qemu-img' path
  2022-09-19  4:09 ` [bug#57809] [PATCH] gnome-boxes: Patch 'qemu-img' path Jacob Hrbek
@ 2022-09-19 13:41   ` Maxime Devos
  2022-09-26 21:03   ` bug#57809: [PATCH] gnome-boxes: Add 'qemu' as propagated-input Ludovic Courtès
  1 sibling, 0 replies; 20+ messages in thread
From: Maxime Devos @ 2022-09-19 13:41 UTC (permalink / raw)
  To: Jacob Hrbek, 57809


[-- Attachment #1.1.1: Type: text/plain, Size: 1249 bytes --]



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 #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 929 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 236 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#57809] [PATCH] gnome-boxes: Fix path for 'qemu-img'
  2022-09-14 18:17 [bug#57809] [PATCH] gnome-boxes: Add 'qemu' as propagated-input Jacob Hrbek
                   ` (5 preceding siblings ...)
  2022-09-19  4:09 ` [bug#57809] [PATCH] gnome-boxes: Patch 'qemu-img' path Jacob Hrbek
@ 2022-09-19 21:36 ` Jacob Hrbek
  2022-09-20  5:18   ` Liliana Marie Prikler
  6 siblings, 1 reply; 20+ messages in thread
From: Jacob Hrbek @ 2022-09-19 21:36 UTC (permalink / raw)
  To: 57809; +Cc: Jacob Hrbek


[-- Attachment #1.1: Type: text/plain, Size: 5144 bytes --]

---
 gnu/packages/gnome.scm | 102 ++++++++++++++++++++++-------------------
 1 file changed, 55 insertions(+), 47 deletions(-)

diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
index d6d22f8eb1..65054fc4ac 100644
--- a/gnu/packages/gnome.scm
+++ b/gnu/packages/gnome.scm
@@ -12115,67 +12115,75 @@ (define-public gnome-boxes
   (package
     (name "gnome-boxes")
     (version "42.3")
-    (source
-     (origin
-       (method url-fetch)
-       (uri (string-append "mirror://gnome/sources/gnome-boxes/"
-                           (version-major version) "/"
-                           "gnome-boxes-" version ".tar.xz"))
-       (sha256
-        (base32 "1lv0bdh935qj6wkv3ixg2pcv8yrapj79z02gw4fal3rhz3xggvsn"))))
+    (source (origin
+              (method url-fetch)
+              (uri (string-append "mirror://gnome/sources/gnome-boxes/"
+                                  (version-major version)
+                                  "/"
+
             "gnome-boxes-"
+                                  version
+                                  ".tar.xz"))
+              (sha256
+               (base32
+                "1lv0bdh935qj6wkv3ixg2pcv8yrapj79z02gw4fal3rhz3xggvsn"))))
     (build-system meson-build-system)
     (arguments
      (list #:glib-or-gtk? #t
            #:configure-flags #~(list "-Drdp=false"
-                                     (string-append "-Dc_link_args=-Wl,-rpath="
-                                                    #$output
-                                                    "/lib/gnome-boxes"))
+                                     (string-append
+                                      "-Dc_link_args=-Wl,-rpath="
+                                      #$output "/lib/gnome-boxes"))
            #:phases #~(modify-phases %standard-phases
                         (add-after 'unpack 'disable-gtk-update-icon-cache
                           (lambda _
-                            (
setenv "DESTDIR" "/"))))))
-    (native-inputs
-     (list desktop-file-utils           ;for update-desktop-database
-           gettext-minimal
-           `(,glib "bin")               ;for glib-compile-resources
-           itstool
-           pkg-config
-           python
-           vala))
-    (inputs
-     (list glib-networking              ;for TLS support
-           gsettings-desktop-schemas
-           gtk+
-           gtk-vnc
-           gtksourceview
-           json-glib
-           libarchive
-           libgudev
-           libhandy
-           libosinfo
-           libsecret
-           libsoup-minimal-2
-           libusb
-           libvirt
-           libvirt-glib
-           libxml2
-           sparql-query
-           spice-gtk
-           tracker
-           vte
-           webkitgtk-with-libsoup2))    ;for webkit2gtk-4.0
+                            (setenv "DESTDIR" "/")))
+                        (add-before 'configure 'substitut
e-qemu-dependencies
+                          (lambda* (#:key inputs #:allow-other-keys)
+                            (substitute* "src/installed-media.vala"
+                              (("qemu-img")
+                               (search-input-file inputs "/bin/qemu-img"))))))))
+    (native-inputs (list desktop-file-utils ;for update-desktop-database
+                         gettext-minimal
+                         `(,glib "bin") ;for glib-compile-resources
+                         itstool
+                         pkg-config
+                         python
+                         vala))
+    (inputs (list glib-networking ;for TLS support
+                  gsettings-desktop-schemas
+                  gtk+
+                  gtk-vnc
+                  gtksourceview
+                  json-glib
+                  libarchive
+                  libgudev
+                  libhandy
+                  libosinfo
+                  libsecret
+
    libsoup-minimal-2
+                  libusb
+                  libvirt
+                  libvirt-glib
+                  libxml2
+                  qemu-minimal ;for qemu-img
+                  sparql-query
+                  spice-gtk
+                  tracker
+                  vte
+                  webkitgtk-with-libsoup2)) ;for webkit2gtk-4.0
     (home-page "https://wiki.gnome.org/Apps/Boxes")
     (synopsis "View, access, and manage remote and virtual systems")
-    (description "GNOME Boxes is a simple application to view, access, and
+    (description
+     "GNOME Boxes is a simple application to view, access, and
 manage remote and virtual systems.  Note that this application requires the
 @code{libvirt} and @code{virtlog} daemons to run.  Use the command
 @command{info '(guix) Virtualization Services'} to learn how to configure
 these services on the Guix System.")
     (license (list
-              ;; For data/icons/empty-boxes.png.
-
  license:cc-by2.0
-              ;; For all others.
-              license:lgpl2.0+))))
+                   ;; For data/icons/empty-boxes.png.
+                   license:cc-by2.0
+                   ;; For all others.
+                   license:lgpl2.0+))))

 (define-public geary
   (package
--
2.37.3


[-- Attachment #1.2: publickey - kreyren@rixotstudio.cz - 1677db82.asc --]
[-- Type: application/pgp-keys, Size: 713 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 249 bytes --]

^ permalink raw reply related	[flat|nested] 20+ messages in thread

* [bug#57809] [PATCH] gnome-boxes: Fix path for 'qemu-img'
  2022-09-19 21:36 ` [bug#57809] [PATCH] gnome-boxes: Fix path for 'qemu-img' Jacob Hrbek
@ 2022-09-20  5:18   ` Liliana Marie Prikler
  2022-09-20  7:55     ` Jacob Hrbek
  0 siblings, 1 reply; 20+ messages in thread
From: Liliana Marie Prikler @ 2022-09-20  5:18 UTC (permalink / raw)
  To: Jacob Hrbek, 57809

Am Montag, dem 19.09.2022 um 21:36 +0000 schrieb Jacob Hrbek:
> ---
>  gnu/packages/gnome.scm | 102 ++++++++++++++++++++++-----------------
> --
Too much noise.  Please separate cosmetic and non-cosmetic changes.

Thanks




^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#57809] [PATCH] gnome-boxes: Fix path for 'qemu-img'
  2022-09-20  5:18   ` Liliana Marie Prikler
@ 2022-09-20  7:55     ` Jacob Hrbek
  2022-09-20 17:12       ` Liliana Marie Prikler
  0 siblings, 1 reply; 20+ messages in thread
From: Jacob Hrbek @ 2022-09-20  7:55 UTC (permalink / raw)
  To: Liliana Marie Prikler; +Cc: 57809


[-- Attachment #1.1: Type: text/plain, Size: 655 bytes --]

>> [Krey format the code it looks like trash]

>> [Krey there is too much cosmetic noise it looks like trash]

...

https://issues.guix.gnu.org/57809#9 is the solution

https://issues.guix.gnu.org/57809#13 is the solution + `$ guix style gnome-boxes`

------- Original Message -------
On Tuesday, September 20th, 2022 at 7:18 AM, Liliana Marie Prikler <liliana.prikler@gmail.com> wrote:


> Am Montag, dem 19.09.2022 um 21:36 +0000 schrieb Jacob Hrbek:
> 

> > ---
> > gnu/packages/gnome.scm | 102 ++++++++++++++++++++++-----------------
> > --
> 

> Too much noise. Please separate cosmetic and non-cosmetic changes.
> 

> Thanks

[-- Attachment #1.2: publickey - kreyren@rixotstudio.cz - 0x1677DB82.asc --]
[-- Type: application/pgp-keys, Size: 661 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 249 bytes --]

^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#57809] [PATCH] gnome-boxes: Fix path for 'qemu-img'
  2022-09-20  7:55     ` Jacob Hrbek
@ 2022-09-20 17:12       ` Liliana Marie Prikler
  2022-09-26 21:07         ` [bug#57809] [PATCH] gnome-boxes: Add 'qemu' as propagated-input Ludovic Courtès
  0 siblings, 1 reply; 20+ messages in thread
From: Liliana Marie Prikler @ 2022-09-20 17:12 UTC (permalink / raw)
  To: Jacob Hrbek; +Cc: 57809

Am Dienstag, dem 20.09.2022 um 07:55 +0000 schrieb Jacob Hrbek:
> > > [Krey format the code it looks like trash]
> 
> > > [Krey there is too much cosmetic noise it looks like trash]
It is admittedly our fault that we're advertising tools which don't do
their job, but it's not our fault that neither you nor your text editor
know how to match up parentheses with indentation levels.




^ permalink raw reply	[flat|nested] 20+ messages in thread

* bug#57809: [PATCH] gnome-boxes: Add 'qemu' as propagated-input
  2022-09-19  4:09 ` [bug#57809] [PATCH] gnome-boxes: Patch 'qemu-img' path Jacob Hrbek
  2022-09-19 13:41   ` Maxime Devos
@ 2022-09-26 21:03   ` Ludovic Courtès
  1 sibling, 0 replies; 20+ messages in thread
From: Ludovic Courtès @ 2022-09-26 21:03 UTC (permalink / raw)
  To: Jacob Hrbek; +Cc: 57809-done

Hi,

Jacob Hrbek <kreyren@rixotstudio.cz> skribis:

> ---
>  gnu/packages/gnome.scm | 12 +++++++++---
>  1 file changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
> index d6d22f8eb1..ff5cb92907 100644
> --- a/gnu/packages/gnome.scm
> +++ b/gnu/packages/gnome.scm
> @@ -12131,9 +12131,14 @@ (define-public gnome-boxes
>                                                      #$output
>                                                      "/lib/gnome-boxes"))
>             #:phases #~(modify-phases %standard-phases
> -                        (add-after 'unpack 'disable-gtk-update-icon-cache
> -                          (lambda _
> -                            (setenv "DESTDIR" "/"))))))
> +                      (add-after 'unpack 'disable-gtk-update-icon-cache
> +                        (lambda _
> +                          (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"))))))))

Applied with minor tweaks and a commit log, thanks!|

Ludo’.




^ permalink raw reply	[flat|nested] 20+ messages in thread

* [bug#57809] [PATCH] gnome-boxes: Add 'qemu' as propagated-input
  2022-09-20 17:12       ` Liliana Marie Prikler
@ 2022-09-26 21:07         ` Ludovic Courtès
  0 siblings, 0 replies; 20+ messages in thread
From: Ludovic Courtès @ 2022-09-26 21:07 UTC (permalink / raw)
  To: Liliana Marie Prikler; +Cc: Jacob Hrbek, 57809

Hey Liliana,

Please let’s not try to find “whose fault it is”.  We’re all trying to
help.

In such a situation, my experience is that it is not only more pleasant
but also more efficient to apply the patch and those remaining tweaks by
yourself so the submitter can see what changes were made and learn from
it.

Thanks in advance,
Ludo’.




^ permalink raw reply	[flat|nested] 20+ messages in thread

end of thread, other threads:[~2022-09-26 21:08 UTC | newest]

Thread overview: 20+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-09-14 18:17 [bug#57809] [PATCH] gnome-boxes: Add 'qemu' as propagated-input Jacob Hrbek
2022-09-14 18:18 ` [bug#57810] " Jacob Hrbek
2022-09-15 13:03 ` [bug#57809] " Maxime Devos
2022-09-18 20:43 ` [bug#57809] [PATCH] gnome-boxes: Patch 'qemu-img' path Jacob Hrbek
2022-09-18 21:05   ` Jacob Hrbek
2022-09-18 21:14     ` Maxime Devos
2022-09-18 21:32       ` Jacob Hrbek
2022-09-18 21:50         ` Maxime Devos
2022-09-19  4:28           ` Jacob Hrbek
2022-09-19 13:35             ` Maxime Devos
2022-09-18 20:46 ` Jacob Hrbek
2022-09-18 20:47 ` [bug#57809] [PATCH] gnome-boxes: Add 'qemu' as propagated-input Jacob Hrbek
2022-09-19  4:09 ` [bug#57809] [PATCH] gnome-boxes: Patch 'qemu-img' path Jacob Hrbek
2022-09-19 13:41   ` Maxime Devos
2022-09-26 21:03   ` bug#57809: [PATCH] gnome-boxes: Add 'qemu' as propagated-input Ludovic Courtès
2022-09-19 21:36 ` [bug#57809] [PATCH] gnome-boxes: Fix path for 'qemu-img' Jacob Hrbek
2022-09-20  5:18   ` Liliana Marie Prikler
2022-09-20  7:55     ` Jacob Hrbek
2022-09-20 17:12       ` Liliana Marie Prikler
2022-09-26 21:07         ` [bug#57809] [PATCH] gnome-boxes: Add 'qemu' as propagated-input Ludovic Courtès

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).