97/c7b6bea030a87fa3626afd00958ee40c3f17ee

   1 Return-Path: <anarcat@anarcat.ath.cx>\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 ABC79429E35\r
   6         for <notmuch@notmuchmail.org>; Mon, 16 Jan 2012 21:12:38 -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: 1.274\r
  10 X-Spam-Level: *\r
  11 X-Spam-Status: No, score=1.274 tagged_above=-999 required=5\r
  12         tests=[RDNS_NONE=1.274] 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 URAsju-Nz5PV for <notmuch@notmuchmail.org>;\r
  16         Mon, 16 Jan 2012 21:12:38 -0800 (PST)\r
  17 Received: from marcos.anarcat.ath.cx (unknown [72.0.72.144])\r
  18         by olra.theworths.org (Postfix) with ESMTP id DC5B4429E2F\r
  19         for <notmuch@notmuchmail.org>; Mon, 16 Jan 2012 21:12:37 -0800 (PST)\r
  20 Received: by marcos.anarcat.ath.cx (Postfix, from userid 1000)\r
  21         id 621A0143139; Tue, 17 Jan 2012 00:12:37 -0500 (EST)\r
  22 From: Antoine =?utf-8?Q?Beaupr=C3=A9?= <anarcat@koumbit.org>\r
  23 To: Jani Nikula <jani@nikula.org>, notmuch@notmuchmail.org\r
  24 Subject: Re: [PATCH 1/2] lib: Add back the synchronization of 'T' flag with\r
  25         deleted tag.\r
  26 In-Reply-To: <874nw88nl8.fsf@nikula.org>\r
  27 References: <1310874973-28437-1-git-send-email-anarcat@koumbit.org>\r
  28         <874nw88nl8.fsf@nikula.org>\r
  29 User-Agent: Notmuch/0.10.2 (http://notmuchmail.org) Emacs/23.3.1\r
  30         (x86_64-pc-linux-gnu)\r
  31 Date: Tue, 17 Jan 2012 00:12:36 -0500\r
  32 Message-ID: <87ehuykj3v.fsf@marcos.anarcat.ath.cx>\r
  33 MIME-Version: 1.0\r
  34 Content-Type: multipart/signed; boundary="=-=-=";\r
  35         micalg=pgp-sha256; protocol="application/pgp-signature"\r
  36 X-BeenThere: notmuch@notmuchmail.org\r
  37 X-Mailman-Version: 2.1.13\r
  38 Precedence: list\r
  39 List-Id: "Use and development of the notmuch mail system."\r
  40         <notmuch.notmuchmail.org>\r
  41 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  42         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  43 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  44 List-Post: <mailto:notmuch@notmuchmail.org>\r
  45 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  46 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  47         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  48 X-List-Received-Date: Tue, 17 Jan 2012 05:12:38 -0000\r
  49 \r
  50 --=-=-=\r
  51 Content-Type: text/plain; charset=utf-8\r
  52 Content-Transfer-Encoding: quoted-printable\r
  53 \r
  54 On Sat, 07 Jan 2012 00:37:07 +0200, Jani Nikula <jani@nikula.org> wrote:\r
  55 > On Sat, 16 Jul 2011 23:56:12 -0400, Antoine Beaupr=C3=A9 <anarcat@koumbit=\r
  56 .org> wrote:\r
  57 > > +    // TODO: this should probably be moved up in the stack to avoid\r
  58 > > +    // opening the config file on every message (!)\r
  59 > > +    config =3D notmuch_config_open (ctx, NULL, NULL);\r
  60 >=20\r
  61 > The config file is for notmuch the command line tool, *not* for the\r
  62 > lib. You can't call the cli from from the lib. The config (or command\r
  63 > line argument) should be passed as argument, but that would require\r
  64 > changing the lib interface.\r
  65 \r
  66 I see. I wasn't aware of that.\r
  67 \r
  68 > > + *   'T' iff the message has the "trashed" tag and\r
  69 > > + *   state->reckless_trash is TRUE.\r
  70 >=20\r
  71 > "trashed" tag?\r
  72 \r
  73 That should probably be "deleted".\r
  74 \r
  75 > The comment (and the commit message) is incorrect. You only check for\r
  76 > reckless_trash in maildir_flags_to_tags, not tags_to_maildir_flags.\r
  77 > With this patch, one-way syncing from tags to flags would be done\r
  78 > unconditionally. And if I understand the problem correctly, you're\r
  79 > fixing the less critical one of the two!\r
  80 \r
  81 Indeed! What an oversight...\r
  82 \r
  83 > I am wondering (but I'm too tired to check) if the original problem\r
  84 > could be avoided by simply refusing to sync "deleted" tag to 'T' flag if\r
  85 > there are more than one file for that message.\r
  86 \r
  87 That would be a great idea.\r
  88 \r
  89 > This is a dangerous feature, which is why it was originally\r
  90 > disabled. Accidentally deleting mail is not something people take\r
  91 > lightly. They'll be amused by "reckless trash" - until it recklessly\r
  92 > deletes an important mail.\r
  93 \r
  94 Yes, I understand this.\r
  95 \r
  96 > However, something like this might be a useful feature to have for\r
  97 > people who want to delete mail. It would need good tests to accompany\r
  98 > it, though.\r
  99 \r
 100 And to be honest, that's where I got off the boat. :) It just got too\r
 101 hard, and anyways I use a custom script that deletes mails from notmuch\r
 102 search tag:deleted, so syncing that flag isn't so important for me.\r
 103 \r
 104 I guess this got everything covered for me. I would be ready to accept\r
 105 this patch being dropped from the queue, although I think it's a key\r
 106 step in having a more general tag to maildir flags synchronisation\r
 107 strategy that would allow to run notmuch from multiple clients, without\r
 108 having to sync databases around.\r
 109 \r
 110 Thanks for the review, this patch is indeed not ready, and I am not sure\r
 111 when I will have time to push it further.\r
 112 \r
 113 Cheers,\r
 114 \r
 115 A.\r
 116 \r
 117 =2D-=20\r
 118 Modern man has a kind of poverty of the spirit which stands\r
 119 in great contrast to his remarkable scientific and technological\r
 120 achievements. We've learned to walk in outer space and yet we\r
 121 haven't learned to walk to earth as brothers and sisters.\r
 122                         - Dr. Martin Luther King, Jr.\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 iQIcBAEBCAAGBQJPFQNEAAoJEHkhUlJ7dZIeSMgP/0sr+KJ+BPf7yU7i3H8rgVe/\r
 131 URVWT1Kt4vnVu0WHMyAxQJMghRgwHi3HHDDHP7CnN98r12Gu7qhtEGbLirv6Bgz5\r
 132 V0qW3NTpGsJxdkU00L9qNDWBi1zjuTDV+NnjgiwU+CphMwFYiowJzzi6hS2frWPl\r
 133 3A1hB+xg2p4Izvx6LdYXY/2hpXnY+zj9K8hzb/ui1/pX0bOxLfbtN6zd4cLfZ4NU\r
 134 WjKHQLb/1+sZjB6ZxL6pHWVv80b4EpdEHxaqxxpOw53zQSOZlXN31aUnCfpVmBsg\r
 135 gJwZV0MtHbT8+tq18c4C+dI7s4dJBJzOwbE4Wd5vLtMl/bAw88aBVzc3t/s2tQJw\r
 136 5/ODLnt5+7PDQZ73c0pjSNLOzCX+YCvtaWTOyTP1nHQJSNvP/n+sVDdfI/5l2T88\r
 137 61Q0doofSUgYl6NNLPpy/gYfgwQ2d7FzVb0lwd5Imq6H5OkVYPpincxNh+vV1TvA\r
 138 bvgHI8YLCwdkPqBrII1KA6ERmQhKbiJT7mdZocT7A92W4bdQzcfHPUmKqCQuWZm3\r
 139 BUGze8h9s/vT9sMHY3FhvRGWKf3mdseKYVITM6hrw29nFqcY2TWrmJoQah7ZkErK\r
 140 qhFR7Ox1P6MPa5K44gCrtVmOewMrOqeRt12rBUOj8SJnzyUuHxTvNc7vNq9XpoSC\r
 141 Rr+4wNmAq1dYoAzTesUu\r
 142 =pOm8\r
 143 -----END PGP SIGNATURE-----\r
 144 --=-=-=--\r