Skip to content
This repository has been archived by the owner on Nov 1, 2021. It is now read-only.

xwayland: don't use SIGUSR1 #2154

Closed
emersion opened this issue May 4, 2020 · 0 comments · Fixed by #2504
Closed

xwayland: don't use SIGUSR1 #2154

emersion opened this issue May 4, 2020 · 0 comments · Fixed by #2504
Labels
enhancement xwayland

Comments

@emersion
Copy link
Member

emersion commented May 4, 2020

Right now to signal the compositor that Xwayland has started, we use SIGUSR1:

wlroots/xwayland/xwayland.c

Line 355 in 46c83cb

kill(ppid, SIGUSR1);

We could use a pipe instead. This would fix quite a few compositor issues with global state.
@emersion emersion mentioned this issue May 25, 2020
Closed
@dumanovskis dumanovskis mentioned this issue Oct 1, 2020
Closed
dominikh added a commit to dominikh/wlroots that referenced this issue Nov 30, 2020
@dominikh
xwayland: use pipe instead of SIGUSR1 to signal readiness
9078f5d 
Closes: swaywm#2154
@dominikh dominikh mentioned this issue Nov 30, 2020
Merged
emersion pushed a commit to dominikh/wlroots that referenced this issue Dec 7, 2020
@dominikh @emersion
xwayland: use pipe instead of SIGUSR1 to signal readiness
ab2cbee 
Closes: swaywm#2154
emersion pushed a commit to dominikh/wlroots that referenced this issue Dec 7, 2020
@dominikh @emersion
xwayland: use pipe instead of SIGUSR1 to signal readiness
970218a 
Closes: swaywm#2154
emersion pushed a commit that referenced this issue Dec 7, 2020
@dominikh @emersion
xwayland: use pipe instead of SIGUSR1 to signal readiness
431ec52 
Closes: #2154
udfn pushed a commit to udfn/wlroots that referenced this issue Dec 8, 2020
@dominikh @udfn
xwayland: use pipe instead of SIGUSR1 to signal readiness
d334eab 
Closes: swaywm#2154
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement xwayland
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

xwayland: use pipe instead of SIGUSR1 to signal readiness dominikh/wlroots
1 participant
@emersion