04/0a3926f382091a9c3b4e61767c5d702e6caa57

   1 Return-Path: <pieter@praet.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 57A65429E54\r
   6         for <notmuch@notmuchmail.org>; Sun, 22 Jan 2012 20:17:16 -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 i9gTwIc+MhHE for <notmuch@notmuchmail.org>;\r
  16         Sun, 22 Jan 2012 20:17:15 -0800 (PST)\r
  17 Received: from mail-wi0-f181.google.com (mail-wi0-f181.google.com\r
  18         [209.85.212.181]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 8C73D429E40\r
  21         for <notmuch@notmuchmail.org>; Sun, 22 Jan 2012 20:17:15 -0800 (PST)\r
  22 Received: by wibhi8 with SMTP id hi8so303278wib.26\r
  23         for <notmuch@notmuchmail.org>; Sun, 22 Jan 2012 20:17:12 -0800 (PST)\r
  24 Received: by 10.180.94.97 with SMTP id db1mr10641843wib.16.1327292232854;\r
  25         Sun, 22 Jan 2012 20:17:12 -0800 (PST)\r
  26 Received: from localhost ([109.131.95.182])\r
  27         by mx.google.com with ESMTPS id em13sm10720314wid.7.2012.01.22.20.17.11\r
  28         (version=TLSv1/SSLv3 cipher=OTHER);\r
  29         Sun, 22 Jan 2012 20:17:11 -0800 (PST)\r
  30 From: Pieter Praet <pieter@praet.org>\r
  31 To: Xavier Maillard <xma@gnu.org>, Austin Clements <amdragon@MIT.EDU>\r
  32 Subject: Re: [PATCH v3 2/2] search: Support automatic tag exclusions\r
  33 In-Reply-To: <m2ty3n2xut.fsf@kcals.intra.maillard.im>\r
  34 References: <1326586654-16840-3-git-send-email-amdragon@mit.edu>\r
  35         <1327000744-25463-1-git-send-email-pieter@praet.org>\r
  36         <m2ty3n2xut.fsf@kcals.intra.maillard.im>\r
  37 User-Agent: Notmuch/0.11+101~g94bf862 (http://notmuchmail.org) Emacs/23.3.1\r
  38         (x86_64-unknown-linux-gnu)\r
  39 Date: Mon, 23 Jan 2012 05:15:18 +0100\r
  40 Message-ID: <87ty3nawbt.fsf@praet.org>\r
  41 MIME-Version: 1.0\r
  42 X-Gm-Message-State:\r
  43  ALoCoQmg8jOoz0TdP6EsrW7ZCUoBj7ghNb/iTQB82rwcj8LlizqbYOyOgz2aP/Judl7//4BbB1jK\r
  44 Content-Type: text/plain; charset=us-ascii\r
  45 Cc: Notmuch Mail <notmuch@notmuchmail.org>\r
  46 X-BeenThere: notmuch@notmuchmail.org\r
  47 X-Mailman-Version: 2.1.13\r
  48 Precedence: list\r
  49 List-Id: "Use and development of the notmuch mail system."\r
  50         <notmuch.notmuchmail.org>\r
  51 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  52         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  53 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  54 List-Post: <mailto:notmuch@notmuchmail.org>\r
  55 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  56 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  57         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  58 X-List-Received-Date: Mon, 23 Jan 2012 04:17:16 -0000\r
  59 \r
  60 On Sun, 22 Jan 2012 23:09:30 +0100, Xavier Maillard <xavier@maillard.im> wrote:\r
  61 > Hey Pieter,\r
  62 > \r
  63 \r
  64 Hi!\r
  65 \r
  66 > On Thu, 19 Jan 2012 20:19:00 +0100, Pieter Praet <pieter@praet.org> wrote:\r
  67 > > Nice feature!  I won't be using it myself, but I can imagine it being\r
  68 > > *very* useful for those who still feel the need to "delete" email :).\r
  69 > \r
  70 > Adding a 'deleted' tag does not mean there will be a delete/purge\r
  71 > process ;) (currently I got 5k messages with the tag deleted ;). \r
  72 > \r
  73 \r
  74 Very true, that's why I put quotes around "delete".\r
  75 \r
  76 Deleting email for real is *so* old-fashioned... :D ;)\r
  77 \r
  78 > > Nitpicking:\r
  79 > > \r
  80 > \r
  81 > [ ... ]\r
  82 > \r
  83 > >   So I'd like to suggest replacing all occurences of "auto_exclude_tags"\r
  84 > >   with "search_exclude_tags" (and simply "exclude_tags" in the args to\r
  85 > >   `_config_get_list' and `_config_set_list', of course).\r
  86 > \r
  87 > +1\r
  88 >  \r
  89 > >   Unfortunately, this would also partially invalidate your recent NEWS\r
  90 > >   submission [2].\r
  91 > > \r
  92 > > - If the 'search.exclude_tags' option is missing from the config file,\r
  93 > >   its value is automatically set to "deleted;spam;", which probably isn't\r
  94 > >   a sane default.  Luckily, you've already provided the solution [3].\r
  95 > \r
  96 > I am against doing something /unsafe/ in the user's back. If there is no\r
  97 > option set intentionnaly by the user, there is nothing notmuch should\r
  98 > do -i.e no exclusion -\r
  99 > \r
 100 \r
 101 Absolutely.  Actually, that's *exactly* what I meant.\r
 102 \r
 103 I thought that would be pretty clear, but perhaps it wasn't.\r
 104 I've reworded some of the commit messages, and will pay more\r
 105 attention to it in the future.\r
 106 \r
 107 > > - To make new users aware of the config option's existence, we should\r
 108 > >   prompt them to configure it during setup.\r
 109 > \r
 110 > +1\r
 111 > \r
 112 > /Xavier\r
 113 \r
 114 Thanks for your input!\r
 115 \r
 116 \r
 117 Peace\r
 118 \r
 119 -- \r
 120 Pieter\r