GNU bug report logs - #75966
[PATCH] gnu: qtwebengine: Set default path for QtWebEngineProcess.

Previous Next

Package: guix-patches;

Reported by: iyzsong <at> envs.net

Date: Fri, 31 Jan 2025 10:32:01 UTC

Severity: normal

Tags: patch

Full log


View this message in rfc822 format

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: iyzsong <at> envs.net
Cc: 宋文武 <iyzsong <at> member.fsf.org>, 75966 <at> debbugs.gnu.org
Subject: [bug#75966] [PATCH] gnu: qtwebengine: Set default path for QtWebEngineProcess.
Date: Mon, 03 Feb 2025 23:37:17 +0900
Hi,

iyzsong <at> envs.net writes:

> From: 宋文武 <iyzsong <at> member.fsf.org>
>
> Since qtwebengine and qtbase are not installed into the same prefix,
> the default path for QtWebEngineProcess won't work.  Fix it so that
> we no longer need to set QTWEBENGINEPROCESS_PATH anymore.
>
> * gnu/packages/qt.scm (qtwebengine-5, qtwebengine)[arguments]:
> Substitute 'QLibraryInfo::LibraryExecutablesPath' in 'substitute-source phase.
> [native-search-paths]: Remove QTWEBENGINEPROCESS_PATH.
> * guix/build/qt-utils.scm (variables-for-wrapping): Remove
> QTWEBENGINEPROCESS_PATH.
>
> Change-Id: Ie0dfaf8c2355a679e2a3ddeacf09654830f9ab2f

Reviewed-by: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>

I note that given then change to qt-utils, that's a Qt world rebuild
change, and should go to a qt-team branch.  Perhaps we could bunch a qt
upgrade on top if time permits; we have a manifest that makes this
rather simple (unless there are new test failures...).  See
etc/teams/qt-team/qt-manifest.scm for example, and the comment inside.

-- 
Thanks,
Maxim




This bug report was last modified 136 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.