masm11 / emacs

Mirror of GNU Emacs
http://www.gnu.org/software/emacs/
GNU General Public License v3.0
198 stars 14 forks source link

emacsclient: check WAYLAND_DISPLAY #13

Closed lheckemann closed 4 years ago

lheckemann commented 4 years ago

This avoids emacsclient -c falling back to xwayland or terminal.

masm11 commented 4 years ago

Thank you, I have forgot emacsclient completely.

This PR's code ignores DISPLAY if WAYLAND_DISPLAY is set. I think it is not good very much.

I'll fix that.

masm11 commented 4 years ago

@lheckemann

Fixed. Temporarily merged your branch to my fix/emacsclient-ignores-wayland-display.

How about this branch?

lheckemann commented 4 years ago

Oh, so your change allows falling back to X if the wayland display exists but isn't reachable? Seems good :+1:

masm11 commented 4 years ago

Merged. Thanks.