Re: get_maintainer.pl bug? (was: Re: [PATCH 0/8] linker-section array fix and clean ups)

From: Johan Hovold
Date: Wed Nov 04 2020 - 10:31:49 EST


On Wed, Nov 04, 2020 at 04:04:00AM -0800, Joe Perches wrote:
> On Wed, 2020-11-04 at 10:16 +0100, Johan Hovold wrote:
> > Running scrips/get_maintainer.pl on this series [1] gave the wrong
> > address for Nick Desaulniers:
> >
> > Nick Desaulniers <ndesaulniers@xxxxxxxxxxx> (commit_signer:1/2=50%,commit_signer:1/8=12%)
> >
> > It seems he recently misspelled his address in a reviewed-by tag to
> > commit 33def8498fdd ("treewide: Convert macro and uses of __section(foo)
> > to __section("foo")") and that is now being picked up by the script.
> >
> > I guess that's to be considered a bug?
>
> No, that's a feature. If it's _really_ a problem, (and I don't
> think it really is), that's what .mailmap is for.

Ah, Nick doesn't actually have any commits touching these files; I was
confused by the "commit_signer" in the script output and didn't expect
Reviewed-by tags to be considered all (and at least not over SoB).

Hmm. Guess it's working as intended.

Johan