f6/90de24e9c434e5771cefe87919204cdf4a963e

   1 Return-Path: <dme@dme.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 B59BD429E5F\r
   6         for <notmuch@notmuchmail.org>; Wed, 18 Jan 2012 00:38:33 -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 8ITFHExyT9cp for <notmuch@notmuchmail.org>;\r
  16         Wed, 18 Jan 2012 00:38:33 -0800 (PST)\r
  17 Received: from mail-ww0-f45.google.com (mail-ww0-f45.google.com\r
  18  [74.125.82.45])        (using TLSv1 with cipher RC4-SHA (128/128 bits))        (No client\r
  19  certificate requested) by olra.theworths.org (Postfix) with ESMTPS id\r
  20  D5D59429E27    for <notmuch@notmuchmail.org>; Wed, 18 Jan 2012 00:38:32 -0800\r
  21  (PST)\r
  22 Received: by wgbdr13 with SMTP id dr13so2093414wgb.2\r
  23         for <notmuch@notmuchmail.org>; Wed, 18 Jan 2012 00:38:31 -0800 (PST)\r
  24 Received: by 10.180.108.232 with SMTP id hn8mr8722964wib.16.1326875911534;\r
  25         Wed, 18 Jan 2012 00:38:31 -0800 (PST)\r
  26 Received: from hotblack-desiato.hh.sledj.net\r
  27         (host81-149-164-25.in-addr.btopenworld.com. [81.149.164.25])\r
  28         by mx.google.com with ESMTPS id r1sm21194301wia.8.2012.01.18.00.38.29\r
  29         (version=TLSv1/SSLv3 cipher=OTHER);\r
  30         Wed, 18 Jan 2012 00:38:30 -0800 (PST)\r
  31 Received: by hotblack-desiato.hh.sledj.net (Postfix, from userid 30000)\r
  32         id 8E1BBA0397; Wed, 18 Jan 2012 08:38:28 +0000 (GMT)\r
  33 To: Austin Clements <amdragon@MIT.EDU>\r
  34 Subject: Re: [PATCH v2 3/3] search: Support automatic tag exclusions\r
  35 In-Reply-To: <20120117203211.GQ16740@mit.edu>\r
  36 References: <1326258173-21163-1-git-send-email-amdragon@mit.edu>\r
  37         <1326496024-14403-1-git-send-email-amdragon@mit.edu>\r
  38         <1326496024-14403-4-git-send-email-amdragon@mit.edu>\r
  39         <87wr8tsvit.fsf@servo.finestructure.net>\r
  40         <cunfwfg57uh.fsf@hotblack-desiato.hh.sledj.net>\r
  41         <20120116192836.GC16740@mit.edu>\r
  42         <CA+eQo_3xxuhgUUXWXWyVD1LFhvhkw2psbA3ZnFnZk=BjjHXy8w@mail.gmail.com>\r
  43         <cunpqeir912.fsf@hotblack-desiato.hh.sledj.net>\r
  44         <20120117203211.GQ16740@mit.edu>\r
  45 User-Agent: Notmuch/0.11+64~g42e8f66 (http://notmuchmail.org) Emacs/24.0.92.1\r
  46         (x86_64-pc-linux-gnu)\r
  47 From: David Edmondson <dme@dme.org>\r
  48 Date: Wed, 18 Jan 2012 08:38:23 +0000\r
  49 Message-ID: <cunsjjdmmm8.fsf@hotblack-desiato.hh.sledj.net>\r
  50 MIME-Version: 1.0\r
  51 Content-Type: multipart/signed; boundary="=-=-=";\r
  52         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  53 Cc: notmuch@notmuchmail.org, Jeremy Nickurak <jeremy@nickurak.ca>\r
  54 X-BeenThere: notmuch@notmuchmail.org\r
  55 X-Mailman-Version: 2.1.13\r
  56 Precedence: list\r
  57 List-Id: "Use and development of the notmuch mail system."\r
  58         <notmuch.notmuchmail.org>\r
  59 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  60         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  61 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  62 List-Post: <mailto:notmuch@notmuchmail.org>\r
  63 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  64 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  65         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  66 X-List-Received-Date: Wed, 18 Jan 2012 08:38:33 -0000\r
  67 \r
  68 --=-=-=\r
  69 Content-Type: text/plain\r
  70 Content-Transfer-Encoding: quoted-printable\r
  71 \r
  72 On Tue, 17 Jan 2012 15:32:11 -0500, Austin Clements <amdragon@MIT.EDU> wrot=\r
  73 e:\r
  74 > Quoth David Edmondson on Jan 17 at  9:08 am:\r
  75 > > On Mon, 16 Jan 2012 15:16:24 -0700, Jeremy Nickurak <jeremy@nickurak.ca=\r
  76 > wrote:\r
  77 > > > On Mon, Jan 16, 2012 at 12:28, Austin Clements <amdragon@mit.edu> wro=\r
  78 te:\r
  79 > > > >> Having "deleted" and "spam" as default settings in the configurati=\r
  80 on\r
  81 > > > >> file might be more reasonable.\r
  82 > > >=20\r
  83 > > > If I read correctly:\r
  84 > > >=20\r
  85 > > > 1) If no exclude options are in the config file, none should be used.\r
  86 > >=20\r
  87 > > Yes.\r
  88 > >=20\r
  89 > > > 2) On notmuch setup, "deleted" and "spam" should be added to .notmuch=\r
  90 -config\r
  91 > >=20\r
  92 > > I might argue between 'should' and 'could', but the sense is correct.\r
  93 >=20\r
  94 > Oh, I think I see.  I don't know if I can do precisely that, since the\r
  95 > config code doesn't know if it's being called from setup, but is\r
  96 > something like this essentially what you're suggesting?\r
  97 >=20\r
  98 >     if (notmuch_config_get_auto_exclude_tags (config, &tmp) =3D=3D NULL) {\r
  99 >         if (is_new) {\r
 100 >             const char *tags[] =3D { "deleted", "spam" };\r
 101 >             notmuch_config_set_auto_exclude_tags (config, tags, 2);\r
 102 >         } else {\r
 103 >             notmuch_config_set_auto_exclude_tags (config, NULL, 0);\r
 104 >         }\r
 105 >     }\r
 106 >=20\r
 107 > (where is_new is TRUE if this is a brand-new config file)\r
 108 \r
 109 I'm not sure, as I haven't looked at the configuration code at\r
 110 all, sorry.\r
 111 \r
 112 Something must create the initial configuration file if none exists. I'd\r
 113 be okay with that code adding 'deleted' and 'spam' to the excluded list.\r
 114 \r
 115 This would mean that an existing user would see no change without taking\r
 116 some action (adding the tags to the configuration file) and a new user\r
 117 would see the new behaviour (automatic exclusion).\r
 118 \r
 119 I'm not completely sure that automatically adding the exclusion of the\r
 120 specified tags via the configuration file for new users is a great\r
 121 idea. It seems as though it will lead to confusion for someone at some\r
 122 point.\r
 123 \r
 124 --=-=-=\r
 125 Content-Type: application/pgp-signature\r
 126 \r
 127 -----BEGIN PGP SIGNATURE-----\r
 128 Version: GnuPG v1.4.11 (GNU/Linux)\r
 129 \r
 130 iEYEARECAAYFAk8WhP8ACgkQaezQq/BJZRYAMwCfWgPJowkOhz1cu+x8kHwvNmkU\r
 131 5iwAniMvaMJLEJORlRwOmHJJLCOjT2J1\r
 132 =ppVY\r
 133 -----END PGP SIGNATURE-----\r
 134 --=-=-=--\r