diff options
author | Bernd Weimer <bernd.weimer@qt.io> | 2023-05-04 16:38:51 +0200 |
---|---|---|
committer | Bernd Weimer <bernd.weimer@qt.io> | 2023-05-11 09:18:36 +0200 |
commit | 6fc4bac54f4d3bfa72634a621f5ef74715329e64 (patch) | |
tree | 55cc74d490c0a9fd9890348cbf1a979a170ff52b | |
parent | c349729973c5e0d03698c4d3af66e5f313b32899 (diff) | |
download | qtapplicationmanager-6fc4bac54f4d3bfa72634a621f5ef74715329e64.tar.gz |
Limit WAYLAND_DISPLAY to control only client side
The WAYLAND_DISPLAY environment variable not only controlled what server
to connect to as a client in a nested setup, but also what server socket
was established. This led to many unusable configurations (either the
socket was already locked or not available). The server socket should be
provided via the CL or configuration file only.
Change-Id: I7fa54c7cc5b5f2baa49b8b68f8de4f98edfc94c0
Reviewed-by: Robert Griebl <robert.griebl@qt.io>
-rw-r--r-- | src/main-lib/configuration.cpp | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/src/main-lib/configuration.cpp b/src/main-lib/configuration.cpp index a689595b..40175ea7 100644 --- a/src/main-lib/configuration.cpp +++ b/src/main-lib/configuration.cpp @@ -1246,13 +1246,6 @@ QString Configuration::waylandSocketName() const if (!socketName.isEmpty()) return socketName; - const char *envName = "WAYLAND_DISPLAY"; - if (qEnvironmentVariableIsSet(envName)) { - socketName = qEnvironmentVariable(envName); - if (!QGuiApplication::platformName().startsWith(qSL("wayland")) || (socketName != qSL("wayland-0"))) - return socketName; - } - if (!m_data->wayland.socketName.isEmpty()) return m_data->wayland.socketName; |