Skip to content
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

session/logind: check for XDG_SESSION_ID first #1705

Merged
merged 1 commit into from May 22, 2019

Conversation

@rindeal
Copy link
Contributor

commented May 22, 2019

In order to support compositors running as systemd user units without display manager,
a mechanism for specifying session ID exactly must exist.

Checking for XDG_SESSION_ID mimics loginctl behaviour https://github.com/systemd/systemd/blob/e95be7def26c6c5feaf08a4135aa4f50c53263a8/src/login/loginctl.c#L856.

session/logind: check for XDG_SESSION_ID first
In order to support compositors running as systemd user units without display manager,
a mechanism for specifying session ID exactly must exist.

Checking for `XDG_SESSION_ID` mimics loginctl behaviour https://github.com/systemd/systemd/blob/e95be7def26c6c5feaf08a4135aa4f50c53263a8/src/login/loginctl.c#L856.

@rindeal rindeal force-pushed the rindeal:patch-2 branch from e51b3ff to 151b7d1 May 22, 2019

@ddevault ddevault requested a review from ascent12 May 22, 2019

@ascent12 ascent12 merged commit 0ab1bb6 into swaywm:master May 22, 2019

3 checks passed

builds.sr.ht: alpine.yml builds.sr.ht job completed successfully
Details
builds.sr.ht: archlinux.yml builds.sr.ht job completed successfully
Details
builds.sr.ht: freebsd.yml builds.sr.ht job completed successfully
Details
@ascent12

This comment has been minimized.

Copy link
Member

commented May 22, 2019

Thanks!

@rindeal rindeal deleted the rindeal:patch-2 branch May 22, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.