notmuch-emacs-mua: escape $PWD (and cd always)
authorTomi Ollila <tomi.ollila@iki.fi>
Sat, 28 May 2016 11:06:04 +0000 (14:06 +0300)
committerDavid Bremner <david@tethera.net>
Sat, 11 Jun 2016 16:09:22 +0000 (13:09 -0300)
Escaping $PWD makes this work in directories like 'foo"bar'...

Cd'ing always makes the working directory to be consistent whether
--body option was used or not (when using emacsclient, but cd'ing
when using emacs does not cause any harm).

Note that documentation of `insert-file` expects programs to
call `insert-file-contents` instead. In our simple case
`insert-file` works better as it does some good checks that we'd
have to implement ourselves. Look lisp/files.el in emacs sources
for more information.

notmuch-emacs-mua

index 08be145b24a7f45ba7eecf76c76622c75f62f8f7..f9d83713b78958093e1fdd731596271ab3af3be4 100755 (executable)
@@ -39,8 +39,10 @@ USE_EMACSCLIENT=
 AUTO_DAEMON=
 CREATE_FRAME=
 
+escape -v pwd "$PWD"
+
 # The crux of it all: construct an elisp progn and eval it.
-ELISP="(prog1 'done (require 'notmuch) (notmuch-mua-new-mail)"
+ELISP="(prog1 'done (require 'notmuch) (cd \"$pwd\") (notmuch-mua-new-mail)"
 
 # Short options compatible with mutt(1).
 while getopts :s:c:b:i:h opt; do
@@ -95,7 +97,7 @@ while getopts :s:c:b:i:h opt; do
            ELISP="${ELISP} (message-goto-bcc) (insert \"${OPTARG}, \")"
            ;;
        --body|i)
-           ELISP="${ELISP} (message-goto-body) (cd \"${PWD}\") (insert-file \"${OPTARG}\")"
+           ELISP="${ELISP} (message-goto-body) (insert-file \"${OPTARG}\")"
            ;;
        --print)
            PRINT_ONLY=1