From: Luben Tuikov <ltuikov@yahoo.com>
To: Petr Baudis <pasky@suse.cz>, Junio C Hamano <junkio@cox.net>
Cc: git@vger.kernel.org, Luben Tuikov <ltuikov@yahoo.com>,
Jakub Narebski <jnareb@gmail.com>,
Ryan Anderson <ryan@michonline.com>,
Johannes Schindelin <Johannes.Schindelin@gmx.de>,
Martin Langhoff <martin@catalyst.net.nz>,
Martyn Smith <martyn@catalyst.net.nz>,
Fredrik Kuivinen <freku045@student.liu.se>,
Linus Torvalds <torvalds@osdl.org>
Subject: Re: perhaps time to remove git_blame from gitweb, and git-annotate?
Date: Fri, 6 Oct 2006 11:55:29 -0700 (PDT) [thread overview]
Message-ID: <20061006185529.9481.qmail@web31802.mail.mud.yahoo.com> (raw)
In-Reply-To: <20061006161637.GS20017@pasky.or.cz>
--- Petr Baudis <pasky@suse.cz> wrote:
> Dear diary, on Thu, Oct 05, 2006 at 10:13:15AM CEST, I got a letter
> where Junio C Hamano <junkio@cox.net> said that...
> > It's been a while since we lost git_blame from %actions list. I
> > am wondering maybe it's time to remove it, after 1.4.3 happens.
>
> I will not mourn git-annotate disappearance (perhaps it could stay an
> alias to git-blame -c; I don't like this UI-wise but we already do this
> kind of thing with git-log / git-whatchanged). I still like gitweb blame
> better than blame2 but I'll just patch blame to look like blame2 (or
> better) and be happy with that.
That's exactly what I don't want to happen. blame2 is much less
bloated than blame both in code and in visual appearance and in using
git. And this was the whole point: quick, short, fast and straight
to the point.
I'd like to keep the blame interface as simple as possible, more
window estate given to the code lines, and as little as possible to
the commit id: a clickable commit-8 and now we also have clickable
line_no to show us the state of the file, is more than enough.
So far, blame2 has been by far the better "annotate" (as it is called
in the other SCMs) interface I've seen in four other SCMs (some proprietary).
Let's keep it like this.
When data-mining code, what I'm interested in is: where did this line
of code come from (commit-8), is it a part of a larger chunk (zebra
coloring) and how it relates to the surrounding code. Blame2 is more
than efficient at this.
Luben
>
> --
> Petr "Pasky" Baudis
> Stuff: http://pasky.or.cz/
> #!/bin/perl -sp0777i<X+d*lMLa^*lN%0]dsXx++lMlN/dsM0<j]dsj
> $/=unpack('H*',$_);$_=`echo 16dio\U$k"SK$/SM$n\EsN0p[lN*1
> lK[d2%Sa2/d0$^Ixp"|dc`;s/\W//g;$_=pack('H*',/((..)*)$/)
>
next prev parent reply other threads:[~2006-10-06 18:55 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-10-05 0:59 [PATCH 1/2] gitweb: blame: Print just a single new line char per table row Luben Tuikov
2006-10-05 2:35 ` Junio C Hamano
2006-10-05 6:48 ` Luben Tuikov
2006-10-05 8:13 ` perhaps time to remove git_blame from gitweb, and git-annotate? Junio C Hamano
2006-10-06 13:07 ` Johannes Schindelin
2006-10-06 17:52 ` Luben Tuikov
2006-10-07 5:37 ` Martin Langhoff (CatalystIT)
2006-10-06 16:16 ` Petr Baudis
2006-10-06 18:55 ` Luben Tuikov [this message]
2006-10-06 19:08 ` Petr Baudis
2006-10-06 20:21 ` Luben Tuikov
2006-10-09 10:32 ` [PATCH 1/1] Remove git-annotate.perl and create a builtin-alias for git-blame Ryan Anderson
2006-10-09 10:37 ` perhaps time to remove git_blame from gitweb, and git-annotate? Ryan Anderson
2006-10-09 22:33 ` Obituary for git-annotate Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20061006185529.9481.qmail@web31802.mail.mud.yahoo.com \
--to=ltuikov@yahoo.com \
--cc=Johannes.Schindelin@gmx.de \
--cc=freku045@student.liu.se \
--cc=git@vger.kernel.org \
--cc=jnareb@gmail.com \
--cc=junkio@cox.net \
--cc=martin@catalyst.net.nz \
--cc=martyn@catalyst.net.nz \
--cc=pasky@suse.cz \
--cc=ryan@michonline.com \
--cc=torvalds@osdl.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).