GNU bug report logs - #18347
24.3.93; Incomplete splash screen display on Cygwin-w32 build

Previous Next

Full log

Message #8 received at 18347 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Ken Brown <kbrown <at> cornell.edu>
Cc: 18347 <at> debbugs.gnu.org
Subject: Re: bug#18347: 24.3.93;
 Incomplete splash screen display on Cygwin-w32 build
Date: Fri, 29 Aug 2014 09:25:01 +0300

> Date: Thu, 28 Aug 2014 17:51:06 -0400
> From: Ken Brown <kbrown <at> cornell.edu>
> 
> --- lisp/startup.el     2014-07-08 09:17:09 +0000
> +++ lisp/startup.el     2014-08-28 20:07:20 +0000
> @@ -1812,7 +1812,7 @@
>     (let (chosen-frame)
>       ;; MS-Windows needs this to have a chance to make the initial
>       ;; frame visible.
> -    (if (eq system-type 'windows-nt)
> +    (if (eq window-system 'w32)
>          (sit-for 0 t))
>       (dolist (frame (append (frame-list) (list (selected-frame))))
>         (if (and (frame-visible-p frame)

It is fundamentally wrong to use window-system the variable in such
circumstances (or almost any other).  Please use the function instead.

> Is this still OK for the native Windows build?  If so, is it OK to 
> install it in the release branch?

Yes and yes.  Although I'd urge you to try to figure out why this is
needed in the cygwin-w32 build (the native-build problem which led to
this code manifested itself in a rather different way, see bug#16014).

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.