76/1cef28b3f45cde1c33f3d40b39b3a3f5b589db

   1 Return-Path: <jrollins@finestructure.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 6BF6A429E27\r
   6         for <notmuch@notmuchmail.org>; Fri, 18 Nov 2011 17:59:01 -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: -2.29\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.29 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3, T_MIME_NO_TEXT=0.01] 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 Ujr--0lI+vTX for <notmuch@notmuchmail.org>;\r
  16         Fri, 18 Nov 2011 17:59:00 -0800 (PST)\r
  17 Received: from outgoing-mail.its.caltech.edu (outgoing-mail.its.caltech.edu\r
  18         [131.215.239.19])\r
  19         by olra.theworths.org (Postfix) with ESMTP id C4E12431FD0\r
  20         for <notmuch@notmuchmail.org>; Fri, 18 Nov 2011 17:59:00 -0800 (PST)\r
  21 Received: from earth-doxen.imss.caltech.edu (localhost [127.0.0.1])\r
  22         by earth-doxen-postvirus (Postfix) with ESMTP id 4007666E01C2;\r
  23         Fri, 18 Nov 2011 17:58:58 -0800 (PST)\r
  24 X-Spam-Scanned: at Caltech-IMSS on earth-doxen by amavisd-new\r
  25 Received: from finestructure.net (DHCP-123-180.caltech.edu [131.215.123.180])\r
  26         (Authenticated sender: jrollins)\r
  27         by earth-doxen-submit (Postfix) with ESMTP id 62F5666E0134;\r
  28         Fri, 18 Nov 2011 17:58:55 -0800 (PST)\r
  29 Received: by finestructure.net (Postfix, from userid 1000)\r
  30         id 43922ACC; Fri, 18 Nov 2011 17:58:55 -0800 (PST)\r
  31 From: Jameson Graef Rollins <jrollins@finestructure.net>\r
  32 To: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>, notmuch@notmuchmail.org\r
  33 Subject: Re: [PATCH] Output unmodified Content-Type header value for JSON\r
  34         format.\r
  35 In-Reply-To: <1321659905-24367-1-git-send-email-dmitry.kurochkin@gmail.com>\r
  36 References: <1321659905-24367-1-git-send-email-dmitry.kurochkin@gmail.com>\r
  37 User-Agent: Notmuch/0.9+81~gd8cf814 (http://notmuchmail.org) Emacs/23.3.1\r
  38         (x86_64-pc-linux-gnu)\r
  39 Date: Fri, 18 Nov 2011 17:58:52 -0800\r
  40 Message-ID: <87fwhkyisj.fsf@servo.finestructure.net>\r
  41 MIME-Version: 1.0\r
  42 Content-Type: multipart/signed; boundary="=-=-=";\r
  43         micalg=pgp-sha256; protocol="application/pgp-signature"\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: Sat, 19 Nov 2011 01:59:01 -0000\r
  57 \r
  58 --=-=-=\r
  59 \r
  60 On Sat, 19 Nov 2011 03:45:05 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:\r
  61 > Before the change, notmuch used g_mime_content_type_to_string(3)\r
  62 > function to output Content-Type header value.  Turns out it outputs\r
  63 > only "type/subtype" part and ignores all parameters.  Also, if there\r
  64 > is no Content-Type header, default "text/plain" value is used.\r
  65 \r
  66 Hi, Dmitry.  Can you explain under what circumstances you would need the\r
  67 extra content-type parameters?  It just seems like a lot of extra noise\r
  68 in the output to me, but that's partially because I can't think of any\r
  69 reason why something that is receiving pre-parsed mime content would\r
  70 need it.  Maybe there's a better way to handle what you're trying to get\r
  71 to.\r
  72 \r
  73 I think it would help a lot if you could submit some sort of test\r
  74 modification that demonstrates the issue.  This is one of the reasons we\r
  75 keep emphasizing that it's good to first have tests in hand that\r
  76 demonstrate issues before patches that address them.\r
  77 \r
  78 >   "content": [{"id": 2,\r
  79 > - "content-type": "text/plain",\r
  80 >   "content": "This is a test signed message.\n"},\r
  81 \r
  82 Without figuring out what's going on, I notice that some of the tests\r
  83 have been modified to remove the content-type fields on a bunch of\r
  84 parts.  I think that is probably not right.\r
  85 \r
  86 jamie.\r
  87 \r
  88 --=-=-=\r
  89 Content-Type: application/pgp-signature\r
  90 \r
  91 -----BEGIN PGP SIGNATURE-----\r
  92 Version: GnuPG v1.4.11 (GNU/Linux)\r
  93 \r
  94 iQIcBAEBCAAGBQJOxw1cAAoJEO00zqvie6q8hDkP/Aum2noyxQn0lJPXu4tXfDPv\r
  95 FwQhuCLJOyuuSR3wtcQlZZPZ1uorY6f06J44QpjB9yAUEv5IUB9KfzbkLgpKLQJd\r
  96 9Cs54Bh3aTCoQLpKXyWSb2y9JoQEI4/tK1Jm98RzIA4RZ4Bg6sop+MQsTLpM5ULn\r
  97 3xKyCuIkKeVoO9B9unIZoWR4aBbmj8nig333MgxzT3aaXXesF0iON+WVi52rfXC5\r
  98 0ebBJoyY1mOJxJoUUmLJBa+2AWnGLyf7XYbl0y4x/RUPbUSGJ+O8LU3O9uLKyxSi\r
  99 NlBtSr/Rj/xUyOFQwnz37P7CxotqSNWpbYqEHl9G+9R2LgsJdDITOOw3Car8lWhm\r
 100 //f1OxW0gyPVrDDrZhplWWOID7SGQ/SWMsSXkPayYxox8ExyxJu0mg/V6x3b4Nqf\r
 101 SSXThr0sSpYrYYlmpZO5vI1GEIC6MrvC5pisP8dOHHb3pNEL6YVQYmYP+CwUaZux\r
 102 ZCyhChiWXbpkBm6tGpAoYdNbDr7Z/uwkdGKvUo0Es7s4qetCes9LhfZLslGACIRy\r
 103 5z7xHaQ+7C9t4XixuW1dAKBZGjImuaJFnRQRBNxZknD/oF65k7nANkpyG8EEVRZw\r
 104 +Ffmrb+Ltio3t29ecBSj7QvJIWqTfIX9vK40VVrJPQerEB7STlKIlhYXkq54HEoE\r
 105 yNIIAuJvEGpATBr66aFN\r
 106 =tZ5D\r
 107 -----END PGP SIGNATURE-----\r
 108 --=-=-=--\r