Re: Merging the pgtk branch

[Eli Zaretskii]

> Date: Mon, 02 Aug 2021 01:38:59 +0900 (JST)
> Cc: emacs-devel@gnu.org
> From: Yuuki Harano <masm+emacs@masm11.me>
> 
> 
> On Sun, 01 Aug 2021 11:53:16 +0300,
>       Eli Zaretskii <eliz@gnu.org> wrote:
> >> -install: all install-arch-indep install-etcdoc install-arch-dep 
> >> install-$(NTDIR) blessmail install-eln
> >> +install: all install-arch-indep install-etcdoc install-arch-dep 
> >> install-$(NTDIR) blessmail install-eln install-gsettings-schemas
> >>    @true
> > 
> > Does this mean gsettings-schemas will be installed by non-PGTK builds
> > as well?  If not, where are the conditions to prevent that?
> 
> Yes, it is installed if gsettings is enabled, even if non-PGTK builds.

Is that useful without the PGTK build?  What is that used for?

> >> --- a/lisp/server.el
> >> +++ b/lisp/server.el
> >> @@ -900,12 +900,17 @@ server-create-window-system-frame
> >>        )
> >  
> >>      (cond (w
> >> -           (server--create-frame
> >> -            nowait proc
> >> -            `((display . ,display)
> >> -              ,@(if parent-id
> >> -                    `((parent-id . ,(string-to-number parent-id))))
> >> -              ,@parameters)))
> >> +           (condition-case nil
> >> +               (server--create-frame
> >> +                nowait proc
> >> +                `((display . ,display)
> >> +                  ,@(if parent-id
> >> +                        `((parent-id . ,(string-to-number parent-id))))
> >> +                  ,@parameters))
> >> +             (error
> >> +              (server-log "Window system unsupported" proc)
> >> +              (server-send-string proc "-window-system-unsupported \n")
> >> +              nil)))
> > 
> > Why is this change needed?
> 
> emacsclient tries WAYLAND_DISPLAY, and DISPLAY if the first try is failed.
> The code catches the display connection failure, and returns the failure
> to emacsclient.

Hmm... what about window-systems that have neither WAYLAND_DISPLAY nor
DISPLAY? will this always fail?  I feel that I'm missing something
here: why wasn't this needed before?

> I'll answer the latter half questions later.

TIA.