git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: Marius Storm-Olsen <marius@trolltech.com>
Cc: Git Mailing List <git@vger.kernel.org>
Subject: Re: 'git format-patch' on older commits, and mailinglist
Date: Tue, 10 Jun 2008 15:37:40 -0700	[thread overview]
Message-ID: <7vfxrkdih7.fsf@gitster.siamese.dyndns.org> (raw)
In-Reply-To: <484E52DE.1070100@trolltech.com> (Marius Storm-Olsen's message of "Tue, 10 Jun 2008 12:09:34 +0200")

Marius Storm-Olsen <marius@trolltech.com> writes:

> Recently, I sent out patches which were fixed up with --amend on older
> commits. When I sent them out, the patches contained
>     Date: <more than 2 days old datestamp>
> in the headers. Now, sending these mails is fine, and mail clients
> generally handles it perfectly fine. However, after doing this I got
> an email from postmaster@vger.kernel.org, basically telling me to not
> do this, since they get a lot of bounces where the return is marked
> with
>     Diagnostic Code: smtp; 550 (4.5 DATE_IN_PAST_48_96 Date: is 48 to
> 96 hours before Received: date)
>
> This is understandable. The question is, do we fix the tools to handle
> this, so that emails are always generated with now() date, and the
> commit content contains a tag for the original commit; or do we simply
> say, always send patches to the mailing list with a current timestamp?

I think the right solution would be for send-email to move Date: to the
beginning of the body part, just like it adds an extra From: there when
the sender is different from the author, and not lie about the mail
transmission date.

Then we won't lose the authorship date information in the resulting commit
and will still keep the mailpath happy.

      reply	other threads:[~2008-06-10 22:38 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-06-10 10:09 'git format-patch' on older commits, and mailinglist Marius Storm-Olsen
2008-06-10 22:37 ` Junio C Hamano [this message]

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=7vfxrkdih7.fsf@gitster.siamese.dyndns.org \
    --to=gitster@pobox.com \
    --cc=git@vger.kernel.org \
    --cc=marius@trolltech.com \
    /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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).