d5/c91d965e75b80fbd41cc8a1ea0bc4a1384ed9c

   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 6F138431FBC\r
   6         for <notmuch@notmuchmail.org>; Fri, 22 Jan 2010 21:48:43 -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.853\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-1.853 tagged_above=-999 required=5\r
  12         tests=[ALL_TRUSTED=-1.8, AWL=-0.054, BAYES_50=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 qm6kUnb7K8nW; Fri, 22 Jan 2010 21:48:42 -0800 (PST)\r
  16 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  17         by olra.theworths.org (Postfix) with ESMTP id 8B0A2431FAE;\r
  18         Fri, 22 Jan 2010 21:48:42 -0800 (PST)\r
  19 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  20         id AFE3025409D; Sat, 23 Jan 2010 18:48:41 +1300 (NZDT)\r
  21 From: Carl Worth <cworth@cworth.org>\r
  22 To: Geo Carncross <geocar-notmuch@internetconnection.net>,\r
  23         notmuch@notmuchmail.org\r
  24 In-Reply-To: <1264104618.3941.108.camel@dwarf>\r
  25 References: <1264104618.3941.108.camel@dwarf>\r
  26 Date: Sat, 23 Jan 2010 18:48:35 +1300\r
  27 Message-ID: <87636tz80c.fsf@yoom.home.cworth.org>\r
  28 MIME-Version: 1.0\r
  29 Content-Type: multipart/signed; boundary="=-=-=";\r
  30         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  31 Subject: Re: [notmuch] [PATCH] d_type fix\r
  32 X-BeenThere: notmuch@notmuchmail.org\r
  33 X-Mailman-Version: 2.1.13\r
  34 Precedence: list\r
  35 List-Id: "Use and development of the notmuch mail system."\r
  36         <notmuch.notmuchmail.org>\r
  37 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  38         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  39 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  40 List-Post: <mailto:notmuch@notmuchmail.org>\r
  41 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  42 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  43         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  44 X-List-Received-Date: Sat, 23 Jan 2010 05:48:43 -0000\r
  45 \r
  46 --=-=-=\r
  47 \r
  48 On Thu, 21 Jan 2010 15:10:18 -0500, Geo Carncross <geocar-notmuch@internetconnection.net> wrote:\r
  49 > A review of notmuch-new.c shows three uses of ->d_type:\r
  50 \r
  51 Excellent, thank you.\r
  52 \r
  53 > Near line 153, in _entries_resemble_maildir() we can simply allow for\r
  54 > DT_UNKNOWN. This would fail if people have MH-style folders which have\r
  55 > three folders called "new" "cur" and "tmp", but that seems unlikely, in\r
  56 > which case the "tmp" folder would simply not be scanned.\r
  57 \r
  58 The check will also fail if someone happens to have two regular files\r
  59 named "cur" and "new" next to a directory named "tmp", (which would then\r
  60 be ignored). But I agree this seems very unlikely.\r
  61 \r
  62 > [snip]\r
  63 \r
  64 The rest of your analysis looks good to me. And it's nice to see how\r
  65 simple the final patch is.\r
  66 \r
  67 > Attached is a patch which was tested with reiserfs. It should also work\r
  68 > with xfs.\r
  69 \r
  70 I had been waiting to apply the previous patch until I could test it on\r
  71 a filesystem returning DT_UNKNOWN. But this one looks clean enough to\r
  72 apply immediately.\r
  73 \r
  74 One thing that would have made that easier would be a commit message,\r
  75 (such as making a commit with "git commit" then using "git format-patch"\r
  76 and "git send-email" to send it). But I'll go ahead and make one up.\r
  77 \r
  78 Thanks again!\r
  79 \r
  80 -Carl\r
  81 \r
  82 --=-=-=\r
  83 Content-Type: application/pgp-signature\r
  84 \r
  85 -----BEGIN PGP SIGNATURE-----\r
  86 Version: GnuPG v1.4.10 (GNU/Linux)\r
  87 \r
  88 iD8DBQFLWo2z6JDdNq8qSWgRAry8AJ0eftnGesu7wGTK5KjQSOGAHqL9swCfSzIQ\r
  89 si3ucTxP72CWo48oubhKBGA=\r
  90 =G2XC\r
  91 -----END PGP SIGNATURE-----\r
  92 --=-=-=--\r