Accepting request 957246 from home:Vogtinator:plasma5.24

- Add patch to fix log output of some processes (boo#1196206):
  * 0001-Ignore-default-sddm-face-icons.patch

OBS-URL: https://build.opensuse.org/request/show/957246
OBS-URL: https://build.opensuse.org/package/show/KDE:Frameworks5/plasma5-workspace?expand=0&rev=631
This commit is contained in:
Christophe Giboudeaux 2022-02-24 11:03:39 +00:00 committed by Git OBS Bridge
parent 737d8d8201
commit 0ff586d82a
3 changed files with 58 additions and 0 deletions

View File

@ -0,0 +1,50 @@
From b525767c86cd90b21d5079a4642f01b64941fd79 Mon Sep 17 00:00:00 2001
From: Fabian Vogt <fabian@ritter-vogt.de>
Date: Wed, 23 Feb 2022 22:44:28 +0100
Subject: [PATCH] startkde: Forward stdout/stderr of started processes
QProcess::start defaults to capturing all output, but that got lost as it is
never read. Set QProcess::ForwardedChannels explicitly where it was missing,
so that output ends up in the correct place again.
(cherry picked from commit d1155030afb5f94fc81e7afc4d1480d21d7d73f9)
---
startkde/plasma-session/startup.cpp | 2 ++
startkde/startplasma.cpp | 1 +
2 files changed, 3 insertions(+)
diff --git a/startkde/plasma-session/startup.cpp b/startkde/plasma-session/startup.cpp
index 246d88b35..9078c1f16 100644
--- a/startkde/plasma-session/startup.cpp
+++ b/startkde/plasma-session/startup.cpp
@@ -246,6 +246,7 @@ bool Startup::startDetached(const QString &program, const QStringList &args)
bool Startup::startDetached(QProcess *process)
{
+ process->setProcessChannelMode(QProcess::ForwardedChannels);
process->start();
const bool ret = process->waitForStarted();
if (ret) {
@@ -428,6 +429,7 @@ StartProcessJob::StartProcessJob(const QString &process, const QStringList &args
{
m_process->setProgram(process);
m_process->setArguments(args);
+ m_process->setProcessChannelMode(QProcess::ForwardedChannels);
auto env = QProcessEnvironment::systemEnvironment();
env.insert(additionalEnv);
m_process->setProcessEnvironment(env);
diff --git a/startkde/startplasma.cpp b/startkde/startplasma.cpp
index a68e7c023..bd848dda0 100644
--- a/startkde/startplasma.cpp
+++ b/startkde/startplasma.cpp
@@ -500,6 +500,7 @@ QProcess *setupKSplash()
KConfigGroup ksplashCfg = cfg.group("KSplash");
if (ksplashCfg.readEntry("Engine", QStringLiteral("KSplashQML")) == QLatin1String("KSplashQML")) {
p = new QProcess;
+ p->setProcessChannelMode(QProcess::ForwardedChannels);
p->start(QStringLiteral("ksplashqml"), {ksplashCfg.readEntry("Theme", QStringLiteral("Breeze"))});
}
}
--
2.34.0

View File

@ -1,3 +1,9 @@
-------------------------------------------------------------------
Thu Feb 24 09:39:31 UTC 2022 - Fabian Vogt <fabian@ritter-vogt.de>
- Add patch to fix log output of some processes (boo#1196206):
* 0001-Ignore-default-sddm-face-icons.patch
-------------------------------------------------------------------
Tue Feb 22 16:21:05 UTC 2022 - Fabian Vogt <fabian@ritter-vogt.de>

View File

@ -41,6 +41,8 @@ Source1: https://download.kde.org/stable/plasma/%{version}/plasma-workspa
Source2: plasma.keyring
%endif
Source3: xprop-kde-full-session.desktop
# PATCH-FIX-UPSTREAM
Patch1: 0001-startkde-Forward-stdout-stderr-of-started-processes.patch
# PATCHES 501-??? are PATCH-FIX-OPENSUSE
Patch501: 0001-Use-qdbus-qt5.patch
Patch502: 0001-Ignore-default-sddm-face-icons.patch