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