b7/2cbfa894585fd529304be9fe0fc7670603d87e

   1 Return-Path: <cworth@cworth.org>\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 B80514196F3\r
   6         for <notmuch@notmuchmail.org>; Wed,  9 Jun 2010 12:33:32 -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: -2.89\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.89 tagged_above=-999 required=5\r
  12         tests=[ALL_TRUSTED=-1, BAYES_00=-1.9, T_MIME_NO_TEXT=0.01]\r
  13         autolearn=ham\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 VZFv8-NsT-qF; Wed,  9 Jun 2010 12:33:22 -0700 (PDT)\r
  17 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  18         by olra.theworths.org (Postfix) with ESMTP id 0E882431FC1;\r
  19         Wed,  9 Jun 2010 12:33:22 -0700 (PDT)\r
  20 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  21         id AFB51568E39; Wed,  9 Jun 2010 12:33:21 -0700 (PDT)\r
  22 From: Carl Worth <cworth@cworth.org>\r
  23 To: Jameson Rollins <jrollins@finestructure.net>,\r
  24         Dirk Hohndel <hohndel@infradead.org>,\r
  25         Notmuch Mail <notmuch@notmuchmail.org>\r
  26 Subject: Re: [PATCH] remove message archiving from show-advance-and-archive\r
  27 In-Reply-To: <87sk4wvwqy.fsf@servo.finestructure.net>\r
  28 References: <87zkz4xnf4.fsf@servo.finestructure.net>\r
  29         <871vcghz8o.fsf@yoom.home.cworth.org>\r
  30         <m3sk4wdq4g.fsf@x201s.gr8dns.org>\r
  31         <87sk4wvwqy.fsf@servo.finestructure.net>\r
  32 User-Agent: Notmuch/0.3.1-61-ge870437 (http://notmuchmail.org) Emacs/23.1.1\r
  33         (i486-pc-linux-gnu)\r
  34 Date: Wed, 09 Jun 2010 12:33:21 -0700\r
  35 Message-ID: <87vd9sgfwe.fsf@yoom.home.cworth.org>\r
  36 MIME-Version: 1.0\r
  37 Content-Type: multipart/signed; boundary="=-=-=";\r
  38         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  39 X-BeenThere: notmuch@notmuchmail.org\r
  40 X-Mailman-Version: 2.1.13\r
  41 Precedence: list\r
  42 List-Id: "Use and development of the notmuch mail system."\r
  43         <notmuch.notmuchmail.org>\r
  44 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  45         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  46 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  47 List-Post: <mailto:notmuch@notmuchmail.org>\r
  48 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  49 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  50         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  51 X-List-Received-Date: Wed, 09 Jun 2010 19:33:32 -0000\r
  52 \r
  53 --=-=-=\r
  54 Content-Transfer-Encoding: quoted-printable\r
  55 \r
  56 On Wed, 09 Jun 2010 15:20:21 -0400, Jameson Rollins <jrollins@finestructure=\r
  57 .net> wrote:\r
  58 > On Wed, 09 Jun 2010 11:20:47 -0700, Dirk Hohndel <hohndel@infradead.org> =\r
  59 wrote:\r
  60 > > On Wed, 09 Jun 2010 10:50:15 -0700, Carl Worth <cworth@cworth.org> wrot=\r
  61 e:\r
  62 > > I really like this. It's consistent and I'm sure I'll get used to it\r
  63 > > quickly. The only question now is "all messages in a thread" or "all\r
  64 > > open messages in a thread". I'd vote for all.\r
  65 \r
  66 I have situations in which I want either one. The advantage of doing\r
  67 "all open messages" is that it's easy to get the "all messages" behavior\r
  68 by way of an existing command that's already useful in its own right,\r
  69 ("open all messages").\r
  70 \r
  71 And I think "all open messages" is actually the right default as\r
  72 well. If I do a focused search and end up reading one message that\r
  73 matched the search, then archiving away that message and not others that\r
  74 just happen to be in the same thread.\r
  75 \r
  76 > I actually vote for the behavior Carl suggests.  The behavior of\r
  77 > archiving the entire thread can be done by popping out to the search and\r
  78 > applying notmuch-search-archive-thread to the thread with:\r
  79 >=20\r
  80 > (define-key notmuch-search-mode-map "a" 'notmuch-search-archive-thread)\r
  81 \r
  82 Or not. ;-)\r
  83 \r
  84 I actually want to also fix the search mode commands like this to only\r
  85 operate on messages that matched the (original search).\r
  86 \r
  87 We had behavior that limited these commands to matching messages once,\r
  88 but that was broken because it re-ran the search to find the list. So\r
  89 that led to behavior like:\r
  90 \r
  91         -inbox # archive all messages in the thread\r
  92 \r
  93 Then if the user realized that was a mistake, trying:\r
  94 \r
  95         +inbox # new search now matches nothing, so no effect\r
  96 \r
  97 Our stop-gap solution here was to re-run a search with fewer\r
  98 restrictions if the first search returned an empty list.\r
  99 \r
 100 The correct solution is to remember the list of messages that matched\r
 101 the first time and always operate on those. That will fix the above\r
 102 problem and also avoid the race condition that can currently happen if\r
 103 notmuch new is being run asynchronously from emacs. (The race is that\r
 104 the user can thing he's archiving a thread with a single message but can\r
 105 actually archive many messages that arrived since the original search\r
 106 because the 'a' command is re-running the search.)\r
 107 \r
 108 =2DCarl\r
 109 \r
 110 =2D-=20\r
 111 carl.d.worth@intel.com\r
 112 \r
 113 --=-=-=\r
 114 Content-Type: application/pgp-signature\r
 115 \r
 116 -----BEGIN PGP SIGNATURE-----\r
 117 Version: GnuPG v1.4.10 (GNU/Linux)\r
 118 \r
 119 iD8DBQFMD+yB6JDdNq8qSWgRAikuAJ94Gcoaq3yY9s8tum5Ttw464VK1YQCfWihF\r
 120 VD9p0k7pD/LoEJnVn83QctY=\r
 121 =Rjpu\r
 122 -----END PGP SIGNATURE-----\r
 123 --=-=-=--\r