Re: [PATCH] checkpatch: extend author Signed-off-by check for split From: header

2020-09-19 Thread Joe Perches
On Sun, 2020-09-20 at 01:08 +0530, Dwaipayan Ray wrote: > On Sun, Sep 20, 2020 at 12:06 AM Lukas Bulwahn > wrote: > > On Sat, 19 Sep 2020, Joe Perches wrote: > > > On Sat, 2020-09-19 at 20:12 +0200, Lukas Bulwahn wrote: > > > > On Sat, 19 Sep 2020, Joe Perches wrote: > > > > > On Sat, 2020-09-19

Re: [PATCH] checkpatch: extend author Signed-off-by check for split From: header

2020-09-19 Thread Dwaipayan Ray
On Sun, Sep 20, 2020 at 12:06 AM Lukas Bulwahn wrote: > > > > On Sat, 19 Sep 2020, Joe Perches wrote: > > > On Sat, 2020-09-19 at 20:12 +0200, Lukas Bulwahn wrote: > > > > > > On Sat, 19 Sep 2020, Joe Perches wrote: > > > > > > > On Sat, 2020-09-19 at 13:42 +0530, Dwaipayan Ray wrote: > > > > > Ch

Re: [PATCH] checkpatch: extend author Signed-off-by check for split From: header

2020-09-19 Thread Lukas Bulwahn
On Sat, 19 Sep 2020, Joe Perches wrote: > On Sat, 2020-09-19 at 20:12 +0200, Lukas Bulwahn wrote: > > > > On Sat, 19 Sep 2020, Joe Perches wrote: > > > > > On Sat, 2020-09-19 at 13:42 +0530, Dwaipayan Ray wrote: > > > > Checkpatch did not handle cases where the author From: header > > > > was

Re: [PATCH] checkpatch: extend author Signed-off-by check for split From: header

2020-09-19 Thread Joe Perches
On Sat, 2020-09-19 at 20:12 +0200, Lukas Bulwahn wrote: > > On Sat, 19 Sep 2020, Joe Perches wrote: > > > On Sat, 2020-09-19 at 13:42 +0530, Dwaipayan Ray wrote: > > > Checkpatch did not handle cases where the author From: header > > > was split into two lines. The author string went empty and >

Re: [PATCH] checkpatch: extend author Signed-off-by check for split From: header

2020-09-19 Thread Lukas Bulwahn
On Sat, 19 Sep 2020, Joe Perches wrote: > On Sat, 2020-09-19 at 13:42 +0530, Dwaipayan Ray wrote: > > Checkpatch did not handle cases where the author From: header > > was split into two lines. The author string went empty and > > checkpatch generated a false NO_AUTHOR_SIGN_OFF warning. > > It

Re: [PATCH] checkpatch: extend author Signed-off-by check for split From: header

2020-09-19 Thread Joe Perches
On Sat, 2020-09-19 at 13:42 +0530, Dwaipayan Ray wrote: > Checkpatch did not handle cases where the author From: header > was split into two lines. The author string went empty and > checkpatch generated a false NO_AUTHOR_SIGN_OFF warning. It's good to provide an example where the current code doe

[PATCH] checkpatch: extend author Signed-off-by check for split From: header

2020-09-19 Thread Dwaipayan Ray
Checkpatch did not handle cases where the author From: header was split into two lines. The author string went empty and checkpatch generated a false NO_AUTHOR_SIGN_OFF warning. Support split From: headers in AUTHOR_SIGN_OFF check by adding an additional clause to resolve author identity in such c