f9/c0c2ed721d895ee98cad00f64cb8916ccd4fee

   1 Return-Path: <pieter@praet.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 1810C431FD0\r
   6         for <notmuch@notmuchmail.org>; Wed, 27 Apr 2011 12:59:19 -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.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] 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 A4aTwZZjlrVZ for <notmuch@notmuchmail.org>;\r
  16         Wed, 27 Apr 2011 12:59:16 -0700 (PDT)\r
  17 Received: from mail-ww0-f45.google.com (mail-ww0-f45.google.com\r
  18  [74.125.82.45])        (using TLSv1 with cipher RC4-SHA (128/128 bits))        (No client\r
  19  certificate requested) by olra.theworths.org (Postfix) with ESMTPS id\r
  20  83C19431FB5    for <notmuch@notmuchmail.org>; Wed, 27 Apr 2011 12:59:15 -0700\r
  21  (PDT)\r
  22 Received: by wwi36 with SMTP id 36so1903391wwi.2\r
  23         for <notmuch@notmuchmail.org>; Wed, 27 Apr 2011 12:59:12 -0700 (PDT)\r
  24 Received: by 10.227.149.142 with SMTP id t14mr2577577wbv.170.1303934352080;\r
  25         Wed, 27 Apr 2011 12:59:12 -0700 (PDT)\r
  26 Received: from localhost (202.69-240-81.adsl-dyn.isp.belgacom.be\r
  27         [81.240.69.202])\r
  28         by mx.google.com with ESMTPS id b20sm663358wbb.67.2011.04.27.12.59.10\r
  29         (version=TLSv1/SSLv3 cipher=OTHER);\r
  30         Wed, 27 Apr 2011 12:59:11 -0700 (PDT)\r
  31 From: Pieter Praet <pieter@praet.org>\r
  32 To: Xavier Maillard <xma@gnu.org>, Carl Worth <cworth@cworth.org>,\r
  33         Matthieu Lemerre <racin@free.fr>, notmuch@notmuchmail.org\r
  34 Subject: Re: Warning when GMime is parsing broken email addresses\r
  35 In-Reply-To: <m2y62vy7vy.fsf@gnu.org>\r
  36 References: <87ipzvk2xh.fsf@free.fr> <877hai3r82.fsf@yoom.home.cworth.org>\r
  37         <m2y62vy7vy.fsf@gnu.org>\r
  38 User-Agent: Notmuch/0.5-98-g63c37b1 (http://notmuchmail.org) Emacs/23.1.50.1\r
  39         (x86_64-pc-linux-gnu)\r
  40 Date: Wed, 27 Apr 2011 21:59:09 +0200\r
  41 Message-ID: <87r58n8nzm.fsf@praet.org>\r
  42 MIME-Version: 1.0\r
  43 Content-Type: text/plain; charset=us-ascii\r
  44 X-BeenThere: notmuch@notmuchmail.org\r
  45 X-Mailman-Version: 2.1.13\r
  46 Precedence: list\r
  47 List-Id: "Use and development of the notmuch mail system."\r
  48         <notmuch.notmuchmail.org>\r
  49 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  50         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  51 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  52 List-Post: <mailto:notmuch@notmuchmail.org>\r
  53 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  54 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  55         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  56 X-List-Received-Date: Wed, 27 Apr 2011 19:59:19 -0000\r
  57 \r
  58 On Wed, 27 Apr 2011 18:30:09 +0200, Xavier Maillard <xavier@maillard.im> wrote:\r
  59 > On Mon, 25 Apr 2011 15:23:41 -0700, Carl Worth <cworth@cworth.org> wrote:\r
  60 > > On Wed, 17 Nov 2010 23:20:26 +0100, Matthieu Lemerre <racin@free.fr> wrote:\r
  61 > > > Maybe it would also be interesting to add a warning/assertion to check\r
  62 > > > that all email adresses added to the database are correct email\r
  63 > > > addresses? I.e. check that the `addr' variable in _index_address_mailbox\r
  64 > > > always has a @. This check is in fact already done using the function\r
  65 > > > strchr, but a bad value is explicitly ignored...\r
  66 > > \r
  67 > > Hmm... we could add a warning like that. But I don't know how useful it\r
  68 > > would actually be. (I, for one, am currently not seeing any warnings\r
  69 > > From "notmuch new"---maybe I should fix that...).\r
  70 > > \r
  71 > > Since GMime is fixed upstream (as of version 2.4.18) another option\r
  72 > > would be to simply make the notmuch build system require a sufficiently\r
  73 > > new version of GMime in order to build.\r
  74 > > \r
  75 > > What do you think?\r
  76 > \r
  77 > My vote goes to the latter. Cleaner and really simple solution.\r
  78 > \r
  79 > /Xavier\r
  80 > _______________________________________________\r
  81 > notmuch mailing list\r
  82 > notmuch@notmuchmail.org\r
  83 > http://notmuchmail.org/mailman/listinfo/notmuch\r
  84 \r
  85 I'd say both.\r
  86 \r
  87 Unfortunately, regressions are not uncommon, and regardless, it'd be\r
  88 nice to be notified when what we stuff in the db is not sane.\r
  89 \r
  90 It would however be a good idea IMHO to check email address more\r
  91 thorougly [1] than simply verifying whether an "@" is present.\r
  92 \r
  93 \r
  94 Peace\r
  95 \r
  96 -Pieter\r
  97 \r
  98 \r
  99 [1] http://www.regular-expressions.info/email.html\r
 100 \r