Re: git: aa2539679084 - main - framework: cleanup conditional-indentations in Mk/

From: Alexey Dokuchaev <danfe_at_freebsd.org>
Date: Sun, 24 Apr 2022 19:13:51 UTC
On Sun, Apr 24, 2022 at 07:51:33PM +0200, Adriaan de Groot wrote:
> On Sunday, 24 April 2022 12:07:00 CEST Tobias C. Berner wrote:
> > The branch main has been updated by tcberner:
> > 
> > commit aa2539679084872cd84112e9df6bfee571570623
> > 
> >     framework: cleanup conditional-indentations in Mk/
> > 
> >     These white space changes contribute greatly to the readability of those
> > files. As we have a version control system, finding out the reasons for the
> > changes prior to these white space changes is still easily possible
> 
> This is a nice moment to point at "git blame" -- which will tell you when a
> line was last changed, and by whom. Naive use of "git blame" will now point a
> lot of lines at tcberner@, even though it was just whitespace. You can tell
> "git blame" to ignore certain revisions, and so discover the original authors
> before the white-space changes.

Or one can just ask for `git blame' just before that revision:

$ git blame aa25396790848^ Mk/bsd.sites.mk # (notice the caret)

> Actually, "blame" is kind of an unfriendly name. We should be cheering for
> the people who create the software we use daily!

Sadly, most of the times I have to issue `git blame' is not to cheer about
a particular change, but, really, blame.  FWIW, Subversion has `svn praise'
which is an alias for `svn blame'.

./danfe