all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Thierry Volpiatto <thierry.volpiatto@gmail.com>
To: 13321@debbugs.gnu.org
Subject: bug#13321: 24.3.0.50; Gmail error when replying from Gnus to github
Date: Thu, 21 Feb 2013 20:40:32 +0100	[thread overview]
Message-ID: <87fw0pihfz.fsf@gmail.com> (raw)
In-Reply-To: <87a9surugk.fsf@gmail.com>

Thierry Volpiatto <thierry.volpiatto@gmail.com> writes:

> Hi Lars,
>
> Thierry Volpiatto <thierry.volpiatto@gmail.com> writes:
>
>> Hi Lars,
>>
>> Thierry Volpiatto <thierry.volpiatto@gmail.com> writes:
>>
>> I have isolated the problem to message.el, so when I load the message.el
>> of emacs-24 branch the problem disapear.
>> I will try to (re)diff the file to found out what is wrong there.
>> Perhaps you can have a look on your side too ?
>
> So the problem comes from `message-get-reply-headers', more exactly this
> piece of code:
>
>       ;; Make sure this message goes to the author if this is a wide
>       ;; reply, since Reply-To address may be a list address a mailing
>       ;; list server added.
>       (when (and wide author)
> 	(setq cc (concat author ", " cc)))
>       (when (or wide (not author))
> 	(setq author (or (message-fetch-field "from") ""))))
>
> In our case the "from" field contain "notification@github.com" which is
> the place where we should never reply.

Here the commit that introduce this bug:

--8<---------------cut here---------------start------------->8---
2559724 * 
        | Author: Katsumi Yamaoka <yamaoka@jpl.org>
        | Date:   Mon Nov 19 06:24:03 2012 +0000
        | 
        |     message.el (message-get-reply-headers): Make sure the reply goes to the author if it is a wide reply
        | 
        |  lisp/ChangeLog  |    5 +++++
        |  lisp/message.el |   13 +++++++++----
        |  2 files changed, 14 insertions(+), 4 deletions(-)
--8<---------------cut here---------------end--------------->8---

With this change, here what is happening, sending a reply in same topic
on github:

;; BAD emacs-24.3.50
;; To: Le Wang <notifications@github.com>
;; Cc: emacs-helm/helm <helm@noreply.github.com>,  emacs-helm/helm <reply+i-[...]1533939@reply.github.com>

;; GOOD emacs-24.2.93
;; To: emacs-helm/helm <reply+i-[...]33939@reply.github.com>
;; Cc: emacs-helm/helm <helm@noreply.github.com>


-- 
Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997 






  reply	other threads:[~2013-02-21 19:40 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-31 17:40 bug#13321: 24.3.0.50; Gmail error when replying from Gnus to github Thierry Volpiatto
2012-12-31 20:26 ` Andreas Schwab
2013-01-02 20:13 ` Lars Magne Ingebrigtsen
2013-01-02 21:14   ` Thierry Volpiatto
2013-01-02 22:50     ` Lars Magne Ingebrigtsen
2013-01-03  5:34       ` Thierry Volpiatto
2013-01-03  6:10         ` Lars Magne Ingebrigtsen
2013-01-03  6:41           ` Thierry Volpiatto
2013-01-08  6:35             ` Thierry Volpiatto
2013-02-14  6:35               ` Thierry Volpiatto
2013-02-21 16:22               ` Thierry Volpiatto
2013-02-21 16:53                 ` Thierry Volpiatto
2013-02-21 19:40                   ` Thierry Volpiatto [this message]
2013-02-22  7:15                     ` Katsumi Yamaoka
2013-02-22  7:33                       ` Thierry Volpiatto
2013-02-22  9:34                         ` Katsumi Yamaoka
2013-02-22 10:06                           ` Thierry Volpiatto
2013-02-22 11:17                             ` Katsumi Yamaoka
2013-02-22 14:09                               ` Thierry Volpiatto
2013-02-23  6:56                               ` Thierry Volpiatto
2013-02-23  8:18                               ` Thierry Volpiatto
2013-02-23 10:06                                 ` Thierry Volpiatto
2013-02-23 10:34                               ` Thierry Volpiatto
2013-02-24  6:49                               ` Thierry Volpiatto
2013-02-25  6:59                                 ` Katsumi Yamaoka
2013-02-25  7:08                                   ` Katsumi Yamaoka
2013-02-25  7:15                                   ` Thierry Volpiatto
2013-02-25  7:33                                     ` Katsumi Yamaoka
2013-02-25  9:17                                       ` Thierry Volpiatto
2013-02-25 11:29                                         ` Katsumi Yamaoka
2013-02-22  7:43                       ` Thierry Volpiatto
2013-02-22  7:51                       ` Thierry Volpiatto
2013-02-22  7:53                       ` Thierry Volpiatto

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87fw0pihfz.fsf@gmail.com \
    --to=thierry.volpiatto@gmail.com \
    --cc=13321@debbugs.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.