42/eea2187060fbe3c71b20d849a497fb06ec531a

   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 1FFFF431FAF\r
   6         for <notmuch@notmuchmail.org>; Thu, 19 Jan 2012 11:37:08 -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 dsKK1jNLZQBr for <notmuch@notmuchmail.org>;\r
  16         Thu, 19 Jan 2012 11:37:07 -0800 (PST)\r
  17 Received: from dmz-mailsec-scanner-6.mit.edu (DMZ-MAILSEC-SCANNER-6.MIT.EDU\r
  18         [18.7.68.35])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 9A188431FAE\r
  20         for <notmuch@notmuchmail.org>; Thu, 19 Jan 2012 11:37:07 -0800 (PST)\r
  21 X-AuditID: 12074423-b7f9c6d0000008c3-3c-4f1870e3cb02\r
  22 Received: from mailhub-auth-3.mit.edu ( [18.9.21.43])\r
  23         by dmz-mailsec-scanner-6.mit.edu (Symantec Messaging Gateway) with SMTP\r
  24         id FF.C6.02243.3E0781F4; Thu, 19 Jan 2012 14:37:07 -0500 (EST)\r
  25 Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103])\r
  26         by mailhub-auth-3.mit.edu (8.13.8/8.9.2) with ESMTP id q0JJb6rp027124; \r
  27         Thu, 19 Jan 2012 14:37:06 -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 q0JJb5qk011923\r
  32         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  33         Thu, 19 Jan 2012 14:37:06 -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 1Rnxmx-00044E-W6; Thu, 19 Jan 2012 14:36:48 -0500\r
  37 Date: Thu, 19 Jan 2012 14:36:47 -0500\r
  38 From: Austin Clements <amdragon@MIT.EDU>\r
  39 To: Pieter Praet <pieter@praet.org>\r
  40 Subject: Re: [PATCH v3 2/2] search: Support automatic tag exclusions\r
  41 Message-ID: <20120119193647.GN16740@mit.edu>\r
  42 References: <1326586654-16840-3-git-send-email-amdragon@mit.edu>\r
  43         <1327000744-25463-1-git-send-email-pieter@praet.org>\r
  44 MIME-Version: 1.0\r
  45 Content-Type: text/plain; charset=us-ascii\r
  46 Content-Disposition: inline\r
  47 In-Reply-To: <1327000744-25463-1-git-send-email-pieter@praet.org>\r
  48 User-Agent: Mutt/1.5.21 (2010-09-15)\r
  49 X-Brightmail-Tracker:\r
  50  H4sIAAAAAAAAA+NgFmpmleLIzCtJLcpLzFFi42IR4hTV1n1cIOFvcHUSu8X1mzOZLX6/vsHs\r
  51         wOTxbNUtZo+OfZdZA5iiuGxSUnMyy1KL9O0SuDLWn2ljL/jBV3H6wBXWBsYX3F2MnBwSAiYS\r
  52         6/8eZYWwxSQu3FvP1sXIxSEksI9R4sfzqSwQzgZGia3975khnJNMEvvn/meEcJYwSjTdecoC\r
  53         0s8ioCpx/+1ZdhCbTUBDYtv+5YwgtoiAssTpJz/B4swCWhJbN34AiwsLuEjs3tYEZvMK6Eh0\r
  54         rz0BViMkUC5x8tMJZoi4oMTJmU9YYHpv/HvJ1MXIAWRLSyz/xwFicgo4SnR99wSpEBVQkZhy\r
  55         chvbBEahWUiaZyFpnoXQvICReRWjbEpulW5uYmZOcWqybnFyYl5eapGumV5uZoleakrpJkZQ\r
  56         WLO7KO9g/HNQ6RCjAAejEg8vp6uEvxBrYllxZe4hRkkOJiVR3rh8oBBfUn5KZUZicUZ8UWlO\r
  57         avEhRgkOZiUR3nk5QDnelMTKqtSifJiUNAeLkjivhtY7PyGB9MSS1OzU1ILUIpisDAeHkgRv\r
  58         PDB+hQSLUtNTK9Iyc0oQ0kwcnCDDeYCGR4HU8BYXJOYWZ6ZD5E8xKkqJ84aCJARAEhmleXC9\r
  59         sLTzilEc6BVhXk+QKh5gyoLrfgU0mAlosEeTGMjgkkSElFQD48mWqg07b8gGPd7+N2TNIYWH\r
  60         bq1xjy3sgg1WRduI2yYtmHVyXlDDxbWH89t95Rt/7o1KmOq0xN1I/G/QJEmHeeH5Zn+C+1Jz\r
  61         f0zfee9L4LuIwr1LrfKv9K9052T0yZn8Yb49e9++iQrRfzw72hWSpok0si67dXH/q4K9hbJv\r
  62         juwU1mlmWaAmosRSnJFoqMVcVJwIAHD+VYwWAwAA\r
  63 Cc: Notmuch Mail <notmuch@notmuchmail.org>\r
  64 X-BeenThere: notmuch@notmuchmail.org\r
  65 X-Mailman-Version: 2.1.13\r
  66 Precedence: list\r
  67 List-Id: "Use and development of the notmuch mail system."\r
  68         <notmuch.notmuchmail.org>\r
  69 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  70         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  71 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  72 List-Post: <mailto:notmuch@notmuchmail.org>\r
  73 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  74 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  75         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  76 X-List-Received-Date: Thu, 19 Jan 2012 19:37:08 -0000\r
  77 \r
  78 Quoth Pieter Praet on Jan 19 at  8:19 pm:\r
  79 > Nice feature!  I won't be using it myself, but I can imagine it being\r
  80 > *very* useful for those who still feel the need to "delete" email :).\r
  81 \r
  82 Same here.  I probably will use the spam tag, though.\r
  83 \r
  84 > Nitpicking:\r
  85 > \r
  86 > - All other config-related functions and args include the section title\r
  87 >   in their name [1], so for the sake of consistency, we might want to\r
  88 >   mirror that.  Also, the "auto"matic part is pretty much a given.\r
  89 > \r
  90 >   So I'd like to suggest replacing all occurences of "auto_exclude_tags"\r
  91 >   with "search_exclude_tags" (and simply "exclude_tags" in the args to\r
  92 >   `_config_get_list' and `_config_set_list', of course).\r
  93 \r
  94 You are technically correct, the best kind of correct.  I'd completely\r
  95 missed this pattern.  This should get fixed ASAP, while this feature\r
  96 still has limited adoption.\r
  97 \r
  98 >   Unfortunately, this would also partially invalidate your recent NEWS\r
  99 >   submission [2].\r
 100 \r
 101 No worries, though maybe you want to tack an updated version of that\r
 102 patch on the end of your series?\r
 103 \r
 104 > - If the 'search.exclude_tags' option is missing from the config file,\r
 105 >   its value is automatically set to "deleted;spam;", which probably isn't\r
 106 >   a sane default.  Luckily, you've already provided the solution [3].\r
 107 \r
 108 I'm good either way.  I got lost in the discussion of defaults but\r
 109 Jamie assured me everything was okay, so I took the path of least\r
 110 resistance and left things as they were.\r
 111 \r
 112 > - To make new users aware of the config option's existence, we should\r
 113 >   prompt them to configure it during setup.\r
 114 \r
 115 Sure.\r
 116 \r
 117 > Patches follow.\r
 118 > \r
 119 > \r
 120 > Peace\r
 121 > \r
 122 > \r
 123 > [1] Eg. `notmuch_config_get_user_name', `notmuch_config_get_new_tags',\r
 124 >         `notmuch_config_get_maildir_synchronize_flags', ...\r
 125 > \r
 126 > [2] id:"1326920330-31496-1-git-send-email-amdragon@mit.edu"\r
 127 > \r
 128 > [3] id:"20120117203211.GQ16740@mit.edu"\r
 129 > \r