[PATCH v2] notmuch-emacs-mua: do not create a frame by default with --client
Jani Nikula
jani at nikula.org
Sat Apr 4 00:30:06 PDT 2015
Make the default behaviour for --client the same as emacsclient
default: do not create a new frame. Add a new option --create-frame,
passing the same option to emacsclient to create a frame.
---
v2: fix killing frame with --create-frame
---
doc/man1/notmuch-emacs-mua.rst | 14 +++++++++-----
notmuch-emacs-mua | 29 ++++++++++++++++++++---------
2 files changed, 29 insertions(+), 14 deletions(-)
diff --git a/doc/man1/notmuch-emacs-mua.rst b/doc/man1/notmuch-emacs-mua.rst
index 36b51cdc3b18..e39d9e5fba1f 100644
--- a/doc/man1/notmuch-emacs-mua.rst
+++ b/doc/man1/notmuch-emacs-mua.rst
@@ -18,10 +18,6 @@ Supported options for **notmuch-emacs-mua** include
``-h, --help``
Display help.
- ``--client``
- Use emacsclient, rather than emacs. This will start
- an emacs daemon process if necessary.
-
``-s, --subject=``\ <subject>
Specify the subject of the message.
@@ -38,7 +34,15 @@ Supported options for **notmuch-emacs-mua** include
Specify a file to include into the body of the message.
``--no-window-system``
- Even if a window system is available, use the current terminal
+ Even if a window system is available, use the current terminal.
+
+ ``--client``
+ Use **emacsclient**, rather than **emacs**. This will start
+ an emacs daemon process if necessary.
+
+ ``--create-frame``
+ When --client is specified, create a new frame instead of trying
+ to use the current Emacs frame.
``--print``
Output the resulting elisp to stdout instead of evaluating it.
diff --git a/notmuch-emacs-mua b/notmuch-emacs-mua
index 79714305b3e2..90f585b699b8 100755
--- a/notmuch-emacs-mua
+++ b/notmuch-emacs-mua
@@ -34,12 +34,12 @@ EMACS=${EMACS-emacs}
EMACSCLIENT=${EMACSCLIENT-emacsclient}
PRINT_ONLY=
+NO_WINDOW=
USE_EMACSCLIENT=
-CLIENT_TYPE="-c"
+CREATE_FRAME=
# The crux of it all: construct an elisp progn and eval it.
ELISP="(prog1 'done (require 'notmuch) (notmuch-mua-new-mail)"
-ELISP="${ELISP} (setq message-exit-actions (list #'save-buffers-kill-terminal))"
# Short options compatible with mutt(1).
while getopts :s:c:b:i:h opt; do
@@ -63,7 +63,7 @@ while getopts :s:c:b:i:h opt; do
opt=${opt%%=*}
;;
# Long options without arguments.
- --help|--print|--no-window-system|--client)
+ --help|--print|--no-window-system|--client|--create-frame)
;;
*)
echo "$0: unknown long option ${opt}, or argument mismatch." >&2
@@ -81,9 +81,6 @@ while getopts :s:c:b:i:h opt; do
--help|h)
exec man notmuch-emacs-mua
;;
- --client)
- USE_EMACSCLIENT="yes"
- ;;
--subject|s)
ELISP="${ELISP} (message-goto-subject) (insert \"${OPTARG}\")"
;;
@@ -103,7 +100,13 @@ while getopts :s:c:b:i:h opt; do
PRINT_ONLY=1
;;
--no-window-system)
- CLIENT_TYPE="-t"
+ NO_WINDOW="-nw"
+ ;;
+ --client)
+ USE_EMACSCLIENT="yes"
+ ;;
+ --create-frame)
+ CREATE_FRAME="--create-frame"
;;
*)
# We should never end up here.
@@ -122,6 +125,14 @@ for arg; do
ELISP="${ELISP} (message-goto-to) (insert \"${arg}, \")"
done
+# If reusing a frame in emacsclient, don't kill the buffer.
+if [ -n "$CREATE_FRAME" ]; then
+ ELISP="${ELISP} (setq message-exit-actions (list #'save-buffers-kill-terminal))"
+elif [ -z "$USE_EMACSCLIENT" ]; then
+ echo "$0: --create-frame is only applicable with --client." >&2
+ exit 1
+fi
+
# End progn.
ELISP="${ELISP})"
@@ -132,7 +143,7 @@ fi
if [ -n "$USE_EMACSCLIENT" ]; then
# Evaluate the progn.
- exec ${EMACSCLIENT} ${CLIENT_TYPE} -a '' --eval "${ELISP}"
+ exec ${EMACSCLIENT} ${CREATE_FRAME} ${NO_WINDOW} -a '' --eval "${ELISP}"
else
- exec ${EMACS} --eval "${ELISP}"
+ exec ${EMACS} ${NO_WINDOW} --eval "${ELISP}"
fi
--
2.1.4
More information about the notmuch
mailing list