Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Thu, 5 Sep 2019 07:46:58 -0300
From:   Mauro Carvalho Chehab <mchehab@kernel.org>
To:     Joe Perches <joe@perches.com>
Cc:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Nishad Kamdar <nishadkamdar@gmail.com>,
        Uwe =?UTF-8?B?S2xlaW5lLUvDtm5pZw==?= 
        <u.kleine-koenig@pengutronix.de>, linux-media@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] media: siano: Use the correct style for SPDX License
 Identifier
Message-ID: <20190905074658.0f229587@coco.lan>
In-Reply-To: <f65bf8e7bc3234358d5956f94e3b4df81bc204e6.camel@perches.com>
References: <20190831151147.GA7082@nishad>
        <20190904153432.7fb54f02@coco.lan>
        <20190904183608.GA495@kroah.com>
        <20190904160010.4532c3f5@coco.lan>
        <20190905055430.GA23826@kroah.com>
        <20190905055614.7958918b@coco.lan>
        <f65bf8e7bc3234358d5956f94e3b4df81bc204e6.camel@perches.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Em Thu, 05 Sep 2019 02:28:05 -0700
Joe Perches <joe@perches.com> escreveu:

> On Thu, 2019-09-05 at 05:56 -0300, Mauro Carvalho Chehab wrote:
> > I'll accept
> > this patch and keep such rule in mind for next SPDX changes.
> > 
> > Anyway, with regards to script, we need to change the rules in order to
> > allow adding SPDX for python scripts, as otherwise the addition of SPDX
> > headers may cause regressions.  
> 
> Not really.
> 
> $ git grep -n '^# SPDX-License-Identifier' -- '*.py' | wc -l
> 62
> $ git grep -n '^# SPDX-License-Identifier' -- '*.py' | \
>   cut -f2 -d: | sort | uniq -c
>      32 1
>      27 2
>       3 3
> $ git grep -n '^# SPDX-License-Identifier' -- '*.py' | \
>   cut -f1 -d: | xargs ./scripts/spdxcheck.py --verbose
> 
> License files:               14
> Exception files:              2
> License IDs                  19
> Exception IDs                 2
> 
> Files checked:               62
> Lines checked:               95
> Files with SPDX:             62
> Files with errors:            0

Not quite sure what you're meant with the above, but:

1) this won't cover all python scripts, as there are some without
.py extension:

	$ git grep -lE "\#\!.*python"|grep -v \.py$
	drivers/staging/greybus/tools/lbtest
	scripts/bloat-o-meter
	scripts/diffconfig
	scripts/show_delta
	tools/hv/lsvmbus
	tools/kvm/kvm_stat/kvm_stat

2) we have already some files that are violating PEP-263 due to
SPDX headers. On a quick look at the results of:

	$ git grep -lE "^\#.*coding" $(git grep -lE "\#\!.*python")
	Documentation/sphinx/kernel_include.py
	Documentation/sphinx/rstFlatTable.py
	tools/perf/python/tracepoint.py
	tools/perf/python/twatch.py
	tools/power/x86/intel_pstate_tracer/intel_pstate_tracer.py

Those three files already have such problem:

	tools/perf/python/tracepoint.py
	tools/perf/python/twatch.py
	tools/power/x86/intel_pstate_tracer/intel_pstate_tracer.py

On those scripts, the coding line is not at the right place, and will be
silently ignored by python, causing them to crash with UTF-8 encoding
chars, if any.

And the other two doesn't have a SPDX header file. Adding a SPDX header
there would violate the "up to the second line" rule.

Thanks,
Mauro