From: Junio C Hamano <gitster@pobox.com>
To: Jeff King <peff@peff.net>
Cc: Stephen Sinclair <radarsat1@gmail.com>, git@vger.kernel.org
Subject: Re: [PATCH] Add committer and author names to top of COMMIT_EDITMSG.
Date: Fri, 11 Jan 2008 21:06:44 -0800 [thread overview]
Message-ID: <7vprw7y6mz.fsf@gitster.siamese.dyndns.org> (raw)
In-Reply-To: <20080112045231.GB5211@coredump.intra.peff.net> (Jeff King's message of "Fri, 11 Jan 2008 23:52:31 -0500")
Jeff King <peff@peff.net> writes:
> This is obviously not 1.5.4 material, so I haven't given it that much
> thought either. But perhaps Stephen's "author message" should simply
> trigger any time the author is pulled from gecos? I suppose that would
> annoy people who use this feature all the time, but they can silence the
> "warning" with a simple git-config.
Well, we could certainly do that.
But I am not entirely happy with the idea of having to make the
default silly and inconvenient, only because otherwise new
people who did not even bother to read and follow the VERY FIRST
subsection of the tutorial that tells them that the first thing
to do is to use user.name and user.email would not notice their
problems, and experts know enough to squelch that broken
default. Middle level people (and newbies will quickly become
one of them) will be inconvenienced even though they followed
the tutorial's instruction, until they find the configuration
variable to turn that silly AUTHOR output off.
next prev parent reply other threads:[~2008-01-12 5:07 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-01-11 20:10 [PATCH] Add committer and author names to top of COMMIT_EDITMSG Stephen Sinclair
2008-01-11 21:26 ` Johannes Schindelin
2008-01-11 23:36 ` Junio C Hamano
2008-01-12 0:09 ` Stephen Sinclair
2008-01-12 0:22 ` Junio C Hamano
2008-01-12 1:33 ` Stephen Sinclair
2008-01-12 1:53 ` Junio C Hamano
2008-01-12 2:25 ` Stephen Sinclair
2008-01-12 4:57 ` Junio C Hamano
2008-01-12 7:26 ` Stephen Sinclair
2008-01-12 8:02 ` Junio C Hamano
2008-01-12 4:52 ` Jeff King
2008-01-12 5:06 ` Junio C Hamano [this message]
2008-01-12 5:30 ` Junio C Hamano
2008-01-12 5:32 ` Jeff King
2008-01-12 5:56 ` Junio C Hamano
2008-01-12 5:58 ` Jeff King
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=7vprw7y6mz.fsf@gitster.siamese.dyndns.org \
--to=gitster@pobox.com \
--cc=git@vger.kernel.org \
--cc=peff@peff.net \
--cc=radarsat1@gmail.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).