b9/d18a42def33e8028abb772bb50f566fcff9d43

   1 Return-Path: <tomi.ollila@nixu.com>\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 85ECA429E26\r
   6         for <notmuch@notmuchmail.org>; Mon, 12 Dec 2011 03:38:30 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 0\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
  12         autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id NNBrNLgxFr9T for <notmuch@notmuchmail.org>;\r
  16         Mon, 12 Dec 2011 03:38:29 -0800 (PST)\r
  17 Received: from mail-gw3.nixu.fi (mail-gw3.nixu.fi [193.209.237.7])\r
  18         (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 86CA3429E25\r
  21         for <notmuch@notmuchmail.org>; Mon, 12 Dec 2011 03:38:29 -0800 (PST)\r
  22 Received: from pps.filterd (mail-gw3 [127.0.0.1])\r
  23         by mail-gw3.nixu.fi (8.14.4/8.14.4) with SMTP id pBCBcNJ7003656;\r
  24         Mon, 12 Dec 2011 13:38:23 +0200\r
  25 Received: from taco2.nixu.fi (taco2.nixu.fi [194.197.118.31])\r
  26         by mail-gw3.nixu.fi with ESMTP id 114cs0stjr-1\r
  27         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  28         Mon, 12 Dec 2011 13:38:23 +0200\r
  29 Received: from taco2.nixu.fi (taco2.nixu.fi [194.197.118.31])\r
  30         by taco2.nixu.fi (8.14.3/8.14.3/Debian-5+lenny1) with ESMTP id\r
  31         pBCBcLq2004979; Mon, 12 Dec 2011 13:38:22 +0200\r
  32 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  33 To: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>,\r
  34         Austin Clements <amdragon@MIT.EDU>, Jani Nikula <jani@nikula.org>\r
  35 Subject: Re: [PATCH] emacs: support "notmuch new" as a notmuch-poll-script\r
  36 In-Reply-To: <87wra2ay4e.fsf@gmail.com>\r
  37 References: <1323640100-18326-1-git-send-email-jani@nikula.org>\r
  38         <87wra26a5u.fsf@gmail.com> <20111211225822.GH2760@mit.edu>\r
  39         <CAB+hUn-ADRtZRCq5Ad6H6R2L246EBR8hOSsk4B2yeRsH4kN16w@mail.gmail.com>\r
  40         <20111212003103.GI2760@mit.edu> <yf6wra2m6y7.fsf@taco2.nixu.fi>\r
  41         <87wra2ay4e.fsf@gmail.com>\r
  42 User-Agent: Notmuch/0.10.2+93~g631d290 (http://notmuchmail.org) Emacs/23.3.1\r
  43         (i686-pc-linux-gnu)\r
  44 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  45         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  46         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  47 Date: Mon, 12 Dec 2011 13:38:21 +0200\r
  48 Message-ID: <yf6r50am34i.fsf@taco2.nixu.fi>\r
  49 MIME-Version: 1.0\r
  50 Content-Type: text/plain; charset=us-ascii\r
  51 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:5.5.7110,\r
  52  1.0.211,       0.0.0000        definitions=2011-12-12_02:2011-12-12, 2011-12-12,\r
  53         1970-01-01 signatures=0\r
  54 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0\r
  55         ipscore=0 suspectscore=0\r
  56         phishscore=0 bulkscore=0 adultscore=0 classifier=spam adjust=0\r
  57         reason=mlx\r
  58         scancount=1 engine=6.0.2-1012030000 definitions=main-1112120057\r
  59 Cc: notmuch@notmuchmail.org\r
  60 X-BeenThere: notmuch@notmuchmail.org\r
  61 X-Mailman-Version: 2.1.13\r
  62 Precedence: list\r
  63 List-Id: "Use and development of the notmuch mail system."\r
  64         <notmuch.notmuchmail.org>\r
  65 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  66         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  67 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  68 List-Post: <mailto:notmuch@notmuchmail.org>\r
  69 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  70 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  71         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  72 X-List-Received-Date: Mon, 12 Dec 2011 11:38:30 -0000\r
  73 \r
  74 On Mon, 12 Dec 2011 14:21:53 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:\r
  75 > On Mon, 12 Dec 2011 12:15:44 +0200, Tomi Ollila <tomi.ollila@iki.fi> wrote:\r
  76 > > On Sun, 11 Dec 2011 19:31:03 -0500, Austin Clements <amdragon@MIT.EDU> wrote:\r
  77 > > > \r
  78 > > > So here's another idea, prefaced with a rant.\r
  79 > > > \r
  80 > > > It's bothered me for a long time that notmuch-emacs didn't just know\r
  81 > > > by default how to check for new mail.  What MUA doesn't know how to\r
  82 > > > check for new mail?  Why does a new user of notmuch have to tell it\r
  83 > > > how to check for new mail?  Of course, this *had* to be configured\r
  84 > > > before because everyone had their own way of checking for new mail.\r
  85 > > > Hooks eliminate this unnecessary flexibility and make "notmuch new"\r
  86 > > > the one true way to check for new mail---as it ought to be---and in\r
  87 > > > turn make the notmuch-poll-script variable obsolete.\r
  88 > > > \r
  89 > > > So, what about changing the default "" setting of notmuch-poll-script\r
  90 > > > from meaning "do nothing and be useless" to meaning "run notmuch new\r
  91 > > > (using notmuch-command)"?  It will then automatically do the right\r
  92 > > > thing for new users, while still being backward-compatible and\r
  93 > > > allowing an escape hatch for bizarre situations.\r
  94 > > \r
  95 > > +1\r
  96 > > \r
  97 > > So, it could work like this:\r
  98 > > \r
  99 > > (defun notmuch-poll ()\r
 100 > >   "FIX DOCSTRING"\r
 101 > >   (interactive)\r
 102 > >   (if (stringp notmuch notmuch-poll-script)\r
 103 > >       (if (string= notmuch-poll-script "")\r
 104 > >         (call-process notmuch-command nil nil nil "new")\r
 105 > >         (call-process notmuch-poll-script))))\r
 106 > > \r
 107 > > I.e. in case notmuch-poll-script == nil, (or not string)\r
 108 > > do nothing. In case notmuch-poll-script == "" execute notmuch new\r
 109 > > and if notmuch-poll-script is string with content execute that.\r
 110 > > \r
 111 > \r
 112 > I think the following scheme would be slightly better and more\r
 113 > consistent:\r
 114 > \r
 115 > * nil - run "notmuch new", the new default\r
 116 > * "" - do nothing\r
 117 > * "script" - run script\r
 118 \r
 119 You're right. This us MUCH better (something irked me when writing\r
 120 my suggestion but that was for ultimate backward compability); The\r
 121 default value for notmuch-poll-script could be changed to nil.\r
 122 \r
 123 > \r
 124 > Regards,\r
 125 >   Dmitry\r
 126 > \r
 127 > > users who want other functionality can reimplement notmuch-poll function\r
 128 > > after notmuch has been loaded (and manage things themselves when internal\r
 129 > > implementation changes ;().\r
 130 > > \r
 131 > > \r
 132 > > Tomi\r
 133 \r
 134 Tomi\r