* lisp/server.el (server-process-filter, server-return-error): Give

emacsclient time to shut down after receiving an error string.

* etc/NEWS: Document tweak to emacsclient exit status.

diff --git a/etc/NEWS b/etc/NEWS
index 62132847efce4a4e550a443d0e67bcf001f43288..4d2bbead518ee6cd295c3e6dbbe42c62531415a5 100644 (file)
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -66,10 +66,15 @@ obsolete in Emacs 23.)
 \f
 * Changes in Emacs 24.1
 
-** New emacsclient argument --parent-id ID can be used to open a
+** emacsclient changes
+
+*** New emacsclient argument --parent-id ID can be used to open a
 client frame in parent X window ID, via XEmbed.  This works like the
 --parent-id argument to Emacs.
 
+*** If emacsclient shuts down as a result of Emacs signalling an
+error, its exit status is 1.
+
 ** Completion can cycle, depending on completion-cycle-threshold.
 
 ** auto-mode-case-fold is now enabled by default.

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index 044db17a44c39b15046d07a48bdb84b37bf275b5..c25e49a8c8d8c643c7b33ce84e315b94f3421f7f 100644 (file)
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,8 @@
+2010-10-03  Chong Yidong  <cyd@stupidchicken.com>
+
+       * server.el (server-process-filter, server-return-error): Give
+       emacsclient time to shut down after receiving an error string.
+
 2010-10-02  Michael Albinus  <michael.albinus@gmx.de>
 
        * files.el (remote-file-name-inhibit-cache): New defcustom.

diff --git a/lisp/server.el b/lisp/server.el
index 0f1b0219a2c75011f5a5a2d33e569e5d7552b00f..e661f055e1a971852950af8f0a30ff855dd9ad2b 100644 (file)
--- a/lisp/server.el
+++ b/lisp/server.el
@@ -876,6 +876,9 @@ The following commands are accepted by the client:
       (server-log "Authentication failed" proc)
       (server-send-string
        proc (concat "-error " (server-quote-arg "Authentication failed")))
+      ;; Before calling `delete-process', give emacsclient time to
+      ;; receive the error string and shut down on its own.
+      (sit-for 1)
       (delete-process proc)
       ;; We return immediately
       (return-from server-process-filter)))
@@ -1129,6 +1132,9 @@ The following commands are accepted by the client:
      proc (concat "-error " (server-quote-arg
                              (error-message-string err))))
     (server-log (error-message-string err) proc)
+    ;; Before calling `delete-process', give emacsclient time to
+    ;; receive the error string and shut down on its own.
+    (sit-for 5)
     (delete-process proc)))
 
 (defun server-goto-line-column (line-col)