All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dennis Kaarsemaker <dennis@kaarsemaker.net>
To: Joe Perches <joe@perches.com>,
	Julia Lawall <julia.lawall@lip6.fr>,
	Alexey Dobriyan <adobriyan@gmail.com>, git <git@vger.kernel.org>
Cc: Greg KH <gregkh@linuxfoundation.org>,
	LKML <linux-kernel@vger.kernel.org>,
	Sasha Levin <levinsasha928@gmail.com>,
	"ksummit-discuss@lists.linuxfoundation.org"
	<ksummit-discuss@lists.linuxfoundation.org>
Subject: Re: [Ksummit-discuss] checkkpatch (in)sanity ?
Date: Sun, 28 Aug 2016 23:24:58 +0200	[thread overview]
Message-ID: <1472419498.4265.16.camel@kaarsemaker.net> (raw)
In-Reply-To: <1472413969.26978.108.camel@perches.com>

On zo, 2016-08-28 at 12:52 -0700, Joe Perches wrote:
> On Sun, 2016-08-28 at 11:59 +0200, Julia Lawall wrote:
> > 
> > On Sun, 28 Aug 2016, Alexey Dobriyan wrote:
> []
> > 
> > > 
> > > The problem is that c-h.pl generates noise in the commit history
> > > and
> > > makes git-blame less useful than it can be.
> > Could it be that this is a problem with git blame, rather than with
> > checkpatch?  Last year there was a discussion on this list about
> > how there
> > is an option to git blame that will cause it to step through the
> > history,
> > and not show only the most recent patch that has modified a given
> > line.
> It is more or less an ease-of-use limitation of git blame.
> 
> There are some that want an ncurses only version of git blame
> that could
> use arrow-key style navigation for historical commit
> line-ranges.
> 
> git gui blame kind of works, but it's not ncurses/text based.
> git-cola kind of works too, but it's not text based either.
> 
> Are there other existing tools for blame history viewing?

tig has a neat way of doing blame history digging: do a `tig blame
filename`, select a line and hit the comma key to re-blame from the
parent of the commit that was blamed for that line.

(hat tip to Jeff King who pointed out this trick recently)

D.

WARNING: multiple messages have this Message-ID (diff)
From: Dennis Kaarsemaker <dennis@kaarsemaker.net>
To: Joe Perches <joe@perches.com>,
	Julia Lawall <julia.lawall@lip6.fr>,
	Alexey Dobriyan <adobriyan@gmail.com>, git <git@vger.kernel.org>
Cc: "Levin, Alexander" <alexander.levin@verizon.com>,
	Greg KH <gregkh@linuxfoundation.org>,
	Sasha Levin <levinsasha928@gmail.com>,
	"ksummit-discuss@lists.linuxfoundation.org" 
	<ksummit-discuss@lists.linuxfoundation.org>,
	LKML <linux-kernel@vger.kernel.org>
Subject: Re: [Ksummit-discuss] checkkpatch (in)sanity ?
Date: Sun, 28 Aug 2016 23:24:58 +0200	[thread overview]
Message-ID: <1472419498.4265.16.camel@kaarsemaker.net> (raw)
In-Reply-To: <1472413969.26978.108.camel@perches.com>

On zo, 2016-08-28 at 12:52 -0700, Joe Perches wrote:
> On Sun, 2016-08-28 at 11:59 +0200, Julia Lawall wrote:
> > 
> > On Sun, 28 Aug 2016, Alexey Dobriyan wrote:
> []
> > 
> > > 
> > > The problem is that c-h.pl generates noise in the commit history
> > > and
> > > makes git-blame less useful than it can be.
> > Could it be that this is a problem with git blame, rather than with
> > checkpatch?  Last year there was a discussion on this list about
> > how there
> > is an option to git blame that will cause it to step through the
> > history,
> > and not show only the most recent patch that has modified a given
> > line.
> It is more or less an ease-of-use limitation of git blame.
> 
> There are some that want an ncurses only version of git blame
> that could
> use arrow-key style navigation for historical commit
> line-ranges.
> 
> git gui blame kind of works, but it's not ncurses/text based.
> git-cola kind of works too, but it's not text based either.
> 
> Are there other existing tools for blame history viewing?

tig has a neat way of doing blame history digging: do a `tig blame
filename`, select a line and hit the comma key to re-blame from the
parent of the commit that was blamed for that line.

(hat tip to Jeff King who pointed out this trick recently)

D.

  parent reply	other threads:[~2016-08-28 21:25 UTC|newest]

