Re: [PATCH] emacs: notmuch-show: remove extraneous shell quoting

2016-09-16 Thread Matt Armstrong
Mark Walters writes: > Hi > > On Fri, 16 Sep 2016, Matt Armstrong wrote: >> Tomi, thanks for your reply asking for some motivation behind this >> patch. I can't reply directly to your message because, for some reason, >> it doesn't appear in my

Re: [PATCH] emacs: notmuch-show: remove extraneous shell quoting

2016-09-16 Thread Tomi Ollila
On Fri, Sep 16 2016, Matt Armstrong wrote: > Tomi, thanks for your reply asking for some motivation behind this > patch. I can't reply directly to your message because, for some reason, > it doesn't appear in my mailbox (I discovered your message while reading > the mail

Re: [PATCH] emacs: notmuch-show: remove extraneous shell quoting

2016-09-16 Thread Mark Walters
Hi On Fri, 16 Sep 2016, Matt Armstrong wrote: > Tomi, thanks for your reply asking for some motivation behind this > patch. I can't reply directly to your message because, for some reason, > it doesn't appear in my mailbox (I discovered your message while reading > the

Re: [PATCH] emacs: notmuch-show: remove extraneous shell quoting

2016-09-16 Thread Matt Armstrong
Tomi, thanks for your reply asking for some motivation behind this patch. I can't reply directly to your message because, for some reason, it doesn't appear in my mailbox (I discovered your message while reading the mail archive on notmuchmail.org). The code dealing with this quoting issue was

Re: [PATCH] emacs: notmuch-show: remove extraneous shell quoting

2016-09-14 Thread Tomi Ollila
On Wed, Sep 14 2016, Matt Armstrong wrote: > Remove shell quoting from notmuch-show--build-buffer. The args list > is ultimately passed to call-process, which passes them verbatim to > the subprocess (typically, notmuch). The quoting, intended for a > shell, is

Re: [PATCH] emacs: notmuch-show: remove extraneous shell quoting

2016-09-14 Thread Matt Armstrong
Sorry, ignore this one. id:1473834053-17591-1-git-send-email-marmstr...@google.com has some typo fixes to the commit message. ___ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch

[PATCH] emacs: notmuch-show: remove extraneous shell quoting

2016-09-14 Thread Matt Armstrong
Remove shell quoting from notmuch-show--build-buffer. The args list is ultimately passed to call-process, which passes them verbatim to the subprocess (typically, notmuch). The quoting, intended for a shell, is unnecessary and confusing. This code originally appeared in

[PATCH] emacs: notmuch-show: remove extraneous shell quoting

2016-09-14 Thread Matt Armstrong
Remove shell quoting from notmuch-show--build-buffer. The args list is ultimately passed to call-process, which passes them verbatim to the subprocess (typically, notmuch). The quoting, intended for a shell, are unnecessary and confusing. This code originally appeared in