All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jakub Narebski <jnareb@gmail.com>
To: "Karl Hasselström" <kha@treskal.com>
Cc: Jaroslav Kysela <perex@perex.cz>,
	Johannes Schindelin <Johannes.Schindelin@gmx.de>,
	git@vger.kernel.org
Subject: Re: [PATCH] Keep committer and committer dates
Date: Wed, 04 Jun 2008 04:12:41 -0700 (PDT)	[thread overview]
Message-ID: <m3ej7dzc7y.fsf@localhost.localdomain> (raw)
In-Reply-To: <20080604102906.GA2126@diana.vm.bytemark.co.uk>

Karl Hasselström <kha@treskal.com> writes:

> On 2008-06-04 11:16:46 +0200, Jaroslav Kysela wrote:
> 
> > On Tue, 3 Jun 2008, Johannes Schindelin wrote:
> >
> > > That feels really funny, given that the guy running git-am _is_
> > > the committer, not whoever provided some extra headers to the
> > > mailbox.
> >
> > Yes, the implementatation does not make sense for public patch
> > handling, but if you do various things locally with git-rebase or
> > git-am (pack picking from another repo), you may consider it useful.
> 
> But still, you're creating new commits, so they should have your name
> on them.

Yes, if you are _creating_ *commits*, then you are *committer*, isn't it?

-- 
Jakub Narebski
Poland
ShadeHawk on #git

  reply	other threads:[~2008-06-04 11:13 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-06-03 14:59 [PATCH] Keep committer and committer dates Jaroslav Kysela
2008-06-03 20:16 ` Johannes Schindelin
2008-06-04  9:16   ` Jaroslav Kysela
2008-06-04 10:29     ` Karl Hasselström
2008-06-04 11:12       ` Jakub Narebski [this message]
2008-06-04 12:29         ` Jaroslav Kysela
2008-06-04 14:22           ` Nicolas Pitre
2008-06-04 15:43           ` Johannes Schindelin
2008-06-05  1:03             ` Stephan Beyer
2008-06-05  4:25               ` Johannes Schindelin
2008-06-05  5:03                 ` Junio C Hamano
2008-06-05  7:10                 ` Johannes Sixt
2008-06-05 12:44                   ` Jaroslav Kysela
2008-06-05 13:13                     ` Johannes Schindelin
2008-06-05 13:38                       ` Jaroslav Kysela
2008-06-05 13:54                         ` Johannes Schindelin
2008-06-05 14:11                           ` Jaroslav Kysela
2008-06-05 14:46                             ` Johannes Schindelin
2008-06-05 15:16                               ` Jaroslav Kysela
2008-06-05 16:34                                 ` Boaz Harrosh

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=m3ej7dzc7y.fsf@localhost.localdomain \
    --to=jnareb@gmail.com \
    --cc=Johannes.Schindelin@gmx.de \
    --cc=git@vger.kernel.org \
    --cc=kha@treskal.com \
    --cc=perex@perex.cz \
    /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.