Re: [PATCH 3/3] commit: Show the committer ident when is different from the parent

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, May 01, 2008 at 10:34:44AM +0200, Santi Béjar wrote:

> >  > 1) the committer ident is different from the parent
> >  > 2) the committer ident is set automatically
> >
> >  Honestly, I think just "2)" is probably fine (where automatically
> >  presumably means "from GECOS"). I see what you are trying to accomplish
> >  with "1)", but it's so workflow specific as to be useless.
> 
> I don't see "1) as workflow specific, it is there to minimize the
> number of time it is shown, ie somebody might prefer the automatic
> ident, so as long as it does not change it is not shown.

Yes, but whether it actually succeeds in minimizing is dependent on the
workflow of the user. I.e., it works great if I tend to build on small,
personal repositories (or topic branches), or if I tend to receive other
people's work by patches that I commit myself. But if I am pulling from
somebody else's repo (as many of us do from git.git), or if I use a
shared repo with the other committers, then I am very likely to be
building on a parent that was committed by somebody else (and
consequently will see the committer mentioned way more often than was
intended).

-Peff
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux