From: phodina via Guix-patches via <guix-patches@gnu.org>
To: Marius Bakke <marius@gnu.org>
Cc: maxim.cournoyer@gmail.com, ludo@gnu.org, mail@brendan.scot,
leo@famulari.name, marekpasnikowski@protonmail.com,
h.goebel@crazy-compilers.com, 57608@debbugs.gnu.org
Subject: [bug#57608] Upstreaming KDE Plasma and rest of packages
Date: Tue, 27 Dec 2022 21:15:55 +0000 [thread overview]
Message-ID: <X3go74odFaq331z5JbwC40xo6HkXw_5_aR6wB1_FX1nrpVHL8oFOE8QKK7I0yAH4ra1rAV06OWQCqcNvh9ZuFVQPUxZ9Jhro7QQ51AmRgG0=@protonmail.com> (raw)
In-Reply-To: <87wn7nz28p.fsf@gnu.org>
[-- Attachment #1: Type: text/plain, Size: 1969 bytes --]
Hi Marius,
Sent with Proton Mail secure email.
------- Original Message -------
On Tuesday, November 22nd, 2022 at 11:56 AM, Marius Bakke <marius@gnu.org> wrote:
> > Is this issue somewhere described? I've lately used more tarballs to git download on other packages.
> > Is the issue connected with any Gitlab/Forge or just the KDE and the release scripts?
>
>
> We have this problem with all the forges. There have been many-a bug
> report about it, and the linter also warns about it (apparently for
> GitHub only, that could be improved).
Thanks for the explanation. I'll keep that in mind.
>
> > > * dropped the !! commits
> > > * Minor tweaks to synopses and descriptions
> >
> > My apologies as I noticed there were few packages with wrong license or missing description.
>
>
> Oh, I did not license audit these packages as I assumed all the KDE
> stuffs were using the same license. Can you submit patches to update
> the licenses?
I've tried to follow the licenses as submitted by the individual packages. I'm currently in process
of double checking.
I found out that the kinfocenter and systemsettings were not behaving correctly - no widgets were
rendered due to missing QML modules. The attached patch solves this.
I have some additional patches for kwin and plasma-workspace but they are not yet enough to launch
KDE desktop.
Also one "small" problem. As I switched to sway to test apps running under Wayland I noticed a pretty
important issue. None of the programs launched were running under Wayland - they ran in XWayland.
Is this known issue?
Do we need to modify the qt-build-system or cmake-build-system? Or shall we add wrap phase for each
package to add the correct path for qtwayland package in env variable QT_QPA_PLATFORM_PLUGIN_PATH?
```
$ echo $XDG_SESSION_TYPE
wayland
$ konsole
qt.qpa.plugin: Could not find the Qt platform plugin "wayland" in ""
```
----
Petr
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-system-settings-Add-phase-wrap-executable-and-ad.patch --]
[-- Type: text/x-patch; name=0001-gnu-system-settings-Add-phase-wrap-executable-and-ad.patch, Size: 3419 bytes --]
From 063ea2790d55780943a0bd07be64421106b22d4d Mon Sep 17 00:00:00 2001
From: Petr Hodina <phodina@protonmail.com>
Date: Mon, 26 Dec 2022 10:50:19 +0100
Subject: [PATCH] gnu: system-settings: Add phase 'wrap-executable and
additional inputs.
* gnu/packages/kde-plasma.scm (system-settings)[inputs]: Add qtquickcontrols-5.
[arguments]: Add phase 'wrap-executable.
[inputs]: Add kcoreaddons, kdeclarative, knewstuff, kuserfeedback,
qtgraphicaleffects and qtquickcontrols2-5.
diff --git a/gnu/packages/kde-plasma.scm b/gnu/packages/kde-plasma.scm
index 89405bacaa..157ac3e1dd 100644
--- a/gnu/packages/kde-plasma.scm
+++ b/gnu/packages/kde-plasma.scm
@@ -2495,9 +2495,34 @@ (define-public system-settings
(base32
"0n7mf6ygi8fgn1m6pk2fadnqj1h58mxqni3h19xbi373wfypq5fl"))))
(build-system qt-build-system)
+ (arguments
+ (list #:phases #~(modify-phases %standard-phases
+ (add-after 'install 'wrap-executable
+ (lambda* (#:key inputs #:allow-other-keys)
+ (let ((out #$output)
+ (qml "/lib/qt5/qml"))
+ (wrap-program (string-append out
+ "/bin/systemsettings5")
+ `("QML2_IMPORT_PATH" ":" prefix
+ (,(string-append out qml)
+ ,@(map (lambda (i)
+ (string-append
+ (assoc-ref inputs i) qml))
+ '("kirigami"
+ "kcoreaddons"
+ "kdeclarative"
+ "knewstuff"
+ "kuserfeedback"
+ "plasma-workspace"
+ "qtdeclarative"
+ "qtgraphicaleffects"
+ "qtquickcontrols"
+ "qtquickcontrols2")))))))))))
(native-inputs (list extra-cmake-modules))
(inputs (list kauth
kcrash
+ kcoreaddons
+ kdeclarative
kitemviews
kitemmodels
kcmutils
@@ -2511,14 +2536,19 @@ (define-public system-settings
kdbusaddons
kconfig
kpackage
+ knewstuff
kactivities
kactivities-stats
kguiaddons
kirigami
knotifications
krunner
+ kuserfeedback
plasma-workspace
- qtdeclarative-5))
+ qtdeclarative-5
+ qtgraphicaleffects
+ qtquickcontrols-5
+ qtquickcontrols2-5))
(synopsis "Control center to configure Plasma Desktop")
(description "This package provides configuration UI for Plasma Desktop.")
(home-page "https://invent.kde.org/plasma/systemsettings")
--
2.38.1
next prev parent reply other threads:[~2022-12-27 21:17 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-09-06 7:14 [bug#57608] Upstreaming KDE Plasma and rest of packages phodina via Guix-patches via
2022-09-19 17:50 ` phodina via Guix-patches via
2022-09-24 13:12 ` Ludovic Courtès
2022-09-24 13:15 ` Ludovic Courtès
2022-10-02 17:14 ` guix-patches--- via
2022-10-04 6:52 ` phodina via Guix-patches via
2022-10-04 11:07 ` guix-patches--- via
2022-10-04 14:01 ` phodina via Guix-patches via
2022-10-06 12:09 ` phodina via Guix-patches via
2022-10-06 18:29 ` kiasoc5 via Guix-patches via
2022-10-07 12:57 ` phodina via Guix-patches via
2022-10-07 14:09 ` Maxim Cournoyer
2022-10-10 13:03 ` guix-patches--- via
2022-11-21 21:11 ` Marius Bakke
2022-11-22 10:20 ` phodina via Guix-patches via
2022-11-22 10:56 ` Marius Bakke
2022-12-27 21:15 ` phodina via Guix-patches via [this message]
2022-12-28 0:48 ` Maxim Cournoyer
2022-10-29 11:59 ` Brendan Tildesley
2022-10-29 20:32 ` guix-patches--- via
2022-10-29 23:49 ` Brendan Tildesley
2022-10-31 7:42 ` phodina via Guix-patches via
2023-09-02 4:23 ` bug#57608: " 宋文武 via Guix-patches via
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='X3go74odFaq331z5JbwC40xo6HkXw_5_aR6wB1_FX1nrpVHL8oFOE8QKK7I0yAH4ra1rAV06OWQCqcNvh9ZuFVQPUxZ9Jhro7QQ51AmRgG0=@protonmail.com' \
--to=guix-patches@gnu.org \
--cc=57608@debbugs.gnu.org \
--cc=h.goebel@crazy-compilers.com \
--cc=leo@famulari.name \
--cc=ludo@gnu.org \
--cc=mail@brendan.scot \
--cc=marekpasnikowski@protonmail.com \
--cc=marius@gnu.org \
--cc=maxim.cournoyer@gmail.com \
--cc=phodina@protonmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.