b9/a2889dfdc4565a6de5689c6c258841c71ff63b

   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 439CA431FBC\r
   6         for <notmuch@notmuchmail.org>; Thu, 14 Jan 2010 14:42: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.825\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-1.825 tagged_above=-999 required=5\r
  12         tests=[ALL_TRUSTED=-1.8, AWL=-0.026, 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 Nz-OTXjGLAoP; Thu, 14 Jan 2010 14:42: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 72787431FAE;\r
  18         Thu, 14 Jan 2010 14:42:42 -0800 (PST)\r
  19 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  20         id E0A01550090; Thu, 14 Jan 2010 14:42:41 -0800 (PST)\r
  21 From: Carl Worth <cworth@cworth.org>\r
  22 To: Adrian Perez de Castro <aperez@igalia.com>, notmuch@notmuchmail.org\r
  23 In-Reply-To: <20100114183854.1d04f111@hikari>\r
  24 References: <4B4ED7E8.20501@exys.org> <878wc0623y.fsf@exys.org>\r
  25         <20100114183854.1d04f111@hikari>\r
  26 Date: Thu, 14 Jan 2010 14:42:41 -0800\r
  27 Message-ID: <87fx68e2am.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] indexing mail?\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: Thu, 14 Jan 2010 22:42:43 -0000\r
  45 \r
  46 --=-=-=\r
  47 Content-Type: text/plain; charset=utf-8\r
  48 Content-Transfer-Encoding: quoted-printable\r
  49 \r
  50 On Thu, 14 Jan 2010 18:38:54 +0100, Adrian Perez de Castro <aperez@igalia.c=\r
  51 om> wrote:\r
  52 > > the offending commit is 2c4555f1a56602ff1dd55a63699810522ba4d91e\r
  53 > >=20\r
  54 > > from readdir (3):\r
  55 > >=20\r
  56 > >      "Currently, only some file systems (among them: Btrfs, ext2, ext3,\r
  57 > >      and ext4) have full  support  returning  the  file\r
  58 > >        type in d_type.  All applications must properly handle a return\r
  59 > >      of DT_UNKNOWN."\r
  60 \r
  61 Yes. The broken code was my mistake. I clearly didn't read the above\r
  62 warning closely enough. Sorry about that!\r
  63 \r
  64 > I am using XFS, which always returns DT_UNKNOWN. Taking into account that\r
  65 > there is a good deal of people using filesystems other than the ones you\r
  66 > mention, and that other non-linux filesystems may also return DT_UNKNOWN,\r
  67 > in my opinion there should be a fall-back. I will try to post a patch\r
  68 > Anytime Soon=E2=84=A2.\r
  69 \r
  70 We definitely want the fallback. I can attempt to code it, but I don't\r
  71 have ready access to an afflicted filesystem, so I'd need help testing\r
  72 anyway.\r
  73 \r
  74 I'd love to see a patch for this bug soon. Be sure to CC me when the\r
  75 patch is sent and that will help me commit it sooner.\r
  76 \r
  77 > Also, I have the feeling that the "d_type" field from "struct dirent" may\r
  78 > not be available in some OSes because it is a BSD extension.\r
  79 \r
  80 I'm generally quite bad at determining whether functionality I'm using\r
  81 in my software is non-portable. As proven in this case, even when the\r
  82 man page tells me something is not portable I don't always notice, (and\r
  83 often, the man pages aren't even that useful).\r
  84 \r
  85 Beyond that, even if something is *known* to be theoretically\r
  86 non-portable, it can be a waste of time to code compatibility paths that\r
  87 nobody will be running in practice.\r
  88 \r
  89 So I've basically gotten to the point where I just code for what works\r
  90 on my system, (not out of disregard for what other people run---just\r
  91 that it's impossible for me to know what subset of functionality is\r
  92 actually relevant). Then, at the same time, I'm quite happy to accept\r
  93 code to improve the portability when people note that things are broken\r
  94 on other systems.\r
  95 \r
  96 See the git history and email archives for examples of how we fixed\r
  97 strndup and getline portability problems.\r
  98 \r
  99 I know that "wait for people to notice it's broken" isn't the nicest\r
 100 thing we could do with our code. But I don't really know a much better\r
 101 way. I'm happy to entertain suggestions here.\r
 102 \r
 103 =2DCarl\r
 104 \r
 105 --=-=-=\r
 106 Content-Type: application/pgp-signature\r
 107 \r
 108 -----BEGIN PGP SIGNATURE-----\r
 109 Version: GnuPG v1.4.10 (GNU/Linux)\r
 110 \r
 111 iD8DBQFLT53h6JDdNq8qSWgRArt3AJwJwYb7MWcyM4KqxJrsmrPcDPL9LgCgkdvY\r
 112 hCOpr8bn3wgfuIQCcb+BpPU=\r
 113 =s9vi\r
 114 -----END PGP SIGNATURE-----\r
 115 --=-=-=--\r