bug#34114: 27.0.50: pdumper and themes with Emacs daemon

[Karl Otness <karl@karlotness.com>]

[-- Attachment #1: Type: text/plain, Size: 392 bytes --]

I might have exhausted those limits as well with a few other changes.
I've prepared and attached a patch that makes the change I described
above, but I'm not sure if it can be installed as-is without the
copyright paperwork.

Thanks,
Karl


On Fri, Jan 18, 2019 at 11:50 PM Daniel Colascione <dancol@dancol.org> wrote:
>
> Should fall under the de minimus exception to paperwork requirements

[-- Attachment #2: 0001-Initialize-faces-in-daemon-mode-with-pdumper-Bug-341.patch --]
[-- Type: text/x-patch, Size: 1171 bytes --]

From 6c85063851f83e51532013a354c0f1ce329d070a Mon Sep 17 00:00:00 2001
From: Karl Otness <karl@karlotness.com>
Date: Fri, 18 Jan 2019 23:56:44 +0000
Subject: [PATCH] Initialize faces in daemon mode with pdumper (Bug#34114)

* src/dispnew.c (init_display_interactive): Call init_faces_initial
in daemon mode.
---
 src/dispnew.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/dispnew.c b/src/dispnew.c
index 88783cd5da..226f4731cd 100644
--- a/src/dispnew.c
+++ b/src/dispnew.c
@@ -6033,6 +6033,10 @@ init_display_interactive (void)
     }
 #endif /* SIGWINCH */
 
+  /* Set up faces of the initial terminal frame.  */
+  if (!noninteractive && NILP (Vinitial_window_system))
+    init_faces_initial ();
+
   /* If running as a daemon, no need to initialize any frames/terminal,
      except on Windows, where we at least want to initialize it.  */
 #ifndef WINDOWSNT
@@ -6176,10 +6180,6 @@ init_display_interactive (void)
   }
 
   calculate_costs (XFRAME (selected_frame));
-
-  /* Set up faces of the initial terminal frame.  */
-  if (!noninteractive && NILP (Vinitial_window_system))
-    init_faces_initial ();
 }
 
 void
-- 
2.20.1