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

Clarify exit command variants #868

Merged
merged 3 commits into from
Nov 14, 2024
Merged

Clarify exit command variants #868

merged 3 commits into from
Nov 14, 2024

Conversation

Vladimir-csp
Copy link
Contributor

Resort to terminate-user only in case of problems with less blanket options.

Resort to `terminate-users` only in case of problems.
@Ghosthree3
Copy link

Ghosthree3 commented Nov 14, 2024

Typo, boud.

How does uwsm stop compare with loginctl terminate-session "$XDG_SESSION_ID"? Will it kill all active graphical session units (ie. if I issue it in tty2 it will kill the tty1 Hyprland instance as well)?
The description on the project page is unclear as to the scope of its effect, but given that the followup option wildcard stops all wayland-wm@* services and says it is effectively the same as uwsm stop I'm assuming it does.

The reason I bring this up is because I think it is unintuitive to the user that issuing an exit command in Hyprland will in fact kill all existing Hyprland (or Wayland) sessions.

@Vladimir-csp
Copy link
Contributor Author

Systemd-managed system assumes there is only one graphical session per user. UWSM operates in this paradigm, you will not be able to launch more than one wayland-wm@.service.

uwsm stop, systemctl --user stop wayland-wm@.service, systemctl --user start wayland-session-shutdown.target are qeuivalent.

loginctl terminate-session "$XDG_SESSION_ID" produces the same result if login session is bound to graphical session.

@Ghosthree3
Copy link

That makes sense, thank you. I suppose additional instances will just have to be started the traditional way, without uwsm.

@vaxerski vaxerski requested a review from fufexan November 14, 2024 20:42
@fufexan fufexan merged commit 052b616 into hyprwm:main Nov 14, 2024
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.

3 participants