Re: [PATCH] blame: fix unblamable and ignored lines in porcelain mode
From: Toon Claes <hidden>
Date: 2025-03-25 08:45:34
Possibly related (same subject, not in this thread)
- 2025-03-25 · Re: [PATCH] blame: fix unblamable and ignored lines in porcelain mode · Junio C Hamano <hidden>
- 2025-03-21 · [PATCH] blame: fix unblamable and ignored lines in porcelain mode · Karthik Nayak <hidden>
Karthik Nayak [off-list ref] writes:
I'm curious, how would it be different, if they blame down to the same commit? My understanding was "unblamable" and "ignored" are tied to commits.
Let me include an example, let's blame `varint.h`.
First have a look at the non-porcelain format:
$ git blame varint.h -l
d2c1898571a6a2324593e92163e8754880e0c1fb (Junio C Hamano 2012-04-03 15:53:08 -0700 1) #ifndef VARINT_H
d2c1898571a6a2324593e92163e8754880e0c1fb (Junio C Hamano 2012-04-03 15:53:08 -0700 2) #define VARINT_H
d2c1898571a6a2324593e92163e8754880e0c1fb (Junio C Hamano 2012-04-03 15:53:08 -0700 3)
554544276a604c144df45efcb060c80aa322088c (Denton Liu 2019-04-29 04:28:14 -0400 4) int encode_varint(uintmax_t, unsigned char *);
554544276a604c144df45efcb060c80aa322088c (Denton Liu 2019-04-29 04:28:14 -0400 5) uintmax_t decode_varint(const unsigned char **);
d2c1898571a6a2324593e92163e8754880e0c1fb (Junio C Hamano 2012-04-03 15:53:08 -0700 6)
d2c1898571a6a2324593e92163e8754880e0c1fb (Junio C Hamano 2012-04-03 15:53:08 -0700 7) #endif /* VARINT_H */
Now if we put `554544276a604c144df45efcb060c80aa322088c` in `.git-blame-ignore-revs`:
$ git -c blame.markUnblamableLines=true -c blame.markIgnoredLines=true blame varint.h --ignore-revs-file .git-blame-ignore-revs -l
d2c1898571a6a2324593e92163e8754880e0c1fb (Junio C Hamano 2012-04-03 15:53:08 -0700 1) #ifndef VARINT_H
d2c1898571a6a2324593e92163e8754880e0c1fb (Junio C Hamano 2012-04-03 15:53:08 -0700 2) #define VARINT_H
d2c1898571a6a2324593e92163e8754880e0c1fb (Junio C Hamano 2012-04-03 15:53:08 -0700 3)
?d2c1898571a6a2324593e92163e8754880e0c1f (Junio C Hamano 2012-04-03 15:53:08 -0700 4) int encode_varint(uintmax_t, unsigned char *);
?d2c1898571a6a2324593e92163e8754880e0c1f (Junio C Hamano 2012-04-03 15:53:08 -0700 5) uintmax_t decode_varint(const unsigned char **);
d2c1898571a6a2324593e92163e8754880e0c1fb (Junio C Hamano 2012-04-03 15:53:08 -0700 6)
d2c1898571a6a2324593e92163e8754880e0c1fb (Junio C Hamano 2012-04-03 15:53:08 -0700 7) #endif /* VARINT_H */
If we compare that to the porcelain format:
$ git blame varint.h -l --porcelain
d2c1898571a6a2324593e92163e8754880e0c1fb 1 1 3
author Junio C Hamano
author-mail [off-list ref]
author-time 1333493588
author-tz -0700
committer Junio C Hamano
committer-mail [off-list ref]
committer-time 1333495484
committer-tz -0700
summary varint: make it available outside the context of pack
filename varint.h
#ifndef VARINT_H
d2c1898571a6a2324593e92163e8754880e0c1fb 2 2
#define VARINT_H
d2c1898571a6a2324593e92163e8754880e0c1fb 3 3
554544276a604c144df45efcb060c80aa322088c 4 4 2
author Denton Liu
author-mail [off-list ref]
author-time 1556526494
author-tz -0400
committer Junio C Hamano
committer-mail [off-list ref]
committer-time 1557037206
committer-tz +0900
summary *.[ch]: remove extern from function declarations using spatch
previous ffac537e6cbbf934b08745a378932722df287a53 varint.h
filename varint.h
int encode_varint(uintmax_t, unsigned char *);
554544276a604c144df45efcb060c80aa322088c 5 5
uintmax_t decode_varint(const unsigned char **);
d2c1898571a6a2324593e92163e8754880e0c1fb 8 6 2
d2c1898571a6a2324593e92163e8754880e0c1fb 9 7
#endif /* VARINT_H */
And now with the `.git-blame-ignore-revs` file:
$ git -c blame.markUnblamableLines=true -c blame.markIgnoredLines=true blame varint.h --ignore-revs-file .git-blame-ignore-revs -l --porcelain
d2c1898571a6a2324593e92163e8754880e0c1fb 1 1 3
author Junio C Hamano
author-mail [off-list ref]
author-time 1333493588
author-tz -0700
committer Junio C Hamano
committer-mail [off-list ref]
committer-time 1333495484
committer-tz -0700
summary varint: make it available outside the context of pack
filename varint.h
#ifndef VARINT_H
d2c1898571a6a2324593e92163e8754880e0c1fb 2 2
#define VARINT_H
d2c1898571a6a2324593e92163e8754880e0c1fb 3 3
d2c1898571a6a2324593e92163e8754880e0c1fb 6 4 2
int encode_varint(uintmax_t, unsigned char *);
d2c1898571a6a2324593e92163e8754880e0c1fb 7 5
uintmax_t decode_varint(const unsigned char **);
d2c1898571a6a2324593e92163e8754880e0c1fb 8 6 2
d2c1898571a6a2324593e92163e8754880e0c1fb 9 7
#endif /* VARINT_H */
So every line now blames down to commit
d2c1898571a6a2324593e92163e8754880e0c1fb. The lines which used to
blame down to 554544276a604c144df45efcb060c80aa322088c should be marked
as "ignored", but we only emit the details once for each commit. The
commit details (author, committer) are only relevant once, but the
"ignored" info can differ for each line (as you also can see in the
non-porcelain format).
We could make the output look something like:
$ git -c blame.markUnblamableLines=true -c blame.markIgnoredLines=true blame varint.h --ignore-revs-file .git-blame-ignore-revs -l --porcelain
d2c1898571a6a2324593e92163e8754880e0c1fb 1 1 3
author Junio C Hamano
author-mail [off-list ref]
author-time 1333493588
author-tz -0700
committer Junio C Hamano
committer-mail [off-list ref]
committer-time 1333495484
committer-tz -0700
summary varint: make it available outside the context of pack
filename varint.h
#ifndef VARINT_H
d2c1898571a6a2324593e92163e8754880e0c1fb 2 2
#define VARINT_H
d2c1898571a6a2324593e92163e8754880e0c1fb 3 3
d2c1898571a6a2324593e92163e8754880e0c1fb 6 4 2
ignored
int encode_varint(uintmax_t, unsigned char *);
d2c1898571a6a2324593e92163e8754880e0c1fb 7 5
ignored
uintmax_t decode_varint(const unsigned char **);
d2c1898571a6a2324593e92163e8754880e0c1fb 8 6 2
d2c1898571a6a2324593e92163e8754880e0c1fb 9 7
#endif /* VARINT_H */
It feels odd to me only the "ignored" info is emitted and the rest
of the details isn't. But that might be just me...
--
Toon