Re: git and time

From: Shawn Pearce <>
Date: 2006-09-28 07:36:07
Junio C Hamano <> wrote:
> Shawn Pearce <> writes:
> > You are missing:
> > +	setup_ident();
> >
> > Without that reflog can't get the proper committer data from the
> > host's gecos information.  This is probably what is desired for
> > most pushes over SSH.
> What's even more interesting is when there is .git/config file
> and you do not override it with environment variables; the log
> entry will be made under the name of the repository in
> such a case.

Yes.  :-)

Shared repository owners might want to not set in their
config files.

Another way around this would be to create a variation of
git_default_config for use in receive-pack.c that doesn't recognize
> >> +	git_config(git_default_config);
> >> +
> >>  	write_head_info();
> Often setup_ident() needs to go together with git_config(), and
> you need to remember that setup must come before config.  These
> rules are a bit cumbersome to follow and I often forget.
> I wonder if we can have a simpler start-up sequence perhaps to
> avoid future mistakes like this?

I think most places we are "starting up" we invoke both, in that
order.  Which sort of implies maybe we could just fold setup_ident
into git_config.

To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to
More majordomo info at
Received on Thu Sep 28 07:36:23 2006

This archive was generated by hypermail 2.1.8 : 2006-09-28 07:37:06 EST