73/f405f18b6bceb845915d5164f077e347bac8f2

   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 C728B431FD7\r
   6         for <notmuch@notmuchmail.org>; Thu,  1 Mar 2012 09:14:41 -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.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 TUOD76Gllg8Q for <notmuch@notmuchmail.org>;\r
  16         Thu,  1 Mar 2012 09:14:41 -0800 (PST)\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  B92D7431FAE    for <notmuch@notmuchmail.org>; Thu,  1 Mar 2012 09:14:40 -0800\r
  21  (PST)\r
  22 Received: by wgbdt12 with SMTP id dt12so28549wgb.2\r
  23         for <notmuch@notmuchmail.org>; Thu, 01 Mar 2012 09:14:38 -0800 (PST)\r
  24 Received-SPF: pass (google.com: domain of pieter@praet.org designates\r
  25         10.180.99.100 as permitted sender) client-ip=10.180.99.100; \r
  26 Authentication-Results: mr.google.com;\r
  27         spf=pass (google.com: domain of pieter@praet.org\r
  28         designates 10.180.99.100 as permitted sender)\r
  29         smtp.mail=pieter@praet.org\r
  30 Received: from mr.google.com ([10.180.99.100])\r
  31         by 10.180.99.100 with SMTP id ep4mr12863236wib.7.1330622078265\r
  32         (num_hops = 1); Thu, 01 Mar 2012 09:14:38 -0800 (PST)\r
  33 Received: by 10.180.99.100 with SMTP id ep4mr10308714wib.7.1330622077970;\r
  34         Thu, 01 Mar 2012 09:14:37 -0800 (PST)\r
  35 Received: from localhost ([109.131.111.80])\r
  36         by mx.google.com with ESMTPS id df3sm43051920wib.1.2012.03.01.09.14.36\r
  37         (version=TLSv1/SSLv3 cipher=OTHER);\r
  38         Thu, 01 Mar 2012 09:14:36 -0800 (PST)\r
  39 From: Pieter Praet <pieter@praet.org>\r
  40 To: David Bremner <david@tethera.net>\r
  41 Subject: Re: [PATCH v6 1/3] emacs: `notmuch-show-get-message-id': optionally\r
  42         return Message-Id sans prefix\r
  43 In-Reply-To: <87399tbzmx.fsf@zancas.localnet>\r
  44 References: <1327397873-20596-1-git-send-email-dme@dme.org>\r
  45         <1329684627-10708-1-git-send-email-pieter@praet.org>\r
  46         <1329684627-10708-2-git-send-email-pieter@praet.org>\r
  47         <87ehtjj7s6.fsf@zancas.localnet> <87hayaol8b.fsf@praet.org>\r
  48         <87399tbzmx.fsf@zancas.localnet>\r
  49 User-Agent: Notmuch/0.11.1+210~g6afc43e (http://notmuchmail.org) Emacs/23.3.1\r
  50         (x86_64-unknown-linux-gnu)\r
  51 Date: Thu, 01 Mar 2012 18:12:12 +0100\r
  52 Message-ID: <87obsgmej7.fsf@praet.org>\r
  53 MIME-Version: 1.0\r
  54 Content-Type: text/plain; charset=us-ascii\r
  55 X-Gm-Message-State:\r
  56  ALoCoQkVxhK1105wvA+mSDJlXCWq4DWInfv5zYXeiIKVlWOPLNGwxJK8i8a6172qn/63jrbK9biM\r
  57 Cc: Notmuch Mail <notmuch@notmuchmail.org>\r
  58 X-BeenThere: notmuch@notmuchmail.org\r
  59 X-Mailman-Version: 2.1.13\r
  60 Precedence: list\r
  61 List-Id: "Use and development of the notmuch mail system."\r
  62         <notmuch.notmuchmail.org>\r
  63 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  64         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  65 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  66 List-Post: <mailto:notmuch@notmuchmail.org>\r
  67 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  68 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  69         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  70 X-List-Received-Date: Thu, 01 Mar 2012 17:14:41 -0000\r
  71 \r
  72 On Wed, 29 Feb 2012 08:19:34 -0400, David Bremner <david@tethera.net> wrote:\r
  73 > On Tue, 28 Feb 2012 19:40:04 +0100, Pieter Praet <pieter@praet.org> wrote:\r
  74 > > \r
  75 > > Though I couldn't help but notice (it's a curse...) that patch #3\r
  76 > > seems to have gained an extra space in its commit message subject:\r
  77 > > \r
  78 > >   Original (id:"1329684627-10708-4-git-send-email-pieter@praet.org"):\r
  79 > >     emacs: add `notmuch-show-stash-mlarchive-link{,-and-go}'\r
  80 > >                                                   ^\r
  81 > \r
  82 > I don't have any applypatch-msg hook. I am using \r
  83 >   "notmuch show --format=raw $messageid | git am -" to\r
  84 > apply the patch.  \r
  85 > \r
  86 > I looked at the file on disk, and something is wrapping the subject\r
  87 > there.  Perhaps somebody's unwrapping code is buggy? I'm not sure what's\r
  88 > at fault here; [...]\r
  89 \r
  90 TL;DR:  Doesn't seem like there's much we can do about it...\r
  91 \r
  92 \r
  93 According to RFC 2822 section 2.2.3 [1], header fields should be folded\r
  94 by inserting a CRLF right before the last whitespace char before the\r
  95 line exceeds 78 characters, and when a field contains comma-separated\r
  96 values (which is somewhat applicable in this case), it should be folded\r
  97 by inserting a CRLF right after a comma.\r
  98 \r
  99 Now, the issue is that unfolding is accomplished by simply removing any\r
 100 CRLF that is immediately followed by a whitespace char:\r
 101 \r
 102 If the subject headers of those patches were folded by simply inserting\r
 103 a CRLF, they couldn't possibly be unfolded again as there wouldn't be a\r
 104 whitespace char right after the CRLF, so Mailman seems to have helpfully\r
 105 inserted a tab...\r
 106 \r
 107 \r
 108 Here's one of those patches at a number of archives:\r
 109 \r
 110 - http://notmuchmail.org/pipermail/notmuch/2012/009575.html\r
 111 - http://mid.gmane.org/1329684627-10708-4-git-send-email-pieter@praet.org\r
 112 - http://mail-archive.com/search?l=mid&q=1329684627-10708-4-git-send-email-pieter@praet.org\r
 113 \r
 114 As you can see (in the source of those pages), both Pipermail and Gmane\r
 115 unfold the subject header by simply removing the line break, and at The\r
 116 Mail Archive, the tab is also replaced by a space.  I assume the latter\r
 117 also happens in gmime ?\r
 118 \r
 119 \r
 120 So, I guess I'll just have to learn to live with it... :)\r
 121 \r
 122 \r
 123 > [...] maybe you could experiment with applying the the patches\r
 124 > from the list and let me know if e.g. going from files is better than\r
 125 > using "notmuch show --format=raw".\r
 126 >\r
 127 \r
 128 Based on my (admittedly very superficial) analysis of the problem,\r
 129 that won't do us any good wrt this issue, unfortunately...\r
 130 \r
 131 But it *would* buy us a (slight) performance increase, so if/when my\r
 132 patch to `notmuch-show-mapc' [2] goes in, we might want to consider\r
 133 making `notmuch-show-pipe-message' use `notmuch-show-get-filename'\r
 134 instead of `notmuch-show-get-message-{id,ids-for-open-messages}'.\r
 135 \r
 136 \r
 137 > d\r
 138 > \r
 139 > \r
 140 \r
 141 \r
 142 Peace\r
 143 \r
 144 -- \r
 145 Pieter\r
 146 \r
 147 [1] http://tools.ietf.org/html/rfc2822.html#section-2.2.3\r
 148 [2] id:"1330122640-18895-5-git-send-email-pieter@praet.org"\r