From xemacs-m  Mon Mar 17 05:40:10 1997
Received: from elc1.dina.kvl.dk (elc1.dina.kvl.dk [130.225.40.228])
	by xemacs.org (8.8.5/8.8.5) with SMTP id FAA12967
	for <xemacs-beta@xemacs.org>; Mon, 17 Mar 1997 05:40:08 -0600 (CST)
Received: from zuse.dina.kvl.dk (zuse.dina.kvl.dk [130.225.40.245]) by elc1.dina.kvl.dk (8.6.12/8.6.4) with ESMTP id MAA21130; Mon, 17 Mar 1997 12:29:22 +0100
Received: (abraham@localhost) by zuse.dina.kvl.dk (8.6.12/8.6.4) id MAA13035; Mon, 17 Mar 1997 12:38:19 +0100
Sender: abraham@dina.kvl.dk
To: xemacs-beta@xemacs.org
Subject: Re: [ 19.15-b99 and 20.1-b7] build FAILURE on Solaris-2.5/X11R6.3/cc-4.0
References: <m2vi6si7nw.fsf@altair.xemacs.org> <vkhgibh8ud.fsf@cdc.noaa.gov> <m2afo222b4.fsf@altair.xemacs.org>
Organization: The Church of Emacs
X-Face: +kRV2]2q}lixHkE{U)mY#+6]{AH=yN~S9@IFiOa@X6?GM<U{B+4e{k79.Ya{~':DblFPCg$
 @60,BfLv2@SKZ19cMWK0/C'v;tM:|6B'R}U1rp6CL&kN({9<zF/V{:JCg27yC)9oZjeqcQawzKfiNL
 t9}`vjmK["dRQC/qGFQq"%u|Q`:6{"Rz}b(dnl_"3$Jtqimi>|8MBp/
From: Per Abrahamsen <abraham@dina.kvl.dk>
Date: 17 Mar 1997 12:38:19 +0100
In-Reply-To: Steven L Baur's message of 17 Mar 1997 03:19:43 -0800
Message-ID: <rj913magus.fsf@zuse.dina.kvl.dk>
Lines: 23
X-Mailer: Gnus v5.4.25/Emacs 19.34
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit


Steven L Baur <steve@miranova.com> writes:

> Comint.el now has a (defface ...) in it.  Apparently we cannot dump
> anything which calls defface at the outer level.  It looks like a
> custom limitation.  I agree with the comment `This should be allowed,
> somehow'.  Suggestions are welcome.  Per, did you put that in to avoid 
> a certain XEmacs coredump?

_defface_ initializes the global face attributes according to the
frame properties (device type, device class, background intensity) of
the selected frame when the first _defface_ declaration is evaluated.
I doubt you want to initialize them according to what happened to be
used at build time.

> I don't know what the right answer is.  It certainly isn't to 
> make some weird kludge to comint.el to avoid the defface at dump time
> but not runtime, I think.  This sort of thing should just work.

One could save all the _defface_ declarations that are dumped into a
variable, and then create the faces at startup from that variable.  If
you don't mind XEmacs startup becoming even slower, that is.

