Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 27 Jan 2023 14:59:00 -0500
From:      Ed Maste <emaste@freebsd.org>
To:        "Bjoern A. Zeeb" <bz@freebsd.org>
Cc:        src-committers@freebsd.org, dev-commits-src-all@freebsd.org,  dev-commits-src-main@freebsd.org
Subject:   Re: git: 247e03ed8b83 - main - Add git-blame ignore file
Message-ID:  <CAPyFy2Dt%2Bw5nwyeGvkgqc7xi3SONiyetbe6f4-RCiJ61rsX=9A@mail.gmail.com>
In-Reply-To: <1o40rq5-n1pr-ss27-q14-qo845q26398@mnoonqbm.arg>
References:  <202301232027.30NKRckV060051@gitrepo.freebsd.org> <1o40rq5-n1pr-ss27-q14-qo845q26398@mnoonqbm.arg>

next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, 27 Jan 2023 at 10:24, Bjoern A. Zeeb <bz@freebsd.org> wrote:
>
> On Mon, 23 Jan 2023, Ed Maste wrote:
>
> > The branch main has been updated by emaste:
> >
> > URL: https://cgit.FreeBSD.org/src/commit/?id=247e03ed8b8341cf25dc3d9ecb846612d984f424
> >
> > commit 247e03ed8b8341cf25dc3d9ecb846612d984f424
> > Author:     Ed Maste <emaste@FreeBSD.org>
> > AuthorDate: 2023-01-23 20:27:25 +0000
> > Commit:     Ed Maste <emaste@FreeBSD.org>
> > CommitDate: 2023-01-23 20:27:25 +0000
> >
> >    Add git-blame ignore file
> >
> >    .git-blame-ignore-revs lists commit hashes that should be skipped by
> >    `git blame` e.g. non-functional whitespace or style cleanup.
> >
> >    The file is populated with a few sample entries.
>
> How is this file to be used?
>
> 1. Do a non-functional commit?
> 2. And then do another commit to add the hash?

Yes, although there's no need to list every non-functional change
here. It's useful mainly for changes that would otherwise obscure the
history/blame - for example, re-indenting libexec/rtld-elf/rtld.c to
use standard indentation would be a good use.

We could omit the comments, but I don't think it's worth the effort to
try to automate this.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAPyFy2Dt%2Bw5nwyeGvkgqc7xi3SONiyetbe6f4-RCiJ61rsX=9A>