5d/d6b074c509d16db543317ddf963f8765a37545

   1 Return-Path: <cworth@cworth.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 439E2429E5B\r
   6         for <notmuch@notmuchmail.org>; Tue, 28 Jun 2011 20:14:16 -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 IfhR43hfHWWU for <notmuch@notmuchmail.org>;\r
  16         Tue, 28 Jun 2011 20:14:15 -0700 (PDT)\r
  17 Received: from arlo.cworth.org (arlo.cworth.org [50.43.72.2])\r
  18         by olra.theworths.org (Postfix) with ESMTP id C253A429E55\r
  19         for <notmuch@notmuchmail.org>; Tue, 28 Jun 2011 20:14:15 -0700 (PDT)\r
  20 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  21         by arlo.cworth.org (Postfix) with ESMTP id 87B5D29A51E;\r
  22         Tue, 28 Jun 2011 20:14:14 -0700 (PDT)\r
  23 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  24         id 75BEB2541A8; Tue, 28 Jun 2011 20:14:14 -0700 (PDT)\r
  25 From: Carl Worth <cworth@cworth.org>\r
  26 To: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>, notmuch@notmuchmail.org\r
  27 Subject: Re: [PATCH 1/2] Add part filename and content-id in notmuch show\r
  28         output if available.\r
  29 In-Reply-To: <1306620228-24351-1-git-send-email-dmitry.kurochkin@gmail.com>\r
  30 References: <1306620228-24351-1-git-send-email-dmitry.kurochkin@gmail.com>\r
  31 User-Agent: Notmuch/0.5 (http://notmuchmail.org) Emacs/23.3.1\r
  32         (i486-pc-linux-gnu)\r
  33 Date: Tue, 28 Jun 2011 20:14:08 -0700\r
  34 Message-ID: <87iprp72in.fsf@yoom.home.cworth.org>\r
  35 MIME-Version: 1.0\r
  36 Content-Type: multipart/signed; boundary="=-=-=";\r
  37         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  38 X-BeenThere: notmuch@notmuchmail.org\r
  39 X-Mailman-Version: 2.1.13\r
  40 Precedence: list\r
  41 List-Id: "Use and development of the notmuch mail system."\r
  42         <notmuch.notmuchmail.org>\r
  43 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  44         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  45 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  46 List-Post: <mailto:notmuch@notmuchmail.org>\r
  47 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  48 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  49         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  50 X-List-Received-Date: Wed, 29 Jun 2011 03:14:16 -0000\r
  51 \r
  52 --=-=-=\r
  53 Content-Type: text/plain; charset=utf-8\r
  54 Content-Transfer-Encoding: quoted-printable\r
  55 \r
  56 On Sun, 29 May 2011 02:03:47 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmai=\r
  57 l.com> wrote:\r
  58 > Also, the patch adds Content-id to text output format of notmuch\r
  59 > show.\r
  60 \r
  61 I don't think I've raised this point before, but I've been tempted to\r
  62 setup a git hook that rejects any commits with a paragraph beginning\r
  63 with the word "also" or the word "additionally". These words are clues\r
  64 to me that the commit is doing multiple independent things and should\r
  65 really be split up.\r
  66 \r
  67 > The patch makes notmuch show always output filename if available,\r
  68 > independent of Content-Disposition.  Both JSON and text output\r
  69 > formats are changed.\r
  70 \r
  71 The commit message claims to change both JSON and text output, (and the\r
  72 patch bears that out). But the test suite only gets changes to text\r
  73 output.\r
  74 \r
  75 This suggests to me that the test suite needs an additional test here\r
  76 (for a "Content-Disposition: inline" attachment with a filename).\r
  77 \r
  78 What can I say, I must be feeling soft today, because in spite of both\r
  79 of those issues I've just pushed these two commits, (I did augment the\r
  80 commit message of the second commit to be more than a single line=E2=80=94I=\r
  81  know\r
  82 that I've discussed that issue before).\r
  83 \r
  84 Thanks, Dmitry, for the improvements.\r
  85 \r
  86 =2DCarl\r
  87 \r
  88 =2D-=20\r
  89 carl.d.worth@intel.com\r
  90 \r
  91 --=-=-=\r
  92 Content-Type: application/pgp-signature\r
  93 \r
  94 -----BEGIN PGP SIGNATURE-----\r
  95 Version: GnuPG v1.4.11 (GNU/Linux)\r
  96 \r
  97 iEYEARECAAYFAk4KmIAACgkQ6JDdNq8qSWjm0gCfdxQ/xyL3eA1eMhHDLooglEha\r
  98 x1IAniiWNWvrNUy1R5nh9G5G6hpP0lWE\r
  99 =fW5D\r
 100 -----END PGP SIGNATURE-----\r
 101 --=-=-=--\r