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.

Sign up for GitHub

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

Jump to bottom

Remove GetSessionByPID fallback #64

Open
emersion opened this issue Mar 31, 2020 · 0 comments · May be fixed by #133
Open

Remove GetSessionByPID fallback #64

emersion opened this issue Mar 31, 2020 · 0 comments · May be fixed by #133

Comments

@emersion
Copy link
Member

See #63

Requires a recent enough logind.
WhyNotHugo added a commit to WhyNotHugo/swayidle that referenced this issue Oct 27, 2022
@WhyNotHugo
Drop logind integration
08d1de9 
This integration is unwanted, as expressed in swaywm#117.

Additionally, the integration is not quite correct; to properly launch a
screen locker before sleeping, swayidle should set a suspend inhibitor
**at startup**, and upon the PrepareForSleep signal, it should start the
screen locker and then release the inhibitor lock. The existing
implementation is racey and may run the locker _after_ resuming from
suspend.

Fixes: swaywm#117
Closes: swaywm#127
Closes: swaywm#121
Closes: swaywm#64
Closes: swaywm#59
Closes: swaywm#125
Closes: swaywm#89
@WhyNotHugo WhyNotHugo linked a pull request Oct 27, 2022 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Drop logind integration WhyNotHugo/swayidle
1 participant
@emersion