Hello community, here is the log from the commit of package plasma5-workspace for openSUSE:Factory checked in at 2017-04-30 21:14:56 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/plasma5-workspace (Old) and /work/SRC/openSUSE:Factory/.plasma5-workspace.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "plasma5-workspace" Sun Apr 30 21:14:56 2017 rev:63 rq:491711 version:5.9.5 Changes: -------- --- /work/SRC/openSUSE:Factory/plasma5-workspace/plasma5-workspace.changes 2017-04-17 10:24:33.061171258 +0200 +++ /work/SRC/openSUSE:Factory/.plasma5-workspace.new/plasma5-workspace.changes 2017-04-30 21:15:05.815153860 +0200 @@ -1,0 +2,39 @@ +Thu Apr 27 21:06:41 UTC 2017 - [email protected] + +- Add upstream patch revert-autostart-krun.patch: + * Fix Plasma start if a non existent executable was supplied as + autostart script / program + +------------------------------------------------------------------- +Tue Apr 25 18:58:06 CEST 2017 - [email protected] + +- Update to 5.9.5 + * New bugfix release + * For more details please see: + * https://www.kde.org/announcements/plasma-5.9.5.php +- Changes since 5.9.4: + * Update the ungrab mouse hack for Qt5.8 + * use KProcess instead of QProcess to launch apps in ksmserver + * launch autostart apps in ksmserver using KRun, not QProcess (kde#369391, kde#370528) + * recognize the WM even if given with a full path + * Don't remove other applets in SystemTray::cleanupTask + * [Media Controller] Enforce PlainText format + * [Media Controller] Enforce PlainText format + * Fix off-by-one. + * [Media Controller] Use double for length instead of int (kde#377623) + * When deleting the panel the notifications applet is in, the containment is being destroyed but the corona is still there and rightfully emits availableScreenRectChanged. This will cause us to crash when we try to access the no-longer existing containment. + * Fix losing wallpaper selection when clicking "Apply". + * Correctly handle when a new primary screen displaces the old + * Fix TasksModel.anyTaskDemandsAttention not updating on window closure. (kde#378254) + * [Windowed Widgets Runner] Fix mimeDataForMatch + * Connect aboutToHide signal from QMenu to relevant libdbusmenu-qt slot (kde#375053) + * [Calculator Runner] Use "approximate" approximation mode (kde#277011) + * [OSD] Allow disabling OSD through config file + +------------------------------------------------------------------- +Sun Apr 16 11:17:50 UTC 2017 - [email protected] + +- No longer generate org.kde.plasma.private.systemtray.appdata.xml manually, + bug fixed in kpackage meanwhile + +------------------------------------------------------------------- Old: ---- plasma-workspace-5.9.4.tar.xz New: ---- plasma-workspace-5.9.5.tar.xz revert-autostart-krun.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ plasma5-workspace.spec ++++++ --- /var/tmp/diff_new_pack.u69bky/_old 2017-04-30 21:15:06.942994861 +0200 +++ /var/tmp/diff_new_pack.u69bky/_new 2017-04-30 21:15:06.946994298 +0200 @@ -20,7 +20,7 @@ %bcond_without lang Name: plasma5-workspace -Version: 5.9.4 +Version: 5.9.5 Release: 0 # Full Plasma 5 version (e.g. 5.9.4) %{!?_plasma5_bugfix: %global _plasma5_bugfix %{version}} @@ -39,6 +39,8 @@ # PATCH-FIX-OPENSUSE 0001-Ignore-default-sddm-face-icons.patch boo#1001364 [email protected] -- Ignore default sddm face icons Patch2: 0001-Ignore-default-sddm-face-icons.patch # PATCHES 100-200 and above are from upstream 5.9 branch +# PATCH-FIX-UPSTREAM +Patch100: revert-autostart-krun.patch # PATCHES 201-300 and above are from upstream master/5.10 branch # PATCH-FIX-UPSTREAM Patch201: 0001-Some-UI-fixes-for-logout-dialog.patch @@ -214,6 +216,7 @@ %patch0 -p1 %patch1 -p1 %patch2 -p1 +%patch100 -p1 %patch201 -p1 %build @@ -237,10 +240,6 @@ rm -rfv %{buildroot}%{_kf5_sharedir}/xsessions rm -rfv %{buildroot}%{_kf5_sharedir}/wayland-sessions %fdupes %{buildroot}/%{_prefix} - # Workaround for boo#1024326, kde#376394 - grep "<summary>" %{buildroot}%{_kf5_appstreamdir}/org.kde.plasma.private.systemtray.appdata.xml && exit 1 # Fixed now? - cp %{buildroot}%{_kf5_appstreamdir}/org.kde.plasma.systemtray.appdata.xml %{buildroot}%{_kf5_appstreamdir}/org.kde.plasma.private.systemtray.appdata.xml - sed -i 's/org\.kde\.plasma\.systemtray/kde.plasma.private.systemtray/g' %{buildroot}%{_kf5_appstreamdir}/org.kde.plasma.private.systemtray.appdata.xml %post -p /sbin/ldconfig ++++++ plasma-workspace-5.9.4.tar.xz -> plasma-workspace-5.9.5.tar.xz ++++++ /work/SRC/openSUSE:Factory/plasma5-workspace/plasma-workspace-5.9.4.tar.xz /work/SRC/openSUSE:Factory/.plasma5-workspace.new/plasma-workspace-5.9.5.tar.xz differ: char 26, line 1 ++++++ revert-autostart-krun.patch ++++++ >From ea3f87c5df0251838da71c473fd7b790c932d8b0 Mon Sep 17 00:00:00 2001 From: David Edmundson <[email protected]> Date: Thu, 27 Apr 2017 15:52:39 +0100 Subject: Revert "launch autostart apps in ksmserver using KRun" KRun::runApplication will show blocking error dialogs if it fails to find the executable This means we don't autostart the next app, which could be fatal if it comes before... Summary: ...kwin/plasma We shouldn't be having blocking calls in ksmserver it can deadlock And even in the best case we'd still end up blocking ksplash for 30 seconds We then port to KProcess which was part of the motivation behind the patch as it has better stdout handling This reverts commit 0f19e92f3e85d064de9cebf280fa8e085485c2e0. BUG: 379254 Also added port of autostarting applications to KProcess It has better stdout handling Test Plan: Logged in, still got my main session Reviewers: #plasma, mart Reviewed By: mart Subscribers: plasma-devel Tags: #plasma Differential Revision: https://phabricator.kde.org/D5618 --- ksmserver/startup.cpp | 17 ++++++++++++++--- 1 file changed, 14 insertions(+), 3 deletions(-) diff --git a/ksmserver/startup.cpp b/ksmserver/startup.cpp index e51278b..767116d 100644 --- a/ksmserver/startup.cpp +++ b/ksmserver/startup.cpp @@ -68,9 +68,9 @@ CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. #include <kconfig.h> #include <kconfiggroup.h> +#include <kio/desktopexecparser.h> #include <KSharedConfig> #include <kprocess.h> -#include <krun.h> #include <KNotifyConfig> #include <KService> @@ -679,8 +679,19 @@ void KSMServer::slotAutoStart() } KService service(serviceName); qCInfo(KSMSERVER) << "Starting autostart service " << serviceName; - if( !KRun::runApplication( service, QList<QUrl>(), nullptr, KRun::RunFlags(), QString(), "0" )) - qCInfo(KSMSERVER) << "autostart service" << serviceName << "failed to launch"; + auto p = new KProcess(this); + auto arguments = KIO::DesktopExecParser(service, QList<QUrl>()).resultingArguments(); + if (arguments.isEmpty()) { + qCInfo(KSMSERVER) << "failed to parse" << serviceName << "for autostart"; + continue; + } + auto program = arguments.takeFirst(); + p->setProgram(program, arguments); + p->start(); + connect(p, static_cast<void (QProcess::*)(int)>(&QProcess::finished), [p](int exitCode) { + qCInfo(KSMSERVER) << "autostart service" << p->program() << "finished with exit code " << exitCode; + p->deleteLater(); + }); } while (true); // Loop till we find a service that we can start. } -- cgit v0.11.2
