Re: [PATCH] blame: add option to print tips (--tips)

2017-01-24 Thread Pranit Bauva
Hey Junio, On Tue, Jan 24, 2017 at 12:06 AM, Junio C Hamano wrote: > Pranit Bauva writes: > >> We can probably make it useful with some extended efforts. I use >> git-blame and I sometimes find that I don't need things like the name >> of the author,

Re: [PATCH] blame: add option to print tips (--tips)

2017-01-23 Thread Junio C Hamano
Pranit Bauva writes: > We can probably make it useful with some extended efforts. I use > git-blame and I sometimes find that I don't need things like the name > of the author, time, timezone and not even the file name and I have to > use a bigger terminal. If we could

Re: [PATCH] blame: add option to print tips (--tips)

2017-01-23 Thread Pranit Bauva
Hey Junio, On Mon, Jan 23, 2017 at 5:05 AM, Junio C Hamano wrote: > That is too early to tell. At this point we only know there are me > who won't use it and you who will, among all the other people in the > world. We can probably make it useful with some extended efforts. I

Re: [PATCH] blame: add option to print tips (--tips)

2017-01-22 Thread Junio C Hamano
Edmundo Carmona Antoranz writes: > I wrote this for human consumption, actually. I see. > Maybe I overestimated the > need for this feature (I definitely find it handy, but it might be just me). That is too early to tell. At this point we only know there are me who

Re: [PATCH] blame: add option to print tips (--tips)

2017-01-22 Thread Edmundo Carmona Antoranz
On Sun, Jan 22, 2017 at 4:58 PM, Junio C Hamano wrote: > > What is the target audience? If you are trying to write a script > that reads output by "git blame", you are strongly discouraged > unless you are reading from "git blame --porcelain" which is more > compact and has

Re: [PATCH] blame: add option to print tips (--tips)

2017-01-22 Thread Junio C Hamano
Edmundo Carmona Antoranz writes: > So, this is a draft of what I mean by "adding tips to blame". > > Example output (sample from builtin/blame.c): > ... > 15:32 $ ./git blame --tips -L 1934,1960 builtin/blame.c >cee7f245dc: git-pickaxe: blame rewritten. > cee7f245dc

Re: [PATCH] blame: add option to print tips (--tips)

2017-01-22 Thread Edmundo Carmona Antoranz
Hello, everybody! So, this is a draft of what I mean by "adding tips to blame". Example output (sample from builtin/blame.c): 15:32 $ ./git blame --tips -L 1934,1960 builtin/blame.c cee7f245dc: git-pickaxe: blame rewritten. cee7f245dc builtin-pickaxe.c (Junio C Hamano