dc/f31fd0e7476191906e9aced6a6a8523ced2420

   1 Return-Path: <tomi.ollila@iki.fi>\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 arlo.cworth.org (Postfix) with ESMTP id 34EC96DE1AC4\r
   6  for <notmuch@notmuchmail.org>; Sun, 14 Feb 2016 11:45:46 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at cworth.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 0.638\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0.638 tagged_above=-999 required=5 tests=[AWL=-0.014,\r
  12   SPF_NEUTRAL=0.652] autolearn=disabled\r
  13 Received: from arlo.cworth.org ([127.0.0.1])\r
  14  by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15  with ESMTP id 3BekBPRYj5mB for <notmuch@notmuchmail.org>;\r
  16  Sun, 14 Feb 2016 11:45:42 -0800 (PST)\r
  17 Received: from guru.guru-group.fi (guru.guru-group.fi [46.183.73.34])\r
  18  by arlo.cworth.org (Postfix) with ESMTP id 55E856DE1ABD\r
  19  for <notmuch@notmuchmail.org>; Sun, 14 Feb 2016 11:45:41 -0800 (PST)\r
  20 Received: from guru.guru-group.fi (localhost [IPv6:::1])\r
  21  by guru.guru-group.fi (Postfix) with ESMTP id B5AD71000CA;\r
  22  Sun, 14 Feb 2016 21:45:45 +0200 (EET)\r
  23 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  24 To: Mark Walters <markwalters1009@gmail.com>, notmuch@notmuchmail.org\r
  25 Subject: Re: [PATCH v5 0/7] emacs: Improve the cited message included in\r
  26  replies\r
  27 In-Reply-To: <1455475199-32622-1-git-send-email-markwalters1009@gmail.com>\r
  28 References: <1455475199-32622-1-git-send-email-markwalters1009@gmail.com>\r
  29 User-Agent: Notmuch/0.21+70~g0bc8909 (http://notmuchmail.org) Emacs/24.3.1\r
  30  (x86_64-unknown-linux-gnu)\r
  31 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  32  $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  33  !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  34 Date: Sun, 14 Feb 2016 21:45:45 +0200\r
  35 Message-ID: <m28u2nm5xy.fsf@guru.guru-group.fi>\r
  36 MIME-Version: 1.0\r
  37 Content-Type: text/plain\r
  38 X-BeenThere: notmuch@notmuchmail.org\r
  39 X-Mailman-Version: 2.1.20\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: <https://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: <https://notmuchmail.org/mailman/listinfo/notmuch>,\r
  49  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  50 X-List-Received-Date: Sun, 14 Feb 2016 19:45:46 -0000\r
  51 \r
  52 On Sun, Feb 14 2016, Mark Walters <markwalters1009@gmail.com> wrote:\r
  53 \r
  54 > This is a slight update and a rebase of this patch set (previous\r
  55 > version at\r
  56 > id:1446894276-7814-1-git-send-email-markwalters1009@gmail.com )\r
  57 >\r
  58 > The only change from last time (apart from fixing rebasing to master)\r
  59 > is the rewording of couple of minor comments inline with dme's review\r
  60 > of the previous version.\r
  61 >\r
  62 > Although I am submitting it is primarily dme's series, with minor\r
  63 > updates from me.\r
  64 >\r
  65 > One particular motivation for this series is that it fixes a long\r
  66 > standing bug we have that replying to a message with an rfc822 part\r
  67 > completely omits that part. It also fixes the bug whereby we don't\r
  68 > include application/octet-stream parts which are actually text/plain\r
  69 > (and thus are displayed in show mode).\r
  70 >\r
  71 > This series makes the reply code use the same code as the show code so\r
  72 > everything works as expected: the reply buffer looks essentially the\r
  73 > same as the show buffer.\r
  74 >\r
  75 > There is one slight difference: the user might want different part\r
  76 > headers displayed when replying; both because the audience is\r
  77 > different (a non-notmuch using recipient) and because the buttons\r
  78 > don't "work" (you can't click on them to show or view a part).\r
  79 >\r
  80 > Dme and I disagree on which of these we would like to see so make that\r
  81 > customisable.\r
  82 \r
  83 Does *NOT* look bad, works and (relevant) tests pass.\r
  84 \r
  85 We'd need at least David to verify that Mark got his (David's that is)\r
  86 patches properly rebased...\r
  87 \r
  88 Tomi\r
  89 \r
  90 >\r
  91 > The key change is patch 3 which switches how reply works. Also note\r
  92 > that patch 2 is almost all whitespace change as the changes modify the\r
  93 > indentation.\r
  94 >\r
  95 > Best wishes\r
  96 >\r
  97 > Mark\r
  98 >\r
  99 >\r
 100 > David Edmondson (6):\r
 101 >   emacs/show: Re-arrange determination if a part header is necessary\r
 102 >   emacs/show: Accommodate the lack of part header buttons\r
 103 >   emacs/mua: Generate improved cited text for replies\r
 104 >   emacs/show: Remove the 'no-buttons option of\r
 105 >     `notmuch-show-insert-bodypart'\r
 106 >   emacs/show: Make the insertion of part headers overridable.\r
 107 >   emacs/mua: Let user specify which parts get a header in citations.\r
 108 >\r
 109 > Mark Walters (1):\r
 110 >   test: fix the tests for the new reply code\r
 111 >\r
 112 >  emacs/notmuch-mua.el  |  60 ++++++++++-----------\r
 113 >  emacs/notmuch-show.el | 144 +++++++++++++++++++++++++++++++-------------------\r
 114 >  test/T310-emacs.sh    |  32 +++++++++++\r
 115 >  test/test-lib.el      |   4 ++\r
 116 >  4 files changed, 156 insertions(+), 84 deletions(-)\r
 117 >\r
 118 > -- \r
 119 > 2.1.4\r
 120 >\r
 121 > _______________________________________________\r
 122 > notmuch mailing list\r
 123 > notmuch@notmuchmail.org\r
 124 > https://notmuchmail.org/mailman/listinfo/notmuch\r