[neon/kde/kglobalaccel/Neon/release-lts] debian/patches: remove patches in branch

Jonathan Riddell null at kde.org
Sun Apr 11 23:33:31 BST 2021


Git commit 5b3938276756c2b4197cbf7dd0f2e05c33df607a by Jonathan Riddell.
Committed on 11/04/2021 at 22:33.
Pushed by jriddell into branch 'Neon/release-lts'.

remove patches in branch

D  +0    -2    debian/patches/series
D  +0    -38   debian/patches/uid.diff
D  +0    -28   debian/patches/uid2.diff

https://invent.kde.org/neon/kde/kglobalaccel/commit/5b3938276756c2b4197cbf7dd0f2e05c33df607a

diff --git a/debian/patches/series b/debian/patches/series
deleted file mode 100644
index 2ade593..0000000
--- a/debian/patches/series
+++ /dev/null
@@ -1,2 +0,0 @@
-uid.diff
-uid2.diff
diff --git a/debian/patches/uid.diff b/debian/patches/uid.diff
deleted file mode 100644
index 2243488..0000000
--- a/debian/patches/uid.diff
+++ /dev/null
@@ -1,38 +0,0 @@
-commit bbec74337b3186bc258bd5adf849b89191789535
-Author: Fabian Vogt <fabian at ritter-vogt.de>
-Date:   Fri Jan 29 11:31:54 2021 +0100
-
-    Don't let kglobalaccel run if KDE_SESSION_UID mismatches
-    
-    Otherwise shortcuts execute actions as the wrong user.
-
-diff --git a/src/runtime/main.cpp b/src/runtime/main.cpp
-index d2917ad..46506b3 100644
---- a/src/runtime/main.cpp
-+++ b/src/runtime/main.cpp
-@@ -10,6 +10,8 @@
- #include "kglobalacceld.h"
- #include "logging_p.h"
- 
-+#include <unistd.h>
-+
- #include <KAboutData>
- #include <KCrash>
- #include <KDBusService>
-@@ -57,6 +59,16 @@ extern "C" Q_DECL_EXPORT int main(int argc, char **argv)
-         return 0;
-     }
- 
-+    // It's possible that kglobalaccel gets started as the wrong user by
-+    // accident, e.g. kdesu dolphin leads to dbus activation. It then installs
-+    // its grabs and the actions are run as the wrong user.
-+    bool isUidset = false;
-+    const int sessionuid = qEnvironmentVariableIntValue("KDE_SESSION_UID", &isUidset);
-+    if(isUidset && static_cast<uid_t>(sessionuid) != getuid()) {
-+        qCWarning(KGLOBALACCELD) << "kglobalaccel running as wrong user, exiting.";
-+        return 0;
-+    }
-+
-     KDBusService service(KDBusService::Unique);
- 
-     app.setQuitOnLastWindowClosed(false);
diff --git a/debian/patches/uid2.diff b/debian/patches/uid2.diff
deleted file mode 100644
index 767a74d..0000000
--- a/debian/patches/uid2.diff
+++ /dev/null
@@ -1,28 +0,0 @@
-commit 33bd15697220f8a1223117fee8b03506fe87d912
-Author: Fabian Vogt <fabian at ritter-vogt.de>
-Date:   Fri Mar 26 13:48:01 2021 +0100
-
-    Only use unistd/getuid when available
-    
-    To fix build on Windows.
-
-Index: kglobalaccel/src/runtime/main.cpp
-===================================================================
---- kglobalaccel.orig/src/runtime/main.cpp
-+++ kglobalaccel/src/runtime/main.cpp
-@@ -75,6 +75,7 @@ extern "C" Q_DECL_EXPORT int main(int ar
-     }
- #endif
- 
-+#ifdef Q_OS_UNIX
-     // It's possible that kglobalaccel gets started as the wrong user by
-     // accident, e.g. kdesu dolphin leads to dbus activation. It then installs
-     // its grabs and the actions are run as the wrong user.
-@@ -84,6 +85,7 @@ extern "C" Q_DECL_EXPORT int main(int ar
-         qCWarning(KGLOBALACCELD) << "kglobalaccel running as wrong user, exiting.";
-         return 0;
-     }
-+#endif
- 
-     KDBusService service(KDBusService::Unique);
- 


More information about the Neon-commits mailing list