Re: [dpdk-dev] [PATCH v4] devtools: add new SPDX license compliance checker

2020-07-14 Thread Stephen Hemminger
On Fri, 12 Jun 2020 11:05:27 +0200 Gaëtan Rivet wrote: > On 24/02/20 13:01 -0800, Stephen Hemminger wrote: > > Simple script to look for drivers and scripts that > > are missing requires SPDX header. > > > > Update the contribution guidelines to indicate that SPDX license > > identfier is requir

Re: [dpdk-dev] [PATCH v4] devtools: add new SPDX license compliance checker

2020-06-12 Thread Thomas Monjalon
12/06/2020 16:53, Stephen Hemminger: > On Thu, 11 Jun 2020 23:39:55 +0200 > Thomas Monjalon wrote: > > > 24/02/2020 22:01, Stephen Hemminger: > > > +tmpfile=$(mktemp) > > > > Please check how other temp files are created in other scripts > > for consitency. > > > > > +git grep -L SPDX-Lic

Re: [dpdk-dev] [PATCH v4] devtools: add new SPDX license compliance checker

2020-06-12 Thread Stephen Hemminger
On Thu, 11 Jun 2020 23:39:55 +0200 Thomas Monjalon wrote: > 24/02/2020 22:01, Stephen Hemminger: > > +tmpfile=$(mktemp) > > Please check how other temp files are created in other scripts > for consitency. > > > +git grep -L SPDX-License-Identifier -- \ > > + ':^.git*' ':^.ci/*' ':^.trav

Re: [dpdk-dev] [PATCH v4] devtools: add new SPDX license compliance checker

2020-06-12 Thread Gaëtan Rivet
On 24/02/20 13:01 -0800, Stephen Hemminger wrote: > Simple script to look for drivers and scripts that > are missing requires SPDX header. > > Update the contribution guidelines to indicate that SPDX license > identfier is required for this project. > > Signed-off-by: Stephen Hemminger > --- > v

Re: [dpdk-dev] [PATCH v4] devtools: add new SPDX license compliance checker

2020-06-12 Thread Gaëtan Rivet
Hi Thomas, Stephen, On 11/06/20 23:39 +0200, Thomas Monjalon wrote: > 24/02/2020 22:01, Stephen Hemminger: > > +tmpfile=$(mktemp) > > Please check how other temp files are created in other scripts > for consitency. > > > +git grep -L SPDX-License-Identifier -- \ > > + ':^.git*' ':^.ci/*' '

Re: [dpdk-dev] [PATCH v4] devtools: add new SPDX license compliance checker

2020-06-11 Thread Thomas Monjalon
24/02/2020 22:01, Stephen Hemminger: > +tmpfile=$(mktemp) Please check how other temp files are created in other scripts for consitency. > +git grep -L SPDX-License-Identifier -- \ > + ':^.git*' ':^.ci/*' ':^.travis.yml' \ > + ':^README' ':^MAINTAINERS' ':^VERSION' ':^ABI_VERSION' \ >

Re: [dpdk-dev] [PATCH v4] devtools: add new SPDX license compliance checker

2020-06-11 Thread Thomas Monjalon
11/06/2020 20:46, Stephen Hemminger: > On Mon, 24 Feb 2020 13:01:30 -0800 > Stephen Hemminger wrote: > > > Simple script to look for drivers and scripts that > > are missing requires SPDX header. > > > > Update the contribution guidelines to indicate that SPDX license > > identfier is required f

Re: [dpdk-dev] [PATCH v4] devtools: add new SPDX license compliance checker

2020-06-11 Thread Stephen Hemminger
On Mon, 24 Feb 2020 13:01:30 -0800 Stephen Hemminger wrote: > Simple script to look for drivers and scripts that > are missing requires SPDX header. > > Update the contribution guidelines to indicate that SPDX license > identfier is required for this project. > > Signed-off-by: Stephen Hemminge

Re: [dpdk-dev] [PATCH v4] devtools: add new SPDX license compliance checker

2020-04-28 Thread Stephen Hemminger
On Mon, 24 Feb 2020 13:01:30 -0800 Stephen Hemminger wrote: > Simple script to look for drivers and scripts that > are missing requires SPDX header. > > Update the contribution guidelines to indicate that SPDX license > identfier is required for this project. > > Signed-off-by: Stephen Hemminge

[dpdk-dev] [PATCH v4] devtools: add new SPDX license compliance checker

2020-02-24 Thread Stephen Hemminger
Simple script to look for drivers and scripts that are missing requires SPDX header. Update the contribution guidelines to indicate that SPDX license identfier is required for this project. Signed-off-by: Stephen Hemminger --- v4 - add MAINTAINERS entry update coding style document cha