2c/d7d57bedd3dcfbc2c317f6afdb5b8e066297ae

   1 Return-Path: <cworth@cworth.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 14FCE429E55\r
   6         for <notmuch@notmuchmail.org>; Wed, 29 Jun 2011 06:47:41 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 0.01\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0.01 tagged_above=-999 required=5\r
  12         tests=[T_MIME_NO_TEXT=0.01] 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 EL03n5CDXGLk for <notmuch@notmuchmail.org>;\r
  16         Wed, 29 Jun 2011 06:47:39 -0700 (PDT)\r
  17 Received: from arlo.cworth.org (arlo.cworth.org [50.43.72.2])\r
  18         by olra.theworths.org (Postfix) with ESMTP id 3C5D4429E32\r
  19         for <notmuch@notmuchmail.org>; Wed, 29 Jun 2011 06:47:39 -0700 (PDT)\r
  20 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  21         by arlo.cworth.org (Postfix) with ESMTP id B694D29A4FD;\r
  22         Wed, 29 Jun 2011 06:47:37 -0700 (PDT)\r
  23 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  24         id A702E2541A8; Wed, 29 Jun 2011 06:47:37 -0700 (PDT)\r
  25 From: Carl Worth <cworth@cworth.org>\r
  26 To: Austin Clements <amdragon@MIT.EDU>, notmuch@notmuchmail.org\r
  27 Subject: Re: [PATCH 1/2] new: Don't update DB mtime if FS mtime equals\r
  28         wall-clock time.\r
  29 In-Reply-To: <1309331455-19584-1-git-send-email-amdragon@mit.edu>\r
  30 References: <1309331455-19584-1-git-send-email-amdragon@mit.edu>\r
  31 User-Agent: Notmuch/0.5 (http://notmuchmail.org) Emacs/23.3.1\r
  32         (i486-pc-linux-gnu)\r
  33 Date: Wed, 29 Jun 2011 06:47:37 -0700\r
  34 Message-ID: <87wrg4696u.fsf@yoom.home.cworth.org>\r
  35 MIME-Version: 1.0\r
  36 Content-Type: multipart/signed; boundary="=-=-=";\r
  37         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  38 Cc: amdragon@mit.edu\r
  39 X-BeenThere: notmuch@notmuchmail.org\r
  40 X-Mailman-Version: 2.1.13\r
  41 Precedence: list\r
  42 List-Id: "Use and development of the notmuch mail system."\r
  43         <notmuch.notmuchmail.org>\r
  44 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  45         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  46 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  47 List-Post: <mailto:notmuch@notmuchmail.org>\r
  48 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  49 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  50         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  51 X-List-Received-Date: Wed, 29 Jun 2011 13:47:41 -0000\r
  52 \r
  53 --=-=-=\r
  54 Content-Transfer-Encoding: quoted-printable\r
  55 \r
  56 On Wed, 29 Jun 2011 03:10:54 -0400, Austin Clements <amdragon@MIT.EDU> wrot=\r
  57 e:\r
  58 > This fixes a race where multiple message deliveries in the same second\r
  59 > with an intervening notmuch new could result in messages being ignored\r
  60 > by notmuch (at least, until a later delivery forced a rescan).\r
  61 \r
  62 Thanks for the fix, Austin!\r
  63 \r
  64 > +     * XXX Bug workaround: If this is a new directory, we *must*\r
  65 > +     * update the mtime; otherwise the next run will see the 0 mtime\r
  66 > +     * and think this is still a new directory containing no files or\r
  67 > +     * subdirs (which is unsound in general).  If fs_mtime =3D=3D\r
  68 > +     * stat_time, we set the database mtime to a bogus (but non-zero!)\r
  69 > +     * value to force a rescan.\r
  70 \r
  71 I like to reserve "XXX" as an indication that some further work is\r
  72 necessary.\r
  73 \r
  74 That doesn't seem to be the case here, (instead, you seem to have\r
  75 thought this issue out quite fully).\r
  76 \r
  77 Other than that, I'm quite happy with the patch.\r
  78 \r
  79 =2DCarl\r
  80 \r
  81 =2D-=20\r
  82 carl.d.worth@intel.com\r
  83 \r
  84 --=-=-=\r
  85 Content-Type: application/pgp-signature\r
  86 \r
  87 -----BEGIN PGP SIGNATURE-----\r
  88 Version: GnuPG v1.4.11 (GNU/Linux)\r
  89 \r
  90 iEYEARECAAYFAk4LLPkACgkQ6JDdNq8qSWi/cgCdH0f6usJhagV/163fW1l5EX46\r
  91 avYAnAqoGk9CHrh1/r94OoHgZ42CwoEW\r
  92 =6aoN\r
  93 -----END PGP SIGNATURE-----\r
  94 --=-=-=--\r