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

Restart xwayland if it exits #226

Closed

Conversation

LogicalOverflow
Copy link
Contributor

@LogicalOverflow LogicalOverflow commented Apr 29, 2024

Needs some (manual) testing, but I am optimistic about that. I'll un-draft the PR once I tested it

Fixes #186

Needs more work and testing
Currently does not work with confines with no region. Also there *might* be a regression that prevents the mouse from moving between outputs, need to test that
@Ottatop
Copy link
Collaborator

Ottatop commented Apr 29, 2024

Can you target this PR against the pointer_constraints branch? Smithay recently updated the process of starting xwayland and this branch contains that update (which probably should've been a separate commit, I know). I can hold off on pushing commits until this PR is merged.

@LogicalOverflow LogicalOverflow changed the title Restart xwayland if it exists Restart xwayland if it exits Apr 30, 2024
@LogicalOverflow
Copy link
Contributor Author

Will do. I'll try to do that either today or tomorrow

@LogicalOverflow
Copy link
Contributor Author

Okay, with the Smithay changes, I don't think this is currently possible, as I can't see a way to get notified that Xwayland exited. I'll re-try implementing this, once there's a way to do that, see Smithay/smithay#1410

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.

Restart xwayland if it crashes
2 participants