03/278b826fd2ae58e4b0b2bf8d7c32295b94922f

   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 16487431FD7\r
   6         for <notmuch@notmuchmail.org>; Mon, 26 Aug 2013 10:58:51 -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.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[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 22RC9-WSL7UC for <notmuch@notmuchmail.org>;\r
  16         Mon, 26 Aug 2013 10:58:42 -0700 (PDT)\r
  17 Received: from mail-ee0-f49.google.com (mail-ee0-f49.google.com\r
  18  [74.125.83.49])        (using TLSv1 with cipher RC4-SHA (128/128 bits))        (No client\r
  19  certificate requested) by olra.theworths.org (Postfix) with ESMTPS id\r
  20  8225F431FAF    for <notmuch@notmuchmail.org>; Mon, 26 Aug 2013 10:58:42 -0700\r
  21  (PDT)\r
  22 Received: by mail-ee0-f49.google.com with SMTP id d41so1745438eek.22\r
  23         for <notmuch@notmuchmail.org>; Mon, 26 Aug 2013 10:58:41 -0700 (PDT)\r
  24 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
  25         d=google.com; s=20120113;\r
  26         h=x-gm-message-state:from:to:subject:in-reply-to:references\r
  27         :user-agent:date:message-id:mime-version:content-type;\r
  28         bh=h9Lxl5EUpFNUQd5yIZxwkR5ko/VEEOMcIN7wrI1YYb0=;\r
  29         b=eMjwUlBwT8up43zT1tDTCyjBDxe4fNHxbxjqd2qrq5mbJeBjRxqulNkssuBb/w+TJG\r
  30         a4yTTFJbjgO0c67aYq3aUz3hOpXPOo4guxqWw2bP4mL9MsGLx1kPmGofmkbxu9Y2UK60\r
  31         hAbCb71NXY6ch41RgV6/u+1TJRE1Kno6XW/J0IMepa8vP1zq63cTX+gy0lQGudJ5XRaX\r
  32         TqTCOD6tp1Gbh62RrJgH7b6OD83e9p49iG2CDM7L/DHSoIz6W1OwlNkgjwsMGpFor4jD\r
  33         L/FUFuFoTG5xLxvEgWlbq8nWvBZGez/Ht/foOyk4U0zxOLpNOqgwJugQ3lKSbm1jeZCn\r
  34         jCow==\r
  35 X-Gm-Message-State:\r
  36  ALoCoQlhcp3CvU/e4HTrtAXNd1fBGexChmu87IFRSappf20sv+JpB310iAjxatb38BOjBuxzvgMT\r
  37 X-Received: by 10.14.107.68 with SMTP id n44mr28056364eeg.26.1377539921273;\r
  38         Mon, 26 Aug 2013 10:58:41 -0700 (PDT)\r
  39 Received: from localhost (dsl-hkibrasgw2-58c36f-91.dhcp.inet.fi.\r
  40         [88.195.111.91]) by mx.google.com with ESMTPSA id\r
  41         n48sm22997399eeg.17.1969.12.31.16.00.00\r
  42         (version=TLSv1.2 cipher=RC4-SHA bits=128/128);\r
  43         Mon, 26 Aug 2013 10:58:40 -0700 (PDT)\r
  44 From: Jani Nikula <jani@nikula.org>\r
  45 To: Tomi Ollila <tomi.ollila@iki.fi>,\r
  46         Geoffrey Ferrari <geoffrey.ferrari@oriel.oxon.org>, notmuch@notmuchmail.org\r
  47 Subject: Re: [PATCH] emacs: bugfix notmuch-mua-reply when signature is present\r
  48 In-Reply-To: <m238q5ycsf.fsf@guru.guru-group.fi>\r
  49 References:\r
  50  <1375961732-14327-1-git-send-email-geoffrey.ferrari@oriel.oxon.org>\r
  51         <m238qjqnt5.fsf@guru.guru-group.fi>     <m238q5ycsf.fsf@guru.guru-group.fi>\r
  52 User-Agent: Notmuch/0.16+32~g01f5508 (http://notmuchmail.org) Emacs/24.3.1\r
  53         (x86_64-pc-linux-gnu)\r
  54 Date: Mon, 26 Aug 2013 20:58:43 +0300\r
  55 Message-ID: <87bo4k2v1o.fsf@nikula.org>\r
  56 MIME-Version: 1.0\r
  57 Content-Type: text/plain\r
  58 X-BeenThere: notmuch@notmuchmail.org\r
  59 X-Mailman-Version: 2.1.13\r
  60 Precedence: list\r
  61 List-Id: "Use and development of the notmuch mail system."\r
  62         <notmuch.notmuchmail.org>\r
  63 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  64         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  65 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  66 List-Post: <mailto:notmuch@notmuchmail.org>\r
  67 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  68 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  69         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  70 X-List-Received-Date: Mon, 26 Aug 2013 17:58:51 -0000\r
  71 \r
  72 On Mon, 19 Aug 2013, Tomi Ollila <tomi.ollila@iki.fi> wrote:\r
  73 > On Fri, Aug 09 2013, Tomi Ollila <tomi.ollila@iki.fi> wrote:\r
  74 >\r
  75 >> On Thu, Aug 08 2013, Geoffrey Ferrari <geoffrey.ferrari@oriel.oxon.org> wrote:\r
  76 >>\r
  77 >>> From: "Geoffrey H. Ferrari" <geoffrey.ferrari@oriel.oxon.org>\r
  78 >>>\r
  79 >>> When composing a reply, notmuch-mua-reply tries to be smart and cite\r
  80 >>> the original message by inserting it before the user signature, if\r
  81 >>> one is present. However, the existing method of backward searching\r
  82 >>> from the end of the buffer to find the signature separator and then\r
  83 >>> moving one line up results in the original message being cited in\r
  84 >>> the message headers. That's because at this point the message looks\r
  85 >>> like this (with | representing point after searching for the\r
  86 >>> signature separator):\r
  87 >>>\r
  88 >>> From: xxx\r
  89 >>> To: xxx\r
  90 >>> Subject: xxx\r
  91 >>> --text follows this line--\r
  92 >>> |--\r
  93 >>> My fancy signature\r
  94 >>>\r
  95 >>> With this patch, a newline is opened instead, so that the orignal\r
  96 >>> message is cited above the signature but still in the message text.\r
  97 >>> ---\r
  98 >>\r
  99 >> This probably fixes #1 in id:8762auwgfl.fsf@steelpick.2x.cz\r
 100 >>\r
 101 >> ( http://mid.gmane.org/8762auwgfl.fsf@steelpick.2x.cz )\r
 102 >>\r
 103 >> ... I just noticed I have no clue how to set message signature to test\r
 104 >> this... ;/\r
 105 >\r
 106 > Now I tried something fancy and executed `echo foobar > ~/.signature`\r
 107 >\r
 108 > Now I get the "--\nfoobar" at the end of all emails I'm, composing -- but\r
 109 > I don't see any problems when replying... so I'd be interested to see\r
 110 > in what circumstances could I get the problem this patch solves exposed.\r
 111 \r
 112 Hi Geoffrey, we can't reproduce the issue this patch claims to fix. Care\r
 113 to provide more info please?\r
 114 \r
 115 BR,\r
 116 Jani.\r
 117 \r
 118 \r
 119 >\r
 120 >>\r
 121 >> Tomi\r
 122 >\r
 123 > Tomi\r
 124 >\r
 125 >\r
 126 >>\r
 127 >>\r
 128 >>\r
 129 >>>  emacs/notmuch-mua.el |    2 +-\r
 130 >>>  1 file changed, 1 insertion(+), 1 deletion(-)\r
 131 >>>\r
 132 >>> diff --git a/emacs/notmuch-mua.el b/emacs/notmuch-mua.el\r
 133 >>> index 2baae5f..86f164d 100644\r
 134 >>> --- a/emacs/notmuch-mua.el\r
 135 >>> +++ b/emacs/notmuch-mua.el\r
 136 >>> @@ -194,7 +194,7 @@ list."\r
 137 >>>        ;; if one is present\r
 138 >>>        (goto-char (point-max))\r
 139 >>>        (if (re-search-backward message-signature-separator nil t)\r
 140 >>> -     (forward-line -1)\r
 141 >>> +     (newline)\r
 142 >>>     (goto-char (point-max)))\r
 143 >>>  \r
 144 >>>        (let ((from (plist-get original-headers :From))\r
 145 >>> -- \r
 146 >>> 1.7.10.4\r
 147 >>>\r
 148 >>> _______________________________________________\r
 149 >>> notmuch mailing list\r
 150 >>> notmuch@notmuchmail.org\r
 151 >>> http://notmuchmail.org/mailman/listinfo/notmuch\r
 152 > _______________________________________________\r
 153 > notmuch mailing list\r
 154 > notmuch@notmuchmail.org\r
 155 > http://notmuchmail.org/mailman/listinfo/notmuch\r