Re: How do I specify a revision for "git blame" by date?

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

 



On Fri, Jun 15, 2012 at 06:02:39AM -0700, perryh@xxxxxxxxxxxxxx wrote:

> > You are looking at two different dates:
> >
> > a) The dates stored within the commit object:
> >
> >    - committer date, similarly shown by %ci: when the commit
> >      was created.
> >    - author date, shown by %ai: when "this" commit was "first
> >      created".
> >
> >    These are properties of the commits, and thus part of the project
> >    history.  Anyone who clones the project sees the same dates.
> >
> > b) The dates in the reflog, tracking the movement of *your local*
> >    refs (like the name implies).  The HEAD reflog tracks what *you*
> >    had "currently checked out" at a given time.
> >
> > An automated search in (b) is possible with the @{} syntax, but
> > note that it tracks the *branch state*.  It says nothing about how
> > "current" a certain resulting commit was.  For example, if you say
> >
> >   git checkout v1.6.0
> >   sleep 2
> >   git checkout -
> >   git show '@{1 second ago}'
> >
> > you will see the commit for v1.6.0 from back in 2008.
> >
> > An automated search in (a) is hard, mostly because in nonlinear
> > history there is not usually a single (and well-defined) commit
> > that could be returned.
> 
> Given that we are discussing "git blame", which reports on the
> history of a file (specifically the commit in which each line
> originated), I think (a) must logically be included in the search
> -- especially if (as in the example) the date specified is earlier
> than any time in (b).  Perhaps such a request needs to use some
> other syntax rather than @{}, but surely the capability ought to
> be provided somehow.

It would have to use a different syntax, as reflog times (selected by
@{}) and commit dates (usually selected via --since and --until) are
measuring fundamentally different things, and have no meaningful
relation to each other.

But that still doesn't address the issue that (a) is not well-defined.
Imagine I have this history:


  A--B--C---G--H
   \       /
    D--E--F

that is, two lines of development splitting at A and merging at H. And
imagine the commit timestamps are (let's just refer to them as integers
for the sake of simplicity, but they are representing days or seconds or
whatever):

  A(1)--B(2)--C(3)--G(7)--H(8)
   \               /
    D(2)--E(4)--F(6)

What does it mean to ask for the commit at time=5? If you were to choose
the commit with the highest timestamp <= 5, you would pick E. But there
are two independent, simultaneous lines of development, and at time=5 on
the top branch, the most current commit was C.

So the question doesn't make sense. There was no one commit at time=5;
there were multiple (as many as you have simultaneous branches). You can
pick a winner out of them, but there are multiple ways to do it. For
example, you might want to say "of all lines of development, the one
that had most recently committed as of time=5" (and of course you can
still have a tie there). Or you could say "along the first-parent
ancestry, the commit that was most recent as of time=5". I am sure one
could come up with others, as well.

Git-blame expects you to give it a well-defined point (as it must, since
it is a backwards walk down history showing what led to a particular set
of content; it wouldn't make sense to feed it multiple starting points).
You could do so by asking rev-list to walk the graph according to your
requirements and feeding the result to blame, like:

  # most recent on any line of development that is merged to HEAD
  git blame `git rev-list -1 --until=5 HEAD`

  # most recent on any line of development in the whole repo
  git blame `git rev-list -1 --until=5 --all`

  # most recent version on the first-parent; if you follow a
  # topic-branch workflow and always merge up into "master", then this
  # will blame what was on master at time=5
  git blame `git rev-list -1 --until=5 --first-parent HEAD`

-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]