78/b6bf430b3f494b9ce776f71c9a39cdbb73176d

   1 Return-Path: <amdragon@mit.edu>\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 C9CA8431FD0\r
   6         for <notmuch@notmuchmail.org>; Wed,  7 Dec 2011 12:10:17 -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.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] 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 pa6+3yCnrE44 for <notmuch@notmuchmail.org>;\r
  16         Wed,  7 Dec 2011 12:10:15 -0800 (PST)\r
  17 Received: from dmz-mailsec-scanner-1.mit.edu (DMZ-MAILSEC-SCANNER-1.MIT.EDU\r
  18         [18.9.25.12])\r
  19         by olra.theworths.org (Postfix) with ESMTP id B3F20431FB6\r
  20         for <notmuch@notmuchmail.org>; Wed,  7 Dec 2011 12:10:15 -0800 (PST)\r
  21 X-AuditID: 1209190c-b7f806d0000008d6-a9-4edfc82741a7\r
  22 Received: from mailhub-auth-2.mit.edu ( [18.7.62.36])\r
  23         by dmz-mailsec-scanner-1.mit.edu (Symantec Messaging Gateway) with SMTP\r
  24         id 9F.B9.02262.728CFDE4; Wed,  7 Dec 2011 15:10:15 -0500 (EST)\r
  25 Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103])\r
  26         by mailhub-auth-2.mit.edu (8.13.8/8.9.2) with ESMTP id pB7KAECu006091; \r
  27         Wed, 7 Dec 2011 15:10:14 -0500\r
  28 Received: from awakening.csail.mit.edu (awakening.csail.mit.edu [18.26.4.91])\r
  29         (authenticated bits=0)\r
  30         (User authenticated as amdragon@ATHENA.MIT.EDU)\r
  31         by outgoing.mit.edu (8.13.6/8.12.4) with ESMTP id pB7KABeh009577\r
  32         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  33         Wed, 7 Dec 2011 15:10:12 -0500 (EST)\r
  34 Received: from amthrax by awakening.csail.mit.edu with local (Exim 4.77)\r
  35         (envelope-from <amdragon@mit.edu>)\r
  36         id 1RYNqK-0002iz-4Y; Wed, 07 Dec 2011 15:11:52 -0500\r
  37 Date: Wed, 7 Dec 2011 15:11:52 -0500\r
  38 From: Austin Clements <amdragon@MIT.EDU>\r
  39 To: Jani Nikula <jani@nikula.org>\r
  40 Subject: Re: [PATCH v3 0/2] notmuch hooks\r
  41 Message-ID: <20111207201152.GA3787@mit.edu>\r
  42 References:\r
  43  <7fbe6befcf31881a9bca672f55b93501249a220c.1322859389.git.jani@nikula.org>\r
  44         <cover.1323176771.git.jani@nikula.org>\r
  45         <87d3c1rtcq.fsf@servo.finestructure.net>        <87borlf4ve.fsf@hermes.hocat.ca>\r
  46  <87mxb4b6l9.fsf@nikula.org>\r
  47 MIME-Version: 1.0\r
  48 Content-Type: text/plain; charset=us-ascii\r
  49 Content-Disposition: inline\r
  50 In-Reply-To: <87mxb4b6l9.fsf@nikula.org>\r
  51 User-Agent: Mutt/1.5.21 (2010-09-15)\r
  52 X-Brightmail-Tracker:\r
  53  H4sIAAAAAAAAA+NgFprGKsWRmVeSWpSXmKPExsUixG6noqt+4r6fwYMl/BZN050t9uzzsrh+\r
  54         cyazxf29C9gdWDzunubyuHX/NbvHs1W3mD0mPJ7PFMASxWWTkpqTWZZapG+XwJXx4MAX9oIm\r
  55         sYqe9XINjIcEuxg5OSQETCT2PfrEBmGLSVy4tx7I5uIQEtjHKLH20FEWCGc9o8S/Q9eYIJwT\r
  56         TBKHdl9mhnCWMEo8WvSOEaSfRUBFYuv6FewgNpuAhsS2/cvB4iICihKbT+5nBGlgFmhnlHjy\r
  57         dhMzSEJYQFPix6xGsCJeAW2JVdc2MEJMfQW0b+cGNoiEoMTJmU9YQGxmAS2JG/9eAt3BAWRL\r
  58         Syz/xwES5gRaduPeRLASUaAjppzcxjaBUWgWku5ZSLpnIXQvYGRexSibklulm5uYmVOcmqxb\r
  59         nJyYl5dapGuol5tZopeaUrqJERT8nJI8OxjfHFQ6xCjAwajEw+u1/L6fEGtiWXFl7iFGSQ4m\r
  60         JVHexgNAIb6k/JTKjMTijPii0pzU4kOMEhzMSiK8X2YB5XhTEiurUovyYVLSHCxK4rwHdzj4\r
  61         CQmkJ5akZqemFqQWwWRlODiUJHi7jgM1ChalpqdWpGXmlCCkmTg4QYbzAA2vBanhLS5IzC3O\r
  62         TIfIn2JUlBLnXQ6SEABJZJTmwfXCktMrRnGgV4R5N+8BquIBJja4bmDoA30kwssXdRdkcEki\r
  63         QkqqgXFKZ/cWe7/soh/yF5TFb/6vjY58IX5w3RqjWBl1m1+zLNR8Vc6tC7M7IOjSlno9ZPqM\r
  64         ZQ87tMw2M5bc3iMt/zSiXmtrmFc///v0uj9TMnbwMS3j3WWz539skfIOgezSEmf1+l3/DZIC\r
  65         j+6Id3sqsOXqQ6NFN/dYFnXN+r5QIuutUviig3/nOyixFGckGmoxFxUnAgDAzzKrKQMAAA==\r
  66 Cc: notmuch@notmuchmail.org\r
  67 X-BeenThere: notmuch@notmuchmail.org\r
  68 X-Mailman-Version: 2.1.13\r
  69 Precedence: list\r
  70 List-Id: "Use and development of the notmuch mail system."\r
  71         <notmuch.notmuchmail.org>\r
  72 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  73         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  74 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  75 List-Post: <mailto:notmuch@notmuchmail.org>\r
  76 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  77 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  78         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  79 X-List-Received-Date: Wed, 07 Dec 2011 20:10:17 -0000\r
  80 \r
  81 Quoth Jani Nikula on Dec 07 at  8:05 pm:\r
  82 > On Tue, 06 Dec 2011 22:16:37 -0500, Tom Prince <tom.prince@ualberta.net> wrote:\r
  83 > > On Tue, 06 Dec 2011 18:47:01 -0800, Jameson Graef Rollins <jrollins@finestructure.net> wrote:\r
  84 > > > Also, what if we make it so that the post-new hook script only runs if\r
  85 > > > notmuch new processes new messages?  All of my post-new functions don't\r
  86 > > > need to be run at all if there is no new mail.\r
  87 > \r
  88 > I think the post-new hook should be run always (provided there have been\r
  89 > no errors). I think it might be surprising not to, and some users might\r
  90 > use the hook for something other than tagging.\r
  91 > \r
  92 > > Or would it make sense to pass this information to the hook somehow?\r
  93 > \r
  94 > It would, but as I wrote in id:"87mxb8kt5r.fsf@nikula.org", I think that\r
  95 > should come as another patch afterwards. I know I can't decide yet what\r
  96 > should be passed and how. Processed message counts (added, deleted,\r
  97 > renamed) could be passed on the command line, but how useful is that\r
  98 > really? The same can be easily achieved through initial tagging.\r
  99 \r
 100 I would worry about creating any hook interface that's difficult to\r
 101 use correctly.  For example, notmuch new could be interrupted right\r
 102 between processing all new messages and calling the hook; then, on the\r
 103 next run, it'll tell the hook that nothing happened.  More generally,\r
 104 any message counts passed to a hook can't be better than lower bounds\r
 105 and I don't see how you could use such information correctly in a\r
 106 hook.\r
 107 \r
 108 Using an initial "new" tag that you remove at the end of a hook,\r
 109 though, is stable.  It guarantees that the hook is aware of any new\r
 110 messages at least once.  If a hook needs a new message count, it\r
 111 should run notmuch count tag:new.\r
 112 \r
 113 > Message-ids could not be passed on the command line (there just can be\r
 114 > too many of them) so it would require setting up a pipe and feeding them\r
 115 > to stdin of the hook. The post-new hook should be run after the database\r
 116 > has been closed, but the message-ids are not saved during notmuch new\r
 117 > processing. Saving them for later gets complicated for not much extra\r
 118 > benefit in addition to creative use of initial tagging, as far as I can\r
 119 > see. Plus interrupting the post-new hook with this setup would screw up\r
 120 > your processing if it only depended on the message-ids.\r
 121 \r
 122 I think this would have to be a separate hook that runs concurrently\r
 123 with new, both so that new doesn't have to buffer this information and\r
 124 so that the majority of post-new hooks that don't need this\r
 125 information don't have to deal with it.  Though, as with passing\r
 126 message counts to post-new, I worry about actually do anything\r
 127 correctly in such a hook in the presence of interruptions and\r
 128 failures.\r
 129 \r
 130 > All in all, I'd postpone all of this until later.\r
 131 \r
 132 That sounds wise.\r