93/f04deb478d3d1df70234b325aa4099528bb5fd

   1 Return-Path: <m.walters@qmul.ac.uk>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5         by olra.theworths.org (Postfix) with ESMTP id D1F41431FDD\r
   6         for <notmuch@notmuchmail.org>; Thu,  4 Jul 2013 01:07:40 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -1.098\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-1.098 tagged_above=-999 required=5\r
  12         tests=[DKIM_ADSP_CUSTOM_MED=0.001, FREEMAIL_FROM=0.001,\r
  13         NML_ADSP_CUSTOM_MED=1.2, RCVD_IN_DNSWL_MED=-2.3] autolearn=disabled\r
  14 Received: from olra.theworths.org ([127.0.0.1])\r
  15         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16         with ESMTP id gf7PjwaIwF2w for <notmuch@notmuchmail.org>;\r
  17         Thu,  4 Jul 2013 01:07:32 -0700 (PDT)\r
  18 Received: from mail2.qmul.ac.uk (mail2.qmul.ac.uk [138.37.6.6])\r
  19         (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))\r
  20         (No client certificate requested)\r
  21         by olra.theworths.org (Postfix) with ESMTPS id 59B32431FBF\r
  22         for <notmuch@notmuchmail.org>; Thu,  4 Jul 2013 01:07:32 -0700 (PDT)\r
  23 Received: from smtp.qmul.ac.uk ([138.37.6.40])\r
  24         by mail2.qmul.ac.uk with esmtp (Exim 4.71)\r
  25         (envelope-from <m.walters@qmul.ac.uk>)\r
  26         id 1UueZa-0003uY-7D; Thu, 04 Jul 2013 09:07:28 +0100\r
  27 Received: from 93-97-24-31.zone5.bethere.co.uk ([93.97.24.31] helo=localhost)\r
  28         by smtp.qmul.ac.uk with esmtpsa (TLSv1:AES128-SHA:128) (Exim 4.71)\r
  29         (envelope-from <m.walters@qmul.ac.uk>)\r
  30         id 1UueZZ-0007tU-VZ; Thu, 04 Jul 2013 09:07:26 +0100\r
  31 From: Mark Walters <markwalters1009@gmail.com>\r
  32 To: Tomi Ollila <tomi.ollila@iki.fi>, notmuch@notmuchmail.org\r
  33 Subject: Re: [PATCH] contrib: pick: use notmuch-start-notmuch\r
  34 In-Reply-To: <m2zju2lqpy.fsf@guru.guru-group.fi>\r
  35 References: <1372921788-16042-1-git-send-email-markwalters1009@gmail.com>\r
  36         <m2zju2lqpy.fsf@guru.guru-group.fi>\r
  37 User-Agent: Notmuch/0.15.2+192~g8222af3 (http://notmuchmail.org) Emacs/23.4.1\r
  38         (i486-pc-linux-gnu)\r
  39 Date: Thu, 04 Jul 2013 09:07:24 +0100\r
  40 Message-ID: <87ip0qg343.fsf@qmul.ac.uk>\r
  41 MIME-Version: 1.0\r
  42 Content-Type: text/plain; charset=us-ascii\r
  43 X-Sender-Host-Address: 93.97.24.31\r
  44 X-QM-SPAM-Info: Sender has good ham record.  :)\r
  45 X-QM-Body-MD5: b34aa5a27364e928f28a4ee6b6f8d364 (of first 20000 bytes)\r
  46 X-SpamAssassin-Score: 0.0\r
  47 X-SpamAssassin-SpamBar: /\r
  48 X-SpamAssassin-Report: The QM spam filters have analysed this message to\r
  49         determine if it is\r
  50         spam. We require at least 5.0 points to mark a message as spam.\r
  51         This message scored 0.0 points. Summary of the scoring: \r
  52         * 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail\r
  53         provider *      (markwalters1009[at]gmail.com)\r
  54         *  0.0 AWL AWL: From: address is in the auto white-list\r
  55 X-QM-Scan-Virus: ClamAV says the message is clean\r
  56 X-BeenThere: notmuch@notmuchmail.org\r
  57 X-Mailman-Version: 2.1.13\r
  58 Precedence: list\r
  59 List-Id: "Use and development of the notmuch mail system."\r
  60         <notmuch.notmuchmail.org>\r
  61 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  62         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  63 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  64 List-Post: <mailto:notmuch@notmuchmail.org>\r
  65 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  66 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  67         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  68 X-List-Received-Date: Thu, 04 Jul 2013 08:07:41 -0000\r
  69 \r
  70 \r
  71 Tomi Ollila <tomi.ollila@iki.fi> writes:\r
  72 \r
  73 > FWIW the code LGTM. Do the pick tests use this and do those pass ?\r
  74 \r
  75 Thanks for the review.\r
  76 \r
  77 The tests do all pass. There are two sets of pick tests: one uses the\r
  78 async parser (the default) which uses this code and the other uses the\r
  79 sync parser (which probably should be removed as the async parser seems\r
  80 stable).\r
  81 \r
  82 Best wishes\r
  83 \r
  84 Mark\r
  85 \r
  86 \r
  87 \r
  88 \r
  89 \r
  90 \r
  91 >\r
  92 > Tomi\r
  93 >\r
  94 >>\r
  95 >> Best wishes\r
  96 >>\r
  97 >> Mark\r
  98 >>\r
  99 >>\r
 100 >>  contrib/notmuch-pick/notmuch-pick.el |    7 +++----\r
 101 >>  1 files changed, 3 insertions(+), 4 deletions(-)\r
 102 >>\r
 103 >> diff --git a/contrib/notmuch-pick/notmuch-pick.el b/contrib/notmuch-pick/notmuch-pick.el\r
 104 >> index fbd7c0b..7f5f729 100644\r
 105 >> --- a/contrib/notmuch-pick/notmuch-pick.el\r
 106 >> +++ b/contrib/notmuch-pick/notmuch-pick.el\r
 107 >> @@ -817,16 +817,15 @@ Complete list of currently available key bindings:\r
 108 >>      (if (equal (car (process-lines notmuch-command "count" search-args)) "0")\r
 109 >>      (setq search-args basic-query))\r
 110 >>      (if notmuch-pick-asynchronous-parser\r
 111 >> -    (let ((proc (start-process\r
 112 >> -                 "notmuch-pick" buffer\r
 113 >> -                 notmuch-command "show" "--body=false" "--format=sexp"\r
 114 >> +    (let ((proc (notmuch-start-notmuch\r
 115 >> +                 "notmuch-pick" buffer #'notmuch-pick-process-sentinel\r
 116 >> +                 "show" "--body=false" "--format=sexp"\r
 117 >>                   message-arg search-args))\r
 118 >>            ;; Use a scratch buffer to accumulate partial output.\r
 119 >>                ;; This buffer will be killed by the sentinel, which\r
 120 >>                ;; should be called no matter how the process dies.\r
 121 >>                (parse-buf (generate-new-buffer " *notmuch pick parse*")))\r
 122 >>            (process-put proc 'parse-buf parse-buf)\r
 123 >> -      (set-process-sentinel proc 'notmuch-pick-process-sentinel)\r
 124 >>        (set-process-filter proc 'notmuch-pick-process-filter)\r
 125 >>        (set-process-query-on-exit-flag proc nil))\r
 126 >>        (progn\r
 127 >> -- \r
 128 >> 1.7.9.1\r
 129 >>\r
 130 >> _______________________________________________\r
 131 >> notmuch mailing list\r
 132 >> notmuch@notmuchmail.org\r
 133 >> http://notmuchmail.org/mailman/listinfo/notmuch\r