Re: [PATCH] builtin-clone: fix for new unpack_trees() semantics

> On Sun, 9 Mar 2008, Johannes Schindelin wrote:
> 
> > In git.git's "next" branch, unpack_trees() must specify source and target 
> > index.
> > 
> > Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
> > ---
> > 
> > 	To be squashed into 10/11
> > 
> >  builtin-clone.c |    2 ++
> >  1 files changed, 2 insertions(+), 0 deletions(-)
> > 
> > diff --git a/builtin-clone.c b/builtin-clone.c
> > index e4047ed..3890e12 100644
> > --- a/builtin-clone.c
> > +++ b/builtin-clone.c
> > @@ -534,6 +534,8 @@ int cmd_clone(int argc, const char **argv, const char *prefix)
> >  		opts.verbose_update = !option_quiet;
> >  		opts.merge = 1;
> >  		opts.fn = twoway_merge;
> > +		opts.src_index = &the_index;
> > +		opts.dst_index = &the_index;
> 
> Actually, I think the sensible thing is to just not do a merge here, since 
> we know there's no index beforehand and the two trees are the same.
> 
> I think the odd twoway merge of two copies of HEAD is just an artifact of 
> clone originally just doing "git checkout HEAD", and that got translated 
> various times failing to notice the special cases.