Re: [PATCH v4] checkpatch.pl: Add SPDX license tag check

2017-12-21 Thread Joe Perches
On Thu, 2017-12-21 at 11:04 -0600, Rob Herring wrote: > Okay, here's what I've ended up with: > > if ($realline == $checklicenseline) { > if ($realfile =~ /\.(?:sh|pl|py)/ && $rawline =~ /\[ > \+]\s*\!\#/) { > $checklicenseline = 2

Re: [PATCH v4] checkpatch.pl: Add SPDX license tag check

2017-12-21 Thread Rob Herring
On Wed, Dec 20, 2017 at 10:28:48PM -0800, Joe Perches wrote: > On Wed, 2017-12-20 at 17:46 -0600, Rob Herring wrote: > > Add SPDX license tag check based on the rules defined in > > Documentation/process/license-rules.rst. To summarize, SPDX license tags > > should be on the 1st line (or 2nd line i

Re: [PATCH v4] checkpatch.pl: Add SPDX license tag check

2017-12-20 Thread Philippe Ombredanne
Rob, On Thu, Dec 21, 2017 at 12:46 AM, Rob Herring wrote: > Add SPDX license tag check based on the rules defined in > Documentation/process/license-rules.rst. To summarize, SPDX license tags > should be on the 1st line (or 2nd line in scripts) using the appropriate > comment style for the file t

Re: [PATCH v4] checkpatch.pl: Add SPDX license tag check

2017-12-20 Thread Joe Perches
On Wed, 2017-12-20 at 17:46 -0600, Rob Herring wrote: > Add SPDX license tag check based on the rules defined in > Documentation/process/license-rules.rst. To summarize, SPDX license tags > should be on the 1st line (or 2nd line in scripts) using the appropriate > comment style for the file type. >

[PATCH v4] checkpatch.pl: Add SPDX license tag check

2017-12-20 Thread Rob Herring
Add SPDX license tag check based on the rules defined in Documentation/process/license-rules.rst. To summarize, SPDX license tags should be on the 1st line (or 2nd line in scripts) using the appropriate comment style for the file type. Cc: Andy Whitcroft Cc: Joe Perches Cc: Greg Kroah-Hartman C