Skip to content

Support disabling the GTK SessionManagerDBus via environment variables #2056

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

HannesWell
Copy link
Member

For example this simplifies disabling the SessionManagerDBus in CI environments where no bus is available.

At our CI-server, which formally doesn't have an X-Server installed and uses xvnc I see the following warnings:

SWT SessionManagerDBus: Failed to connect to org.gnome.SessionManager: Failed to execute child process “dbus-launch” (No such file or directory)
SWT SessionManagerDBus: Failed to connect to org.xfce.SessionManager: Failed to execute child process “dbus-launch” (No such file or directory)

Setting the system-property org.eclipse.swt.internal.SessionManagerDBus.disable disables the SessionManagerDBus right from the start and prevents these warnings. But I have to set that system-property in the Maven/Tycho pom.xml as well as in all headless eclipse executions we run in our build.
It would be more convenient, if I can just set an environment property for the entire pipeline that would be used as proposed here.

Copy link
Contributor

github-actions bot commented Apr 24, 2025

Test Results

   539 files  ±0     539 suites  ±0   29m 59s ⏱️ + 1m 34s
 4 337 tests ±0   4 321 ✅ ±0   15 💤 ±0  1 ❌ ±0 
16 601 runs  ±0  16 463 ✅ ±0  137 💤 ±0  1 ❌ ±0 

For more details on these failures, see this check.

Results for commit cffb6a2. ± Comparison against base commit 788ba23.

♻️ This comment has been updated with latest results.

@HannesWell HannesWell force-pushed the session-manager-disable-env-variable branch 2 times, most recently from 73d4fbe to e0e6d15 Compare April 26, 2025 21:10
For example this simplifies disabling the SessionManagerDBus in CI
environments where no bus is available.
@akurtakov akurtakov force-pushed the session-manager-disable-env-variable branch from e0e6d15 to cffb6a2 Compare May 2, 2025 05:16
@akurtakov
Copy link
Member

Failing Windows test is unrelated. Thanks for this one!

@akurtakov akurtakov merged commit 8efc599 into eclipse-platform:master May 2, 2025
15 of 17 checks passed
@HannesWell HannesWell deleted the session-manager-disable-env-variable branch May 2, 2025 05:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants