Re: Directory renames without breaking git log.

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

 



Kai Blin <kai@xxxxxxxxx> writes:

> On Thursday 04 September 2008 21:49:27 Junio C Hamano wrote:
>
>> > git blame still works, and git log --sparse path/to/file works, of
>> > course. --sparse makes giving a path a bit pointless, of course, but we
>> > probably can live with that for time being. I'm still open for
>> > suggestions, of course. :)
>>
>> Give both directories, like:
>>
>> 	"git log -- newdir olddir"
>>
>> perhaps?
>
> Better, but really ugly, as we'll have to keep doing this for the rest
> of the project's life to get the full history. And while it's all nice
> and fun for git log -- source3/configure.in source/configure.in, it's
> less fun for deeper paths.

Yes, following across subtree merges _could_ be improved.

But another thing I should mention in this context is that you should not
take --follow option (at least in the current form) too seriously.

I see it's been a while --- the last time I did this was October 2006 if I
am not mistaken.  It's time of the year I should point at one of the most
important articles ever written on this mailing list:

    http://thread.gmane.org/gmane.comp.version-control.git/27/focus=217

After understanding what Linus envisioned back then, why what he said are
important are important and why what he dismissed as uninteresting are
indeed uninteresting, things to think about now are:

 - "blame" (especially with -C -C), as you found out already, does answer
   the more important question "where did this come from"; and

 - the question "log --follow $this_file" is asking is exactly "where did
   this file come from".  Remember what adjective was used for the
   question in the article?

I also should mention that --follow was done by Linus as a hack with
known limitations.

Potential improvements to follow possible renames fully would involve:

 * allow not just a single path but a set of pathspecs to be recorded
   during --follow traversal;

 * allow the above information be associated with individual commits, not
   as a single global state in the traversal machinery;

 * enhance the logic to update the pathspecs information kept above when
   you hit renames while traversing the history.  An important part of
   this job involves inferring a wholesale rename of a directory by
   looking at many files moved from one place to another, which we
   currently do not do anywhere in git.

If you do all of the above, it will become a feature, not a hack with
known limitation.  But I do not think anybody so far thought it is worth
the effort, only to answer the "where did this file come from" (ituot ue
zaf m eqzeunxq cgqefuaz).

Personally I feel that this is slightly closer to "I might do this myself
if I had infinite amount of time and I am really bored" than "I don't
care; patches welcome" category.
--
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