GNU bug report logs - #57257
[PATCH 00/31] Update LXQt to 1.1.0

Previous Next

Full log

View this message in rfc822 format

From: 宋文武 <iyzsong <at> envs.net>
To: Ricardo Wurmus <rekado <at> elephly.net>
Cc: 57257 <at> debbugs.gnu.org, iyzsong <at> member.fsf.org
Subject: [bug#57257] [PATCH 00/31] Update LXQt to 1.1.0
Date: Mon, 22 Aug 2022 08:55:03 +0800

Ricardo Wurmus <rekado <at> elephly.net> writes:

> Hi,
>
> overall this looks good to me.  Thank you for working on it!
>
> There are just a few questions:
>
> - does it really work to use relative file names?  Would it be possible
>   to test this roughly in a system test?
Yes, the relative file names are for install, at runtime most things
works as designed with XDG_DATA_DIRS and XDG_CONFIG_DIRS.

A system test could verify items in lxqt-config, will look it later...

>
> - you’re removing a lot of patch-source phases.  Why are these no longer
>   needed?

Yeah, I find that use relative file names in LXQtConfigVars.cmake remove
the need of patching other cmake files.
>
> - you’re also removing patch-translations-dir phases.  Do translations
>   still work after applying these changes?

Yes, at runtime they are found under XDG_DATA_DIRS, by
lxqttranslator.cpp:

*searchPath << XdgDirs::dataDirs(QL1C('/') + QL1S(LXQT_RELATIVE_SHARE_TRANSLATIONS_DIR));

> [...]
>
> Feel free to push this!

Okay, thanks for the review!

Previous Next

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