On Fri, 31 Jul 2020 00:06:23 +0200 Thomas Monjalon <tho...@monjalon.net> wrote:
> 15/07/2020 01:21, Stephen Hemminger: > > Simple script to look for drivers and scripts that > > are missing requires SPDX header. > > > > Signed-off-by: Stephen Hemminger <step...@networkplumber.org> > [...] > > +#! /bin/sh > > +# SPDX-License-Identifier: BSD-3-Clause > > +# Copyright 2020 Microsoft Corporation > > +# > > +# Produce a list of files with incorrect license tags > > + > > +errors=0 > > +warnings=0 > > +quiet=false > > +verbose=false > > + > > +print_usage () { > > + echo "usage: $(basename $0) [-q] [-v]" > > + exit 1 > > +} > > + > > +check_spdx() { > > + if $verbose; then > > + echo "Files without SPDX License" > > + echo "--------------------------" > > + fi > > + git grep -L SPDX-License-Identifier -- \ > > + ':^.git*' ':^.ci/*' ':^.travis.yml' \ > > + ':^README' ':^MAINTAINERS' ':^VERSION' ':^ABI_VERSION' \ > > + ':^*/Kbuild' ':^*/README' \ > > + ':^license/' ':^config/' ':^buildtools/' \ > > + ':^*.cocci' ':^*.abignore' \ > > + ':^*.def' ':^*.map' ':^*.ini' ':^*.data' ':^*.cfg' ':^*.txt' \ > > + ':^*.svg' ':^*.png'\ > > I don't agree with this list of files. > But I guess we can start with that and be more strict in future. I suppose README, MAINTAINERS, .ini and .txt files could be removed from the exception list.