All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eric Blake <eblake@redhat.com>
To: git@vger.kernel.org
Cc: libvir-list@redhat.com
Subject: git send-email should not allow 'y' for in-reply-to
Date: Fri, 11 Jan 2013 09:39:06 -0700	[thread overview]
Message-ID: <50F0402A.1000108@redhat.com> (raw)
In-Reply-To: <50EFD066.60501@redhat.com>

[-- Attachment #1: Type: text/plain, Size: 949 bytes --]

[raising this UI wart to the git list]

On 01/11/2013 01:42 AM, Peter Krempa wrote:
> On 01/11/13 07:31, Chunyan Liu wrote:
>> This patch series is to...
[snip]

> 
> Please don't answer "y" when git send email shows the following prompt:
> 
> "Message-ID to be used as In-Reply-To for the first email?"
> 
> you should respond with a message ID there. Unfortunately we have a
> growing thread that contains submissions with this mistake.

Anyone willing to patch upstream 'git send-email' to reject a simple 'y'
rather than blindly sending a bad messageID for the in-reply-to field,
to help future users avoid this mistake?  Obviously, it won't help until
the patch eventually percolates into distros, so it would be a few more
months before we see the benefits, but down the road it will prevent
confusing threads.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 619 bytes --]

       reply	other threads:[~2013-01-11 16:39 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1357885869-20815-1-git-send-email-cyliu@suse.com>
     [not found] ` <50EFD066.60501@redhat.com>
2013-01-11 16:39   ` Eric Blake [this message]
2013-01-11 16:47     ` git send-email should not allow 'y' for in-reply-to Jeff King
2013-01-11 17:51       ` Eric Blake
2013-01-11 18:43       ` Hilco Wijbenga
2013-01-11 18:54         ` Jeff King
2013-02-24  9:03           ` Junio C Hamano
2013-01-11 20:13 Matt Seitz (matseitz)
2013-01-11 21:23 ` Jeff King
2013-01-11 21:53   ` Antoine Pelisse
2013-01-11 22:18     ` Junio C Hamano
2013-01-11 22:42       ` Antoine Pelisse
2013-01-11 23:54         ` Junio C Hamano
2013-01-12  1:02           ` Ben Aveling
2013-01-12  2:56             ` Junio C Hamano

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=50F0402A.1000108@redhat.com \
    --to=eblake@redhat.com \
    --cc=git@vger.kernel.org \
    --cc=libvir-list@redhat.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 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.