Thread overview: 72+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-27 20:40 [Ksummit-discuss] checkkpatch (in)sanity ? Joe Perches
2016-08-27 20:40 ` Joe Perches
2016-08-28  1:06 ` [Ksummit-discuss] " Levin, Alexander
2016-08-28  1:06   ` Levin, Alexander
2016-08-28  1:42   ` [Ksummit-discuss] " Joe Perches
2016-08-28  1:42     ` Joe Perches
2016-08-28  2:20     ` [Ksummit-discuss] " Joe Perches
2016-08-28  2:20       ` Joe Perches
2016-08-28  2:47     ` [Ksummit-discuss] " Levin, Alexander
2016-08-28  2:47       ` Levin, Alexander
2016-08-28 17:15       ` [Ksummit-discuss] " Joe Perches
2016-08-28 17:15         ` Joe Perches
2016-08-28 17:59         ` [Ksummit-discuss] " Greg KH
2016-08-28 17:59           ` Greg KH
2016-08-28 22:37         ` [Ksummit-discuss] " Levin, Alexander
2016-08-28 22:37           ` Levin, Alexander
2016-08-28 23:20           ` [Ksummit-discuss] " Joe Perches
2016-08-28 23:20             ` Joe Perches
2016-08-29  2:22             ` [Ksummit-discuss] " Levin, Alexander
2016-08-29  2:22               ` Levin, Alexander
2016-08-29  8:20               ` [Ksummit-discuss] " Christoph Hellwig
2016-08-29  8:20                 ` Christoph Hellwig
2016-08-29  7:15           ` [Ksummit-discuss] " Alexandre Belloni
2016-08-29  7:15             ` Alexandre Belloni
2016-08-29  9:01             ` Arnd Bergmann
2016-08-29  9:01               ` Arnd Bergmann
2016-08-29 12:47               ` Joe Perches
2016-08-29 12:47                 ` Joe Perches
2016-08-29 17:16                 ` Josh Triplett
2016-08-29 17:16                   ` Josh Triplett
2016-08-29 17:41                   ` Joe Perches
2016-08-29 17:41                     ` Joe Perches
2016-08-29 17:46         ` Luck, Tony
2016-08-29 17:46           ` Luck, Tony
2016-08-29 18:01           ` Joe Perches
2016-08-29 18:47             ` Joe Perches
2016-08-29 19:08             ` Josh Triplett
2016-08-29 19:08               ` Josh Triplett
2016-08-29 21:07             ` Arnd Bergmann
2016-08-29 21:07               ` Arnd Bergmann
2016-08-29 11:15     ` Kalle Valo
2016-08-29 11:15       ` Kalle Valo
2016-08-29 12:30       ` [Ksummit-discuss] " Joe Perches
2016-08-29 12:30         ` Joe Perches
2016-08-29 18:01         ` [Ksummit-discuss] " Kalle Valo
2016-08-29 18:01           ` Kalle Valo
2016-08-29 19:00           ` [Ksummit-discuss] " Joe Perches
2016-08-29 19:00             ` Joe Perches
2016-08-29 21:00           ` [Ksummit-discuss] " Arnd Bergmann
2016-08-29 21:00             ` Arnd Bergmann
2016-08-28  7:56   ` Alexey Dobriyan
2016-08-28  7:56     ` Alexey Dobriyan
2016-08-28  9:59     ` Julia Lawall
2016-08-28  9:59       ` Julia Lawall
2016-08-28 19:52       ` Joe Perches
2016-08-28 19:52         ` Joe Perches
2016-08-28 20:35         ` Jiri Kosina
2016-08-28 20:35           ` Jiri Kosina
2016-08-28 21:24         ` Dennis Kaarsemaker [this message]
2016-08-28 21:24           ` Dennis Kaarsemaker
2016-08-28 21:57           ` Joe Perches
2016-08-28 21:57             ` Joe Perches
2016-08-29 19:06 ` Dan Carpenter
2016-08-29 19:06   ` Dan Carpenter
2016-08-29 19:10   ` Josh Triplett
2016-08-29 19:10     ` Josh Triplett
2016-08-29 19:17     ` Dan Carpenter
2016-08-29 19:17       ` Dan Carpenter
2016-08-29 19:34       ` Joe Perches
2016-08-29 19:34         ` Joe Perches
2016-08-29 19:21     ` Joe Perches
2016-08-29 19:21       ` Joe Perches

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=1472419498.4265.16.camel@kaarsemaker.net \
    --to=dennis@kaarsemaker.net \
    --cc=adobriyan@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=joe@perches.com \
    --cc=julia.lawall@lip6.fr \
    --cc=ksummit-discuss@lists.linuxfoundation.org \
    --cc=levinsasha928@gmail.com \
    --cc=linux-kernel@vger.kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.