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


Message #23 received at 75966 <at> debbugs.gnu.org (full text, mbox):

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: Re: bug#75966: [PATCH] gnu: qtwebengine: Set default path for
 QtWebEngineProcess.
Date: Sat, 01 Feb 2025 23:38:06 +0900
Hi,

宋文武 <iyzsong <at> envs.net> writes:

> Maxim Cournoyer <maxim.cournoyer <at> gmail.com> writes:
>
>> Could you mention what is the rationale in the commit message?  Is the
>> QTWEBENGINEPROCESS_PATH environment variable problematic is some ways
>> (perhaps mixing Qt5 and Qt6 qtwebengine?).
>
> It seems fine to mixing QtWebEngineProcess from Qt5 and Qt6, but the
> default path is wrong due to we have different install prefixes,
> normally there is no need to set QTWEBENGINEPROCESS_PATH ourself.
>
>>
>> Shouldn't we keep the native-search-path for it?  Perhaps not all
>> software are patched yet, and it shouldn't hurt?  Or would it defeat the
>> aim of this series?
>
> The rationale is to drop the wrapping for QTWEBENGINEPROCESS_PATH, as
> there is no need to use it if the default path is working from the beginning.
>
> v2 patches send, thanks.

Makes sense now, thank your for the explanation.

-- 
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.