On Mon, 2020-09-21 at 09:39 +0200, Lukas Bulwahn wrote: > On Sun, 20 Sep 2020, Joe Perches wrote: > > On Sun, 2020-09-20 at 21:52 +0530, Dwaipayan Ray wrote: [] > > > For example, Lukas checked commits between v5.4 and > > > v5.8 and he found: > > > 175 Missing Signed-off-by: line by nominal patch authorrep > > > 'Daniel Vetter <[email protected]>' > > > > > > Infact in all of those commits he signed off using a different > > > mail, Daniel Vetter <[email protected]>. > > > > > > So is it possible to resolve these using perhaps .mailmap > > > entries? Or should only the name mismatch part be better > > > handled? Or perhaps both? > > > > Dunno. It certainly can be improved... > > Try adding some more logic and see what you come up with. > > > > btw: > > > > The most frequent NO_AUTHOR_SIGN_OFF messages for v5.7..v5.8 are > > > > 98 WARNING: From:/SoB: email address mismatch: 'From: Daniel Vetter > > <[email protected]>' != 'Signed-off-by: Daniel Vetter > > <[email protected]>' > > 24 WARNING: From:/SoB: email address mismatch: 'From: Thinh Nguyen > > <[email protected]>' != 'Signed-off-by: Thinh Nguyen > > <[email protected]>' > > 19 WARNING: From:/SoB: email address mismatch: 'From: Wolfram Sang > > <[email protected]>' != 'Signed-off-by: Wolfram Sang > > <[email protected]>' > > 11 WARNING: From:/SoB: email address mismatch: 'From: Luke Nelson > > <[email protected]>' != 'Signed-off-by: Luke Nelson > > <[email protected]>' > > 8 WARNING: From:/SoB: email address mismatch: 'From: Christophe Leroy > > <[email protected]>' != 'Signed-off-by: Christophe Leroy > > <[email protected]>' > > 6 WARNING: From:/SoB: email address mismatch: 'From: Davidlohr Bueso > > <[email protected]>' != 'Signed-off-by: Davidlohr Bueso <[email protected]>' > > 5 WARNING: Missing Signed-off-by: line by nominal patch author '"Paul > > A. Clarke" <[email protected]>' > > 4 WARNING: Missing Signed-off-by: line by nominal patch author 'Huang > > Ying <[email protected]>' > > 3 WARNING: Missing Signed-off-by: line by nominal patch author > > '"Stephan Müller" <[email protected]>' [] > I hope Dwaipayan can come up with a '.AUTHOR_SIGN_OFF.mailmap' file that > we can use to distinguish the known developers that knowingly and > intentionally use different identities vs. the 'newbies' that should > validly be warned.
I hope not. One .mailmap should be enough.
