Re: [PATCH v3 0/6] Make Emacs search use sexp format

diff --git a/be/44755d3d0abd06eab273ad59c889e1a7e676f4 b/be/44755d3d0abd06eab273ad59c889e1a7e676f4
new file mode 100644 (file)
index 0000000..33ea8e1
--- /dev/null
+++ b/be/44755d3d0abd06eab273ad59c889e1a7e676f4
@@ -0,0 +1,120 @@
+Return-Path: <tomi.ollila@iki.fi>\r
+X-Original-To: notmuch@notmuchmail.org\r
+Delivered-To: notmuch@notmuchmail.org\r
+Received: from localhost (localhost [127.0.0.1])\r
+       by olra.theworths.org (Postfix) with ESMTP id 37875429E3C\r
+       for <notmuch@notmuchmail.org>; Fri, 31 May 2013 23:50:51 -0700 (PDT)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: 0\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
+       autolearn=disabled\r
+Received: from olra.theworths.org ([127.0.0.1])\r
+       by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
+       with ESMTP id V-7egyRW-Lzp for <notmuch@notmuchmail.org>;\r
+       Fri, 31 May 2013 23:50:42 -0700 (PDT)\r
+Received: from guru.guru-group.fi (guru.guru-group.fi [46.183.73.34])\r
+       by olra.theworths.org (Postfix) with ESMTP id B17F4431FDE\r
+       for <notmuch@notmuchmail.org>; Fri, 31 May 2013 23:50:42 -0700 (PDT)\r
+Received: from guru.guru-group.fi (localhost [IPv6:::1])\r
+       by guru.guru-group.fi (Postfix) with ESMTP id 5EEF0100086\r
+       for <notmuch@notmuchmail.org>; Sat,  1 Jun 2013 09:50:40 +0300 (EEST)\r
+From: Tomi Ollila <tomi.ollila@iki.fi>\r
+To: notmuch@notmuchmail.org\r
+Subject: Re: [PATCH v3 0/6] Make Emacs search use sexp format\r
+In-Reply-To: <1370047208-12785-1-git-send-email-amdragon@mit.edu>\r
+References: <1370047208-12785-1-git-send-email-amdragon@mit.edu>\r
+User-Agent: Notmuch/0.15.2+115~g12cf6af (http://notmuchmail.org) Emacs/24.3.1\r
+       (x86_64-unknown-linux-gnu)\r
+X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
+       $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
+       !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
+Date: Sat, 01 Jun 2013 09:50:40 +0300\r
+Message-ID: <m2y5auz5mn.fsf@guru.guru-group.fi>\r
+MIME-Version: 1.0\r
+Content-Type: text/plain\r
+X-BeenThere: notmuch@notmuchmail.org\r
+X-Mailman-Version: 2.1.13\r
+Precedence: list\r
+List-Id: "Use and development of the notmuch mail system."\r
+       <notmuch.notmuchmail.org>\r
+List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
+List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
+List-Post: <mailto:notmuch@notmuchmail.org>\r
+List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
+List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
+X-List-Received-Date: Sat, 01 Jun 2013 06:50:51 -0000\r
+\r
+On Sat, Jun 01 2013, Austin Clements <amdragon@MIT.EDU> wrote:\r
+\r
+> This is v3 of id:1369934016-22308-1-git-send-email-amdragon@mit.edu.\r
+> This tweaks the shell invocation as suggested by Tomi and fixes two\r
+> comment typos pointed out by Mark.  It also adds a NEWS patch.  I'm\r
+> going to go ahead and mark this ready because of Tomi's and Mark's\r
+> reviews of v2.\r
+\r
++1\r
+\r
+Tomi\r
+\r
+\r
+>\r
+> The diff from v2 follows\r
+>\r
+> diff --git a/NEWS b/NEWS\r
+> index 80abd97..2a4bde6 100644\r
+> --- a/NEWS\r
+> +++ b/NEWS\r
+> @@ -66,6 +66,17 @@ notmuch-vim, but of course that is their decision.\r
+>  Emacs Interface\r
+>  ---------------\r
+>  \r
+> +Better handling of errors in search buffers\r
+> +\r
+> +  Instead of interleaving errors in search result buffers, search mode\r
+> +  now reports errors in the minibuffer.\r
+> +\r
+> +Faster search results\r
+> +\r
+> +  Communication between search mode and the notmuch CLI is now more\r
+> +  efficient because it uses the CLI's S-expression support.  As a\r
+> +  result, search mode should now fill search buffers faster.\r
+> +\r
+>  No Emacs 22 support\r
+>  \r
+>    The Emacs 22 support added late 2010 was sufficient only for a short\r
+> diff --git a/emacs/notmuch-lib.el b/emacs/notmuch-lib.el\r
+> index 180f63d..a206808 100644\r
+> --- a/emacs/notmuch-lib.el\r
+> +++ b/emacs/notmuch-lib.el\r
+> @@ -497,8 +497,8 @@ status."\r
+>       (command (or (executable-find notmuch-command)\r
+>                    (error "command not found: %s" notmuch-command)))\r
+>       (proc (apply #'start-process name buffer\r
+> -                  "sh" "-c"\r
+> -                  "ERR=\"$1\"; shift; exec \"$0\" \"$@\" 2>\"$ERR\""\r
+> +                  "/bin/sh" "-c"\r
+> +                  "exec 2>\"$1\"; shift; exec \"$0\" \"$@\""\r
+>                    command err-file args)))\r
+>      (process-put proc 'err-file err-file)\r
+>      (process-put proc 'sub-sentinel sentinel)\r
+> @@ -533,8 +533,8 @@ status."\r
+>          (when warnings\r
+>            (notmuch-logged-error (car warnings) (cdr warnings)))))\r
+>        (error\r
+> -       ;; Emacs behaves strangely if error an error escapes from a\r
+> -       ;; sentinel, so turns errors into messages.\r
+> +       ;; Emacs behaves strangely if an error escapes from a sentinel,\r
+> +       ;; so turn errors into messages.\r
+>         (message "%s" (error-message-string err))))\r
+>      (ignore-errors (delete-file err-file))))\r
+>  \r
+>\r
+>\r
+> _______________________________________________\r
+> notmuch mailing list\r
+> notmuch@notmuchmail.org\r
+> http://notmuchmail.org/mailman/listinfo/notmuch\r