61/21a117fb684730e55277dead485d61d8fe2ac1

   1 Return-Path: <david@tethera.net>\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 B442A431FBD\r
   6         for <notmuch@notmuchmail.org>; Fri, 27 Jun 2014 08:46:17 -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\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
  12         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 V9vDRIqjFUN7 for <notmuch@notmuchmail.org>;\r
  16         Fri, 27 Jun 2014 08:46:10 -0700 (PDT)\r
  17 Received: from yantan.tethera.net (yantan.tethera.net [199.188.72.155])\r
  18         (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 81BFA431FAE\r
  21         for <notmuch@notmuchmail.org>; Fri, 27 Jun 2014 08:46:10 -0700 (PDT)\r
  22 Received: from remotemail by yantan.tethera.net with local (Exim 4.80)\r
  23         (envelope-from <david@tethera.net>)\r
  24         id 1X0YLg-0001Eg-Ks; Fri, 27 Jun 2014 12:46:00 -0300\r
  25 Received: (nullmailer pid 28669 invoked by uid 1000); Fri, 27 Jun 2014\r
  26         15:45:56 -0000\r
  27 From: David Bremner <david@tethera.net>\r
  28 To: Jakub Wilk <jwilk@debian.org>, 749890@bugs.debian.org\r
  29 Subject: Re: Bug#749890: python3-notmuch: missing header in mbox message ->\r
  30         NullPointerError\r
  31 In-Reply-To: <20140626213100.GA8930@jwilk.net>\r
  32 References: <8738ewudra.fsf@zancas.localnet>\r
  33         <20140623201918.GA7346@jwilk.net> <87ha37fjm3.fsf@zancas.localnet>\r
  34         <20140626213100.GA8930@jwilk.net>\r
  35 User-Agent: Notmuch/0.18.1 (http://notmuchmail.org) Emacs/24.3.1\r
  36         (x86_64-pc-linux-gnu)\r
  37 Date: Fri, 27 Jun 2014 12:45:56 -0300\r
  38 Message-ID: <878uoifj9n.fsf@zancas.localnet>\r
  39 MIME-Version: 1.0\r
  40 Content-Type: text/plain\r
  41 Cc: notmuch@notmuchmail.org\r
  42 X-BeenThere: notmuch@notmuchmail.org\r
  43 X-Mailman-Version: 2.1.13\r
  44 Precedence: list\r
  45 List-Id: "Use and development of the notmuch mail system."\r
  46         <notmuch.notmuchmail.org>\r
  47 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  48         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  49 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  50 List-Post: <mailto:notmuch@notmuchmail.org>\r
  51 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  52 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  53         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  54 X-List-Received-Date: Fri, 27 Jun 2014 15:46:17 -0000\r
  55 \r
  56 Jakub Wilk <jwilk@debian.org> writes:\r
  57 \r
  58 > * David Bremner <david@tethera.net>, 2014-06-26, 18:26:\r
  59 >>>0.18.1~rc0-1 is much better, thanks!\r
  60 >>>\r
  61 >>>I still get NullPointerError for one of my messages, though. :-( The \r
  62 >>>message is in the MBOXCL format (where message body size is indicated \r
  63 >>>by the Content-Length field), and has lines starting with "From " in \r
  64 >>>the message body. I've attached a new test case.\r
  65 >>\r
  66 >>That message (and at a guess other MBOXCL files) is ignored as a \r
  67 >>non-mail file by 0.18.1 "notmuch new".\r
  68 >\r
  69 > Indeed.\r
  70 >\r
  71 >>Is this another case of files which where indexed with an older version \r
  72 >>of notmuch causing problems with a newer version?\r
  73 >\r
  74 > Yes, that's why I meant. Sorry for not being clear.\r
  75 \r
  76 As a point of information, I bisected with the following test script:\r
  77 \r
  78 #!/usr/bin/env bash\r
  79 test_description='"notmuch new" in several variations'\r
  80 . ./test-lib.sh\r
  81 \r
  82 test_begin_subtest "Support single-message mbox with content length (deprecated)"\r
  83 cat > "${MAIL_DIR}"/mbox_file2 <<EOF\r
  84 >From jwilk  Fri May 30 14:09:05 2014\r
  85 Subject: Hello world!\r
  86 Content-Length: 12\r
  87 Lines: 1\r
  88 \r
  89 >From world!\r
  90 \r
  91 EOF\r
  92 output=$(NOTMUCH_NEW 2>&1)\r
  93 test_expect_equal "$output" \\r
  94 "Added 1 new message to the database."\r
  95 \r
  96 \r
  97 test_done\r
  98 \r
  99 The commit where the behaviour changed to reject MBOXCL files with \r
 100 'From ' in the body was 610f0e09929. This was between 0.14 and 0.15.\r
 101 I'd say this was unintentional, although it isn't clear to me yet how\r
 102 easy it is fix.\r
 103 \r
 104 \r