Re: Validating MAINTAINERS entries?

[Rob Herring <robherring2@gmail.com>]

On Wed, Aug 10, 2022 at 8:01 AM Mark Brown <broonie@kernel.org> wrote:
>
> On Wed, Aug 10, 2022 at 07:54:59AM -0600, Rob Herring wrote:
> > On Wed, Aug 10, 2022 at 7:25 AM Lee Jones <lee@kernel.org> wrote:
> > > On Wed, 10 Aug 2022, Mark Brown wrote:
>
> > > > > Education is not the answer.
>
> > Yes. I'm convinced there is no way to solve these problems on the
> > sender side. I see plenty of cases of not running get_maintainers.pl.
>
> We can't solve problems, but we can make things a bit better.
>
> > You've got to filter out what you want on your end. And lei is great
> > for that.
>
> lei is too new for Debian stable :/

Meaning it's not packaged or needs newer dependencies?

> > >   --git-min-percent => minimum percentage of commits required (default: 5)
>
> > > 5 is not a lot of percent for seldomly touched source files.
>
> > Send a patch. I would also bump --git-min-signatures from 1.
>
> Will that do the right thing in cases like recently added files?

Yes, because I'm sure checkpatch.pl was run on the patch adding the
files and it tells you 'added file, need a MAINTAINERS entry?'.

Really, I would (and do) turn off gitfallback completely by default.
If someone wants to be CCed and there's not a MAINTAINERS entry, then
that's on them.

Rob