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 <[email protected]> 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 <[email protected]> > > > Cc: Joonas Lahtinen <[email protected]> > > > Signed-off-by: Rodrigo Vivi <[email protected]> > > > > Reviewed-by: Jani Nikula <[email protected]> > > 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. I really don't want to send mail from @intel.com to anywhere near public stuff. -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 > [email protected] > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
