ba/ada1f0362baf36d703ce6aba2f0086d58c54c9

   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 5E593431FB6\r
   6         for <notmuch@notmuchmail.org>; Fri, 28 Jan 2011 14:02:52 -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.99\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.99 tagged_above=-999 required=5\r
  12         tests=[ALL_TRUSTED=-1, 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 5WMHylK+vv-X; Fri, 28 Jan 2011 14:02:51 -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 8281F431FB5;\r
  18         Fri, 28 Jan 2011 14:02:50 -0800 (PST)\r
  19 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  20         id 3952E2540DE; Sat, 29 Jan 2011 05:59:51 +1000 (EST)\r
  21 From: Carl Worth <cworth@cworth.org>\r
  22 To: Thomas Schwinge <thomas@schwinge.name>, notmuch@notmuchmail.org\r
  23 Subject: Re: [PATCH] Add a few tests for searching LWN emails.\r
  24 In-Reply-To: <1296124309-20072-1-git-send-email-thomas@schwinge.name>\r
  25 References: <1296124309-20072-1-git-send-email-thomas@schwinge.name>\r
  26 User-Agent: Notmuch/0.5 (http://notmuchmail.org) Emacs/23.2.1\r
  27         (i486-pc-linux-gnu)\r
  28 Date: Sat, 29 Jan 2011 05:59:38 +1000\r
  29 Message-ID: <87vd18n6j9.fsf@yoom.home.cworth.org>\r
  30 MIME-Version: 1.0\r
  31 Content-Type: multipart/signed; boundary="=-=-=";\r
  32         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  33 Cc: lwn@lwn.net, Thomas Schwinge <thomas@schwinge.name>\r
  34 X-BeenThere: notmuch@notmuchmail.org\r
  35 X-Mailman-Version: 2.1.13\r
  36 Precedence: list\r
  37 List-Id: "Use and development of the notmuch mail system."\r
  38         <notmuch.notmuchmail.org>\r
  39 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  40         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  41 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  42 List-Post: <mailto:notmuch@notmuchmail.org>\r
  43 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  44 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  45         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  46 X-List-Received-Date: Fri, 28 Jan 2011 22:02:52 -0000\r
  47 \r
  48 --=-=-=\r
  49 \r
  50 On Thu, 27 Jan 2011 03:31:49 -0700, Thomas Schwinge <thomas@schwinge.name> wrote:\r
  51 > These tests should pass -- but they currently don't.\r
  52 \r
  53 Hi Thomas,\r
  54 \r
  55 Thanks for sending these test cases. This is actually my favorite way to\r
  56 receive bug reports. I really appreciate it!\r
  57 \r
  58 > What we get from these emails, is an author named ``LWN.net'', and the\r
  59 > ``Weekly Notification'' / ``Mailing Lists'' bits are stripped away.  I\r
  60 > suspect this may be a misinterpretation in the notmuch address parser,\r
  61 > related to the dot in the name.  I have not yet looked at the relevant\r
  62 > code.\r
  63 \r
  64 Yes, I believe this is related to the dot in the name. From my\r
  65 recollection a name with an address requires quoting. So the header that\r
  66 is currently formatted as:\r
  67 \r
  68         From: LWN.net Weekly Notification <lwn@lwn.net>\r
  69 \r
  70 should instead be:\r
  71 \r
  72         From: "LWN.net Weekly Notification" <lwn@lwn.net>\r
  73 \r
  74 I verified that adding this quoting fixes the tests[*]. I've CCed\r
  75 lwn@lwn.net in case the kind editors there would like to add quoting\r
  76 here.\r
  77 \r
  78 I don't have any of the relevant RFCs in front of me now, so I don't\r
  79 know exactly how an address with this missing quoting should be\r
  80 parsed. But I recall having failures trying to send mail to an address\r
  81 like this formatted without the quoting.\r
  82 \r
  83 So I'm not sure what could reasonably be changed here in GMime or not. I\r
  84 definitely do want to fix notmuch so that it indexes all of the text\r
  85 here regardless if it's formatted in an RFC-compliant way or not.\r
  86 \r
  87 -Carl\r
  88 \r
  89 [*] Except for test VI which has a bug in that it searches for the word\r
  90 "mailing" in the subject header, but no such word exists in the\r
  91 message's subject header.\r
  92 \r
  93 --=-=-=\r
  94 Content-Type: application/pgp-signature\r
  95 \r
  96 -----BEGIN PGP SIGNATURE-----\r
  97 Version: GnuPG v1.4.10 (GNU/Linux)\r
  98 \r
  99 iD8DBQFNQyAq6JDdNq8qSWgRAo/XAJ9HPYSosTjl8QwYBqBsJmv6G2xB0wCffdm8\r
 100 wK5/lhf1VR9BKv0bviGpuM8=\r
 101 =Hxun\r
 102 -----END PGP SIGNATURE-----\r
 103 --=-=-=--\r