5a/8ecf61d8575e2c86d1d9e7dae3f7caf82c9935

   1 Return-Path: <jani@nikula.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 A47F6431FD4\r
   6         for <notmuch@notmuchmail.org>; Mon, 11 Nov 2013 08:08:11 -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.699\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.699 tagged_above=-999 required=5\r
  12         tests=[HTML_MESSAGE=0.001, 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 PQC+VUCHHRJK for <notmuch@notmuchmail.org>;\r
  16         Mon, 11 Nov 2013 08:08:02 -0800 (PST)\r
  17 Received: from mail-ve0-f175.google.com (mail-ve0-f175.google.com\r
  18         [209.85.128.175]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 75945431FD2\r
  21         for <notmuch@notmuchmail.org>; Mon, 11 Nov 2013 08:08:02 -0800 (PST)\r
  22 Received: by mail-ve0-f175.google.com with SMTP id jw12so2778149veb.34\r
  23         for <notmuch@notmuchmail.org>; Mon, 11 Nov 2013 08:08:01 -0800 (PST)\r
  24 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
  25         d=1e100.net; s=20130820;\r
  26         h=x-gm-message-state:mime-version:in-reply-to:references:date\r
  27         :message-id:subject:from:to:cc:content-type;\r
  28         bh=+X9VNmBbbKr4ewAD6phXl5773i1d3SZdR4IoNVgSMXY=;\r
  29         b=mbcnn7OABkCyXq3raJpSmBoJXMddR9rbsSRdn4vxXaGErTMEPgc4nYkOoq9GVfqcpT\r
  30         ZAtWboL5Iq9r8GDrvKKjzAQXjxMOivk2OjcpK1mAnPi93a7AgvxN2xBYKe4Y8LWAWOka\r
  31         6dGHuOu58jb1ikrOJUEEpcrlKDUSoxd2s7m+jiM44HCGReD7u06ouky/qC4wRYi70bMg\r
  32         YsmnQFmG6HYFP+z2cUrrBc+d+rOGBhsBmK3EM4o7VXaiOdWI7QODmxrPgBU0BW2iHZkh\r
  33         ePIFQUzeyX5gjnSbkxaVt6Tau7FW/Vy9fK4RDQGupCaDfrQy1Q3dp1kCaakOfBS3d8G0\r
  34         W9fg==\r
  35 X-Gm-Message-State:\r
  36  ALoCoQnAlyVi0b6C+HAFOVY6ELH4AWooS7tr3tetFKZxZQQcGx4V/BXQGzW+8+oNbv9oLcVgu2Ul\r
  37 MIME-Version: 1.0\r
  38 X-Received: by 10.58.67.168 with SMTP id o8mr4627136vet.22.1384185674376; Mon,\r
  39         11 Nov 2013 08:01:14 -0800 (PST)\r
  40 Received: by 10.58.96.20 with HTTP; Mon, 11 Nov 2013 08:01:14 -0800 (PST)\r
  41 Received: by 10.58.96.20 with HTTP; Mon, 11 Nov 2013 08:01:14 -0800 (PST)\r
  42 In-Reply-To: <m2txfjyoft.fsf@guru.guru-group.fi>\r
  43 References: <1384100482-15453-1-git-send-email-david@tethera.net>\r
  44         <52805350.70004@fifthhorseman.net> <87iovzywua.fsf@zancas.localnet>\r
  45         <m2txfjyoft.fsf@guru.guru-group.fi>\r
  46 Date: Mon, 11 Nov 2013 18:01:14 +0200\r
  47 Message-ID:\r
  48  <CAB+hUn8muZLYu5YPYtAgq-i37cU+K-2-T4DGqsTjkKBSULELwQ@mail.gmail.com>\r
  49 Subject: Re: fix for failing tests with gmime 2.6.19\r
  50 From: Jani Nikula <jani@nikula.org>\r
  51 To: Tomi Ollila <tomi.ollila@iki.fi>\r
  52 Content-Type: multipart/alternative; boundary=047d7b33d0f46cc3ad04eae8d729\r
  53 Cc: Notmuch Mail <notmuch@notmuchmail.org>,\r
  54         Daniel Kahn Gillmor <dkg@fifthhorseman.net>\r
  55 X-BeenThere: notmuch@notmuchmail.org\r
  56 X-Mailman-Version: 2.1.13\r
  57 Precedence: list\r
  58 List-Id: "Use and development of the notmuch mail system."\r
  59         <notmuch.notmuchmail.org>\r
  60 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  61         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  62 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  63 List-Post: <mailto:notmuch@notmuchmail.org>\r
  64 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  65 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  66         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  67 X-List-Received-Date: Mon, 11 Nov 2013 16:08:11 -0000\r
  68 \r
  69 --047d7b33d0f46cc3ad04eae8d729\r
  70 Content-Type: text/plain; charset=UTF-8\r
  71 \r
  72 Gmime doesn't do anything standards incompliant, so I'd prefer patching the\r
  73 test suite. Which is also why I'd like to work around this in notmuch.\r
  74 On Nov 11, 2013 4:59 PM, "Tomi Ollila" <tomi.ollila@iki.fi> wrote:\r
  75 \r
  76 > On Mon, Nov 11 2013, David Bremner <david@tethera.net> wrote:\r
  77 >\r
  78 > > Daniel Kahn Gillmor <dkg@fifthhorseman.net> writes:\r
  79 > >\r
  80 > >>\r
  81 > >> Please don't introduce this cruft into the notmuch codebase.  It should\r
  82 > >> be fixed in gmime, not worked-around notmuch.\r
  83 > >>\r
  84 > >> I've just uploaded gmime 2.6.19-2 to unstable to address this issue.\r
  85 > >>\r
  86 > >\r
  87 > > Hi Daniel;\r
  88 > >\r
  89 > > Thanks a lot for that.\r
  90 > >\r
  91 > > What I (still) wonder about is all the people not running Debian, in the\r
  92 > > interval between the release of notmuch 0.17 and the next upstream\r
  93 > > release of gmime (and propagation to various distros).  Even on Debian,\r
  94 > > building on testing and backports complicates things a bit.\r
  95 >\r
  96 > Something like this could also be used...\r
  97 >\r
  98 > diff --git a/notmuch-reply.c b/notmuch-reply.c\r
  99 > index 9d6f843..2ab0f6e 100644\r
 100 > --- a/notmuch-reply.c\r
 101 > +++ b/notmuch-reply.c\r
 102 > @@ -26,6 +26,7 @@\r
 103 >  static void\r
 104 >  show_reply_headers (GMimeMessage *message)\r
 105 >  {\r
 106 > +#if ! GMIME_CHECK_VERSION(2,6,19)\r
 107 >      GMimeStream *stream_stdout = NULL;\r
 108 >\r
 109 >      stream_stdout = g_mime_stream_file_new (stdout);\r
 110 > @@ -35,6 +36,17 @@ show_reply_headers (GMimeMessage *message)\r
 111 >         g_mime_object_write_to_stream (GMIME_OBJECT(message),\r
 112 > stream_stdout);\r
 113 >         g_object_unref(stream_stdout);\r
 114 >      }\r
 115 > +#else\r
 116 > +    char * msg = g_mime_object_to_string (GMIME_OBJECT(message));\r
 117 > +    char * rp = strstr (msg, "References:  ");\r
 118 > +    if (rp) {\r
 119 > +       fwrite (msg, 1, rp - msg + 12, stdout); // Up to 'References: '\r
 120 > +       fputs (rp + 13, stdout);\r
 121 > +    }\r
 122 > +    else {\r
 123 > +       fputs (msg, stdout);\r
 124 > +    }\r
 125 > +#endif\r
 126 >  }\r
 127 >\r
 128 >  static void\r
 129 >\r
 130 >\r
 131 > >\r
 132 > > d\r
 133 >\r
 134 > Tomi\r
 135 > _______________________________________________\r
 136 > notmuch mailing list\r
 137 > notmuch@notmuchmail.org\r
 138 > http://notmuchmail.org/mailman/listinfo/notmuch\r
 139 >\r
 140 \r
 141 --047d7b33d0f46cc3ad04eae8d729\r
 142 Content-Type: text/html; charset=UTF-8\r
 143 Content-Transfer-Encoding: quoted-printable\r
 144 \r
 145 <p dir=3D"ltr">Gmime doesn&#39;t do anything standards incompliant, so I&#3=\r
 146 9;d prefer patching the test suite. Which is also why I&#39;d like to work =\r
 147 around this in notmuch.</p>\r
 148 <div class=3D"gmail_quote">On Nov 11, 2013 4:59 PM, &quot;Tomi Ollila&quot;=\r
 149  &lt;<a href=3D"mailto:tomi.ollila@iki.fi">tomi.ollila@iki.fi</a>&gt; wrote=\r
 150 :<br type=3D"attribution"><blockquote class=3D"gmail_quote" style=3D"margin=\r
 151 :0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">\r
 152 On Mon, Nov 11 2013, David Bremner &lt;<a href=3D"mailto:david@tethera.net"=\r
 153 >david@tethera.net</a>&gt; wrote:<br>\r
 154 <br>\r
 155 &gt; Daniel Kahn Gillmor &lt;<a href=3D"mailto:dkg@fifthhorseman.net">dkg@f=\r
 156 ifthhorseman.net</a>&gt; writes:<br>\r
 157 &gt;<br>\r
 158 &gt;&gt;<br>\r
 159 &gt;&gt; Please don&#39;t introduce this cruft into the notmuch codebase. =\r
 160 =C2=A0It should<br>\r
 161 &gt;&gt; be fixed in gmime, not worked-around notmuch.<br>\r
 162 &gt;&gt;<br>\r
 163 &gt;&gt; I&#39;ve just uploaded gmime 2.6.19-2 to unstable to address this =\r
 164 issue.<br>\r
 165 &gt;&gt;<br>\r
 166 &gt;<br>\r
 167 &gt; Hi Daniel;<br>\r
 168 &gt;<br>\r
 169 &gt; Thanks a lot for that.<br>\r
 170 &gt;<br>\r
 171 &gt; What I (still) wonder about is all the people not running Debian, in t=\r
 172 he<br>\r
 173 &gt; interval between the release of notmuch 0.17 and the next upstream<br>\r
 174 &gt; release of gmime (and propagation to various distros). =C2=A0Even on D=\r
 175 ebian,<br>\r
 176 &gt; building on testing and backports complicates things a bit.<br>\r
 177 <br>\r
 178 Something like this could also be used...<br>\r
 179 <br>\r
 180 diff --git a/notmuch-reply.c b/notmuch-reply.c<br>\r
 181 index 9d6f843..2ab0f6e 100644<br>\r
 182 --- a/notmuch-reply.c<br>\r
 183 +++ b/notmuch-reply.c<br>\r
 184 @@ -26,6 +26,7 @@<br>\r
 185 =C2=A0static void<br>\r
 186 =C2=A0show_reply_headers (GMimeMessage *message)<br>\r
 187 =C2=A0{<br>\r
 188 +#if ! GMIME_CHECK_VERSION(2,6,19)<br>\r
 189 =C2=A0 =C2=A0 =C2=A0GMimeStream *stream_stdout =3D NULL;<br>\r
 190 <br>\r
 191 =C2=A0 =C2=A0 =C2=A0stream_stdout =3D g_mime_stream_file_new (stdout);<br>\r
 192 @@ -35,6 +36,17 @@ show_reply_headers (GMimeMessage *message)<br>\r
 193 =C2=A0 =C2=A0 =C2=A0 =C2=A0 g_mime_object_write_to_stream (GMIME_OBJECT(mes=\r
 194 sage), stream_stdout);<br>\r
 195 =C2=A0 =C2=A0 =C2=A0 =C2=A0 g_object_unref(stream_stdout);<br>\r
 196 =C2=A0 =C2=A0 =C2=A0}<br>\r
 197 +#else<br>\r
 198 + =C2=A0 =C2=A0char * msg =3D g_mime_object_to_string (GMIME_OBJECT(message=\r
 199 ));<br>\r
 200 + =C2=A0 =C2=A0char * rp =3D strstr (msg, &quot;References: =C2=A0&quot;);<=\r
 201 br>\r
 202 + =C2=A0 =C2=A0if (rp) {<br>\r
 203 + =C2=A0 =C2=A0 =C2=A0 fwrite (msg, 1, rp - msg + 12, stdout); // Up to &#3=\r
 204 9;References: &#39;<br>\r
 205 + =C2=A0 =C2=A0 =C2=A0 fputs (rp + 13, stdout);<br>\r
 206 + =C2=A0 =C2=A0}<br>\r
 207 + =C2=A0 =C2=A0else {<br>\r
 208 + =C2=A0 =C2=A0 =C2=A0 fputs (msg, stdout);<br>\r
 209 + =C2=A0 =C2=A0}<br>\r
 210 +#endif<br>\r
 211 =C2=A0}<br>\r
 212 <br>\r
 213 =C2=A0static void<br>\r
 214 <br>\r
 215 <br>\r
 216 &gt;<br>\r
 217 &gt; d<br>\r
 218 <br>\r
 219 Tomi<br>\r
 220 _______________________________________________<br>\r
 221 notmuch mailing list<br>\r
 222 <a href=3D"mailto:notmuch@notmuchmail.org">notmuch@notmuchmail.org</a><br>\r
 223 <a href=3D"http://notmuchmail.org/mailman/listinfo/notmuch" target=3D"_blan=\r
 224 k">http://notmuchmail.org/mailman/listinfo/notmuch</a><br>\r
 225 </blockquote></div>\r
 226 \r
 227 --047d7b33d0f46cc3ad04eae8d729--\r