0b/17999d5f2295cb18d41987dab04637cc6a6539

   1 Return-Path: <micah@riseup.net>\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 764D5431FC7\r
   6         for <notmuch@notmuchmail.org>; Wed, 27 Jan 2010 11:00:04 -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.095\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.095 tagged_above=-999 required=5 tests=[AWL=0.903,\r
  12         BAYES_50=0.001, RCVD_IN_DNSWL_LOW=-1, UNPARSEABLE_RELAY=0.001]  autolearn=ham\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 5q7O2m-dXFzQ for <notmuch@notmuchmail.org>;\r
  16         Wed, 27 Jan 2010 11:00:01 -0800 (PST)\r
  17 Received: from mx1.riseup.net (mx1.riseup.net [204.13.164.18])\r
  18         by olra.theworths.org (Postfix) with ESMTP id C3FD6431FAE\r
  19         for <notmuch@notmuchmail.org>; Wed, 27 Jan 2010 11:00:01 -0800 (PST)\r
  20 Received: from [127.0.0.1] (localhost [127.0.0.1])\r
  21         (Authenticated sender: micah@mx1.riseup.net)\r
  22         with ESMTPSA id E0B4125F69C\r
  23 Received: by lillypad (Postfix, from userid 1000)\r
  24         id 39AA52CC063; Wed, 27 Jan 2010 10:56:01 -0500 (EST)\r
  25 From: micah anderson <micah@riseup.net>\r
  26 To: Andreas =?iso-8859-1?q?Kl=F6ckner?= <lists@informa.tiker.net>,\r
  27         notmuch@notmuchmail.org\r
  28 In-Reply-To: <200912141421.52561.lists@informa.tiker.net>\r
  29 References: <200912141421.52561.lists@informa.tiker.net>\r
  30 Date: Thu, 28 Jan 2010 04:55:55 +1300\r
  31 Message-ID: <87ljfjfsok.fsf@lillypad.riseup.net>\r
  32 MIME-Version: 1.0\r
  33 Content-Type: multipart/signed; boundary="=-=-=";\r
  34         micalg=pgp-sha512; protocol="application/pgp-signature"\r
  35 X-Virus-Scanned: clamav-milter 0.95.3 at mx1\r
  36 X-Virus-Status: Clean\r
  37 Subject: Re: [notmuch] [patch] store folder information\r
  38 X-BeenThere: notmuch@notmuchmail.org\r
  39 X-Mailman-Version: 2.1.13\r
  40 Precedence: list\r
  41 List-Id: "Use and development of the notmuch mail system."\r
  42         <notmuch.notmuchmail.org>\r
  43 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  44         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  45 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  46 List-Post: <mailto:notmuch@notmuchmail.org>\r
  47 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  48 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  49         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  50 X-List-Received-Date: Wed, 27 Jan 2010 19:00:04 -0000\r
  51 \r
  52 --=-=-=\r
  53 Content-Transfer-Encoding: quoted-printable\r
  54 \r
  55 \r
  56 Hi Andreas,\r
  57 \r
  58 I'm just writing because of the patch you sent to the notmuch list on\r
  59 December 15th. It seems like many people are wanting this functionality,\r
  60 I know I am myself and Carl has also indicated the same. However, there\r
  61 were a couple of minor suggestions for improvements for your patch that\r
  62 have not seen a reply from you yet. I'm particularly eager to see this\r
  63 get accepted upstream, and it sounds like the changes necessary to do so\r
  64 are relatively minor.\r
  65 \r
  66 I'm wondering what your plans are for addressing these issues? I've come\r
  67 to depend on this functionality, and would love to see it incorporated\r
  68 upstream!=20\r
  69 \r
  70 Specifically these were:\r
  71 \r
  72 1. Unrelated whitespace:\r
  73 \r
  74 On December 16th,2009 Ruben Pollan <meskio@sindominio.net> wrote:\r
  75 \r
  76 > [meskio@blackspot:src/notmuch.orig]$ git apply ~/0001-Preseve-folder-info=\r
  77 rmation-when-indexing.patch\r
  78 > /home/meskio/0001-Preseve-folder-information-when-indexing.patch:136: tra=\r
  79 iling whitespace.\r
  80 >                status notmuch_database_add_message (notmuch, next,\r
  81 > /home/meskio/0001-Preseve-folder-information-when-indexing.patch:137: tra=\r
  82 iling whitespace.\r
  83 >                                                       folder_base_name,\r
  84 > warning: 2 lines add whitespace errors.\r
  85 >\r
  86 > It's just whitespaces at the end of the lines.\r
  87 \r
  88 2. An unrelated hunk creeping in:\r
  89 \r
  90 On Tue, 15 Dec 2009 13:22:19 -0800, Carl Worth <cworth@cworth.org> wrote:\r
  91 > On Mon, 14 Dec 2009 14:21:50 -0500, Andreas Kl=3DC3=3DB6ckner <lists@info=\r
  92 rma.=3D\r
  93 tiker.net> wrote:\r
  94 > >\r
  95 > > @@ -116,6 +116,8 @@ skip_re_in_subject (const char *subject)\r
  96 > >         s++;\r
  97 > >     if (strncasecmp (s, "re:", 3) =3D3D=3D3D 0)\r
  98 > >         s +=3D3D 3;\r
  99 > > +        else if (strncasecmp (s, "aw:", 3) =3D3D=3D3D 0)\r
 100 > > +       s +=3D3D 3;\r
 101 > >     else\r
 102 > >         break;\r
 103 > >      }\r
 104 >=3D20\r
 105 > This hunk looks unrelated to the rest. Could you submit that separately,\r
 106 > please?\r
 107 \r
 108 \r
 109 3. Redundant trailing directory name traversal:\r
 110 \r
 111 > > +    gchar *full_folder_name =3D3D NULL;\r
 112 > > +    gchar *folder_base_name =3D3D NULL;\r
 113 > > +\r
 114 > > +    /* Find name of "folder" containing the email. */\r
 115 > > +    full_folder_name =3D3D g_strdup(path);\r
 116 > > +    while (1)\r
 117 > > +    {\r
 118 > > +        folder_base_name =3D3D g_path_get_basename(full_folder_name);\r
 119 >\r
 120 > The trailing directory name is available already during the\r
 121 > traversal. So you don't need to search it back out again. See the patch\r
 122 > in the following message:\r
 123 >\r
 124 >       id:87fx8bygi7.fsf@linux.vnet.ibm.com\r
 125 >\r
 126 > which simply passes the trailing directory name along, (but skipping a\r
 127 > name of "cur" or "new" while traversing).\r
 128 \r
 129 4. supporting hierarchical folders (perhaps this is a later improvement\r
 130 that does not need to be added before the original patch is accepted?):\r
 131 \r
 132 > Beyond that, though, I imagine some people have hierarchical folders as\r
 133 > well, so it probably makes sense to store them as well.\r
 134 >\r
 135 > To do that, it's probably just a matter of calling gen_terms on the\r
 136 > complete filename. I haven't tested, but doing that should allow\r
 137 > Xapian's phrase searching to do the right thing for something like:\r
 138 >\r
 139 >       filename:portion/of/the/path/name\r
 140 \r
 141 5. Probably the patch needs to be rebased off of master at this point.\r
 142 \r
 143 Micah\r
 144 \r
 145 --=-=-=\r
 146 Content-Type: application/pgp-signature\r
 147 \r
 148 -----BEGIN PGP SIGNATURE-----\r
 149 Version: GnuPG v1.4.10 (GNU/Linux)\r
 150 \r
 151 iQIcBAEBCgAGBQJLYGIMAAoJEIy/mjIoYaeQxogP/2AV4t+6G50aqwKs9EAY9Ewq\r
 152 PRY75DLhuBbf5pYi6/s8eurMhtYDRZIuc1E9XgDc5goAE9vLuQQ2I2qwKIdVJFbt\r
 153 mv5gcLzE9VjVa+W0rvxQTFxIAGbDNZ4EuWs6ajl9uvrmioCTvXH0pP7CQ5bWckuX\r
 154 /jJr28w1JjgMiP6Wz9PQ5JCxg7WSgsTxsqv2XskR+J9XHjJiSAQdr7lhLsBUUZjY\r
 155 nqcgvC5YbmGwaJkFpmEWcb9gKp+QjyTJ3hpQJDfOyst/+CzL99Z1Dw6ODKq+98sf\r
 156 0AxixzleGhaIVush4wKI1IYI7l6x3YBHAMVFE42fyJqKBazO8VGCULUMsOwBSk01\r
 157 Bh5iaOTrW5NwAUfvsjKsBvYZhcszMvmRkfINfoRAs3G4DziJ2QmDScwBDVfWYR/5\r
 158 7sqLHRBPsyoogC+HonNk0hKy3BjfYZZWZr74M7Z2Vi44BR3ATlbxxxvGwSx6S9ig\r
 159 k1LjzGz4WOax35Q4TntrdbVV7IDyBafpy7mBgavm9x5kAcCjbk/auFrFZESzmTjZ\r
 160 zDB4/ea7g8tJp6Ky2Q6YkzZrh5B6FjV7HQf3cxQuutgaXX2fcexJTflxubGJZYBx\r
 161 2VmYVveKTgFtGJFGxyH8MIxLlBHohascFDTN3uO5bEbZYf3QYQx6mTI19IB9+4k8\r
 162 BsFiJMOulwjoLJ5rEqtm\r
 163 =Oye4\r
 164 -----END PGP SIGNATURE-----\r
 165 --=-=-=--\r