Subject: bug#38926: closed (Re: [bug#39004] [PATCH] gnu: pcmanfm-qt: fix open file issue)

From: [email-masked] (GNU bug Tracking System)
Date: Sat, 11 Jan 2020 20:29:02 +0000

Your bug report

#38926: pcmanfm-qt unable to open files by double click

which was filed against the guix package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
38926: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=38926
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: [bug#39004] [PATCH] gnu: pcmanfm-qt: fix open file issue
From: Marius Bakke <[email-masked]>
To: Reza Alizadeh Majd <[email-masked]>, [email-masked]
Cc: [email-masked]
Date: Sat, 11 Jan 2020 21:28:42 +0100
List-Id: <debbugs-submit.debbugs.gnu.org>

Hi Reza,

"Reza Alizadeh Majd" <[email-masked]> writes:

> it could be great to have this patch merged, if there is no other issues 
>  existed with this change. 

I've applied the patch, with minor cosmetic adjustments (fixing typo,
naming the input "glib:bin" as we usually do), as well as a commit
message following our guidelines.

Thanks for the bug report and fix!

Subject: pcmanfm-qt unable to open files by double click
From: "Reza Alizadeh Majd" <[email-masked]>
To: [email-masked]
Date: Sat, 04 Jan 2020 23:54:59 +0330
List-Id: <debbugs-submit.debbugs.gnu.org>

Hi, 

there is an issue with `pcmanfm-qt`. when we try to open a known file using
double click, we receive following error:

--8<---------------cut here---------------start------------->8---
Failed to execute child process “gio-launch-desktop” (No such file or directory)
--8<---------------cut here---------------end--------------->8---

since `gio-launch-desktop` is located in `bin` output of `glib` package,
adding that as a propagated input for `pcmanfm-qt` fixes this issue.

--8<---------------cut here---------------start------------->8---
diff --git a/gnu/packages/lxqt.scm b/gnu/packages/lxqt.scm
index 5a3708e30a..2337daa7c3 100644
--- a/gnu/packages/lxqt.scm
+++ b/gnu/packages/lxqt.scm
@@ -991,6 +991,8 @@ components to build desktop file managers which belongs to LXDE.")
      `(("pkg-config" ,pkg-config)
        ("qttools" ,qttools)
        ("lxqt-build-tools" ,lxqt-build-tools)))
+    (propagated-inputs 
+      `(("glib" ,glib "bin")))
     (arguments
      '(#:tests? #f                      ; no tests
        #:phases
--8<---------------cut here---------------end--------------->8---

is this change is acceptable? if so I can submit a regarding patch to
`guix-patches` mailing list.

--
Regards
Reza Alizadeh Majd
PantherX Team