f2/afc073bebd6799a898654edb5ce9bd0c36b373

   1 Return-Path: <matthias.guedemann@ovgu.de>\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 52B08431FD0\r
   6         for <notmuch@notmuchmail.org>; Mon, 23 May 2011 06:39:13 -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 meC2nu8Vf-wX for <notmuch@notmuchmail.org>;\r
  16         Mon, 23 May 2011 06:39:12 -0700 (PDT)\r
  17 Received: from mail.uni-magdeburg.de (mail.uni-magdeburg.de [141.44.1.10])\r
  18         by olra.theworths.org (Postfix) with ESMTP id 74E2F431FB6\r
  19         for <notmuch@notmuchmail.org>; Mon, 23 May 2011 06:39:12 -0700 (PDT)\r
  20 Received: from pc44es141.cs.uni-magdeburg.de ([141.44.32.169]:53051)\r
  21         by mail.uni-magdeburg.de with esmtp (EXIM Version 4.69)\r
  22         id 1QOVLd-0003ID-Ur; Mon, 23 May 2011 15:39:11 +0200\r
  23 From: Matthias Guedemann <Matthias.Guedemann@ovgu.de>\r
  24 To: Dirk Hohndel <hohndel@infradead.org>,\r
  25         "notmuch\@notmuchmail.org" <notmuch@notmuchmail.org>\r
  26 Subject: Re: problems with multipart/mixed\r
  27 In-Reply-To: <m3oc2tmzfp.fsf@x201s.gr8dns.org>\r
  28 References: <878vu0ftni.fsf@pc44es141.cs.uni-magdeburg.de>\r
  29         <m3oc2tmzfp.fsf@x201s.gr8dns.org>\r
  30 User-Agent: Notmuch/0.5-103-gc51d5b3 (http://notmuchmail.org) Emacs/23.2.1\r
  31         (x86_64-pc-linux-gnu)\r
  32 Date: Mon, 23 May 2011 15:39:07 +0200\r
  33 Message-ID: <877h9hmt8k.fsf@pc44es141.cs.uni-magdeburg.de>\r
  34 MIME-Version: 1.0\r
  35 Content-Type: text/plain; charset=us-ascii\r
  36 X-Scan-Signature: 516e4beb4d07a1619ec37fccea24750e\r
  37 X-BeenThere: notmuch@notmuchmail.org\r
  38 X-Mailman-Version: 2.1.13\r
  39 Precedence: list\r
  40 List-Id: "Use and development of the notmuch mail system."\r
  41         <notmuch.notmuchmail.org>\r
  42 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  43         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  44 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  45 List-Post: <mailto:notmuch@notmuchmail.org>\r
  46 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  47 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  48         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  49 X-List-Received-Date: Mon, 23 May 2011 13:39:13 -0000\r
  50 \r
  51 \r
  52 > If you did then I'm in the same boat. Notmuch/emacs used to display both\r
  53 \r
  54 welcome to the boat!\r
  55 \r
  56 I found the reason why that happens. Basically it appears because of a\r
  57 bug in the ongoing multipart development.\r
  58 \r
  59 In my example, the problem is as follows: currently the outer multipart\r
  60 message gets ID 1, the first html part ID 2, and the attachment ID 3.\r
  61 \r
  62 This seems to be correct in the JSON output. But for whatever reason,\r
  63 show in emacs does not display the html part at all and the attachment\r
  64 as text/html (it is text/plain). So it seems that the show is off by 1\r
  65 somewhere.\r
  66 \r
  67 Now a quick-and-dirty way around this:\r
  68 \r
  69 reset to c51d5b3cdb5ca0816816e88ca6f7136a24e74eee\r
  70 \r
  71 and apply this patch:\r
  72 \r
  73 \r
  74 ----8<---------\r
  75 diff --git a/show-message.c b/show-message.c\r
  76 index ff9146e..6913d7f 100644\r
  77 --- a/show-message.c\r
  78 +++ b/show-message.c\r
  79 @@ -30,8 +30,8 @@ show_message_part (GMimeObject *part, int *part_count,\r
  80         GMimeMultipart *multipart = GMIME_MULTIPART (part);\r
  81         int i;\r
  82  \r
  83 -       *part_count = *part_count + 1;\r
  84 -       (*show_part) (part, part_count);\r
  85 +       //*part_count = *part_count + 1;\r
  86 +       //(*show_part) (part, part_count);\r
  87  \r
  88         for (i = 0; i < g_mime_multipart_get_count (multipart); i++) {\r
  89             show_message_part (g_mime_multipart_get_part (multipart, i),\r
  90 ----8<---------\r
  91 \r
  92 current master is already some steps further, so probably this will be\r
  93 fixed in a correct way soon, but for now I'll use this.\r
  94 \r
  95 regards\r
  96 Matthias\r