Subject: bug#44771: closed (Re: [bug#44771] [PATCH] gnu: Properly use $QTWEBENGINEPROCESS_PATH)

From: [email-masked] (GNU bug Tracking System)
Date: Sat, 28 Nov 2020 16:36:02 +0000

Your bug report

#44771: [PATCH] gnu: Properly use $QTWEBENGINEPROCESS_PATH

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

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

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

Subject: Re: [bug#44771] [PATCH] gnu: Properly use $QTWEBENGINEPROCESS_PATH
From: Christopher Baines <[email-masked]>
To: Sergey Trofimov <[email-masked]>
Cc: [email-masked]
Date: Sat, 28 Nov 2020 16:34:58 +0000
List-Id: <debbugs-submit.debbugs.gnu.org>

Sergey Trofimov <[email-masked]> writes:

> ---
>  gnu/packages/ebook.scm        | 2 +-
>  gnu/packages/web-browsers.scm | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/gnu/packages/ebook.scm b/gnu/packages/ebook.scm
> index 1609a631b7..e80ad45383 100644
> --- a/gnu/packages/ebook.scm
> +++ b/gnu/packages/ebook.scm
> @@ -281,7 +281,7 @@
>                    (lambda (binary)
>                      (wrap-program binary
>                        ;; Make QtWebEngineProcess available.
> -                      `("QTWEBENGINEPROCESS_PATH" ":" =
> +                      `("QTWEBENGINEPROCESS_PATH" =
>                          ,(list (string-append
>                                  qtwebengine
>                                  "/lib/qt5/libexec/QtWebEngineProcess")))))
> diff --git a/gnu/packages/web-browsers.scm b/gnu/packages/web-browsers.scm
> index 51af66ef54..bf1d45e5cc 100644
> --- a/gnu/packages/web-browsers.scm
> +++ b/gnu/packages/web-browsers.scm
> @@ -538,7 +538,7 @@ interface.")
>                                        (assoc-ref inputs "qtwebengine")
>                                        "/lib/qt5/libexec/QtWebEngineProcess")))
>                 (wrap-program bin
> -                 `("QTWEBENGINEPROCESS_PATH" ":" prefix (,qt-process-path)))
> +                 `("QTWEBENGINEPROCESS_PATH" = (,qt-process-path)))
>                 #t))))))
>      (home-page "https://qutebrowser.org/")
>      (synopsis "Minimal, keyboard-focused, vim-like web browser")

Thanks, I split this in to two commits, wrote commit messages and pushed
to master in f4d139d02f0b0404540b74f3249de050b519b880.

Chris

Subject: [PATCH] gnu: Properly use $QTWEBENGINEPROCESS_PATH
From: Sergey Trofimov <[email-masked]>
To: [email-masked]
Date: Fri, 20 Nov 2020 22:19:06 +0100
List-Id: <debbugs-submit.debbugs.gnu.org>

I've noticed that installing supercollider breaks qutebrowser. It happens
because QTWEBENGINEPROCESS_PATH is not a search path, but rather a full path
to a file.  So supercollider puts this var into the user profile and then
qutebrowser's wrapper incorrectly concatenates it with the path coming from
the qtwebengine dependency. Calibre package has the same defect.