* git-blame and blame GUI wishlist
@ 2007-08-05 22:09 Jakub Narebski
0 siblings, 0 replies; only message in thread
From: Jakub Narebski @ 2007-08-05 22:09 UTC (permalink / raw)
To: git
1. In git-gui blame viewer you can click on shortened sha1 of a commit
to see file at given version. Unfortunately git-gui goes to other
version of a file by reblaming a file starting from scratch.
It would be nice to have
git blame --incremental=<revision1> <revision2> -- <file>
which would blame only lines which differ between <revision1>
and <revision2>. We can assume that <revision2> is ancestor of
<revision1>, I think. I'm not completely sure if it is possible
and reasonable, i.e. if one can assume that only lines which
differ need reblaming...
And of course use this feature in "git-gui blame" which should
speed up exploring file history a bit.
2. Sometimes I'm interested only in part of a file. Thus I'd like
"git-gui blame" to understand line-range parameter of git-blame.
It means that I'd like to be able to say for example:
git gui blame -L '/^sub git_feed/,/^}$/' HEAD -- gitweb/gitweb.perl
and have only given range of lines blamed, with the rest grayed out
in the "inactive control" style. Of course in the case I specified
line range incorrectly, I'd like to have menu entry or a button to
reblame _whole_ file... Or perhaps do blaming of a whole file in the
background, after blaming specified range fo lines.
This should speed up "git gui blame" a bit. By the way, of course
git-gui should start with the selected line-range visible.
3. Sometimes when reviewing patches I do wonder: why the preimage looks
like it looks like. It would be useful in that situation to be able
to blame the patch (context lines and pre-image lines). Although I'm
not suer if it would bevery useful and used often [enough].
P.S. qgit uses some bottom-up all-files mechanism to get the blame of
all the files together. Could anyone describe in few sentences the
algorithm qgit uses? Would it be useful to put it available e.g. in
gitlib-thin for other viewers, or not; what do you think?
P.P.S. Of couse the examples and ideas are not limited to "git gui
blame"...
--
Jakub Narebski
Poland
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2007-08-05 23:38 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-08-05 22:09 git-blame and blame GUI wishlist Jakub Narebski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).