[Intel-gfx] [PATCH] dim: Accept author x signed-off based on email.

[Rodrigo Vivi]

On Tue, Sep 26, 2017 at 01:24:35PM +0000, Jani Nikula wrote:
> On Tue, 26 Sep 2017, Daniel Vetter <daniel at ffwll.ch> wrote:
> > On Thu, Sep 21, 2017 at 06:27:28AM -0700, Rodrigo Vivi wrote:
> >> On Thu, Sep 21, 2017 at 11:12:52AM +0000, Jani Nikula wrote:
> >> > On Wed, 20 Sep 2017, Rodrigo Vivi <rodrigo.vivi at intel.com> wrote:
> >> > > It seems Patchwork or SMTP servers are messing some patches
> >> > > and changing the original git's author name on git per "Last, First".
> >> > > So we end up with a mismatch were signed-off uses one name format
> >> > > and author is using another format.
> >> > >
> >> > > So, let's check for email addresses instead.
> >> > >
> >> > > v2: Avoid useles warning and only check for email.
> >> > >
> >> > > Cc: Jani Nikula <jani.nikula at intel.com>
> >> > > Cc: Joonas Lahtinen <joonas.lahtinen at linux.intel.com>
> >> > > Signed-off-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
> >> > 
> >> > Reviewed-by: Jani Nikula <jani.nikula at intel.com>
> >> 
> >> pushed, thanks.
> >
> > Yeah except this breaks funny people like me who use different mails for
> > stuff. My idea with fixing this was that we parse the mail name and check
> > for both "Last, First" and "First Last" to work around the outlook crap.

The name parsing with last and first can be really nasty and end up having other
corner cases soon.

Like, my legal name in us is still:
de Oliveira Vivi, Rodrigo
not
Vivi, Rodrigo de Oliveira

I can imagine many other possible combinations with this and other names.

> >
> > I really don't want to send mail from @intel.com to anywhere near public
> > stuff.

I'm not sure if it is outlook fault honestly. On my patches where this funny
mess happenend I checked my email on the mailing list and original and everything
seemed sane there. The only place where Last, First started was post pwclient.

> 
> Ok, I'm fine with reverting, and finding another way.

I'm fine with the revert as well, but before reverting let me propose one thing
that is name_match || email_match so we cover both cases.

What do you think?

> 
> BR,
> Jani.
> 
> > -Daniel
> >
> >> 
> >> > 
> >> > > ---
> >> > >  dim | 2 +-
> >> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> >> > >
> >> > > diff --git a/dim b/dim
> >> > > index dbaeb1ec944d..a63000fb67a8 100755
> >> > > --- a/dim
> >> > > +++ b/dim
> >> > > @@ -689,7 +689,7 @@ function checkpatch_commit_push
> >> > >  	sha1=$1
> >> > >  
> >> > >  	# use real names for people with many different email addresses
> >> > > -	author=$(git show -s $sha1 --format="format:%an")
> >> > > +	author=$(git show -s $sha1 --format="format:%ae")
> >> > >  	committer=$(git show -s $sha1 --format="format:%cn")
> >> > >  
> >> > >  	# check for author sign-off
> >> > 
> >> > -- 
> >> > Jani Nikula, Intel Open Source Technology Center
> >> _______________________________________________
> >> Intel-gfx mailing list
> >> Intel-gfx at lists.freedesktop.org
> >> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
> 
> -- 
> Jani Nikula, Intel Open Source Technology Center