From: Toon Claes <toon@iotcl•com>
To: Karthik Nayak <karthik.188@gmail•com>,
Patrick Steinhardt <ps@pks•im>,
Junio C Hamano <gitster@pobox•com>
Cc: git@vger•kernel.org, jltobler@gmail•com,
Christian Couder <chriscool@tuxfamily•org>
Subject: Re: [PATCH] blame: fix unblamable and ignored lines in porcelain mode
Date: Tue, 25 Mar 2025 09:45:17 +0100 [thread overview]
Message-ID: <875xjxwlb6.fsf@iotcl.com> (raw)
In-Reply-To: <CAOLa=ZTHb6ZFmpDpSwzUTiDs44gY-W3aqBooAMyORK8Xs3YQxQ@mail.gmail.com>
Karthik Nayak <karthik.188@gmail•com> 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 <gitster@pobox•com>
author-time 1333493588
author-tz -0700
committer Junio C Hamano
committer-mail <gitster@pobox•com>
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 <liu.denton@gmail•com>
author-time 1556526494
author-tz -0400
committer Junio C Hamano
committer-mail <gitster@pobox•com>
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 <gitster@pobox•com>
author-time 1333493588
author-tz -0700
committer Junio C Hamano
committer-mail <gitster@pobox•com>
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 <gitster@pobox•com>
author-time 1333493588
author-tz -0700
committer Junio C Hamano
committer-mail <gitster@pobox•com>
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
next prev parent reply other threads:[~2025-03-25 8:45 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-03-21 16:39 [PATCH] blame: fix unblamable and ignored lines in porcelain mode Karthik Nayak
2025-03-23 15:58 ` Junio C Hamano
2025-03-24 10:16 ` Patrick Steinhardt
2025-03-24 10:37 ` Toon Claes
2025-03-24 20:04 ` Karthik Nayak
2025-03-25 8:45 ` Toon Claes [this message]
2025-03-25 10:31 ` Karthik Nayak
2025-03-25 19:44 ` Junio C Hamano
2025-03-24 20:00 ` Karthik Nayak
2025-03-24 19:56 ` Karthik Nayak
2025-03-26 21:06 ` [PATCH v2] blame: print unblamable and ignored commits " Karthik Nayak
2025-03-26 22:49 ` Eric Sunshine
2025-03-27 11:07 ` Karthik Nayak
2025-03-29 19:06 ` Junio C Hamano
2025-03-28 7:00 ` Patrick Steinhardt
2025-03-29 10:26 ` Karthik Nayak
2025-03-29 18:21 ` [PATCH v3] " Karthik Nayak
2025-03-30 4:56 ` Junio C Hamano
2025-03-30 9:28 ` Phillip Wood
2025-03-30 20:43 ` [PATCH v4] " Karthik Nayak
2025-03-31 7:05 ` Patrick Steinhardt
2025-03-31 7:34 ` Karthik Nayak
2025-03-31 10:24 ` phillip.wood123
2025-03-31 10:47 ` Phillip Wood
[not found] ` <CAOLa=ZSQ7PiasRk23Hxp7Gk5vU-x83N4e4WTxG3eVsxK0zKnWA@mail.gmail.com>
[not found] ` <f39c6468-aade-489a-bc7b-c3d342a22cb8@gmail.com>
[not found] ` <CAOLa=ZQMYn2eYndX0saTKnuzAacjtNZeTb9PCrcNC50nneAq5g@mail.gmail.com>
2025-04-02 13:07 ` Phillip Wood
2025-04-03 16:03 ` [PATCH v5] " Karthik Nayak
2025-04-04 15:58 ` Phillip Wood
2025-04-08 0:32 ` 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=875xjxwlb6.fsf@iotcl.com \
--to=toon@iotcl$(echo .)com \
--cc=chriscool@tuxfamily$(echo .)org \
--cc=git@vger$(echo .)kernel.org \
--cc=gitster@pobox$(echo .)com \
--cc=jltobler@gmail$(echo .)com \
--cc=karthik.188@gmail$(echo .)com \
--cc=ps@pks$(echo .)im \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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