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

Use logind "auto" session #63

Merged
merged 1 commit into from
Mar 31, 2020
Merged

Use logind "auto" session #63

merged 1 commit into from
Mar 31, 2020

Conversation

electrickite
Copy link
Contributor

Use the logind magic "auto" session to detect the session name. As a fallback, attempt to determine session name by PID.

This approach should succeed even when launched from a process not associated with a specific session, for example when started as a systemd user service.

@emersion
Copy link
Member

Where is this documented? I can't find any reference to the magical "auto" session ID.

@electrickite
Copy link
Contributor Author

@emersion The feature was merged into systemd last June. See:

Force pushed a new version now. It introduces the magic seat and session names "self" and "auto". The former is like the empty string, the latter introduces the new behaviour that looks for the display session if the caller is not part of a session.

However, please hold off on this PR. #48 also uses auto, so if that gets merged I'll close this one.

Copy link
Member

@emersion emersion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for finding a reference to the systemd pull request.

#48 also uses auto, so if that gets merged I'll close this one.

#48 should just do one thing: add support for inhibitors.

@emersion emersion merged commit ee4f8eb into swaywm:master Mar 31, 2020
@emersion
Copy link
Member

Thanks!

PalatinCoder added a commit to PalatinCoder/dotfiles that referenced this pull request Apr 29, 2021
This currently requires the development version of swayidle since the
required patch swaywm/swayidle#63 hasn't landed in a release as of
version 1.6.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants