[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] linux-user: Add GPL-2.0+ SPDX license identifier

From: Philippe Mathieu-Daudé
Subject: Re: [PATCH] linux-user: Add GPL-2.0+ SPDX license identifier
Date: Thu, 4 Nov 2021 11:02:00 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.2.0

On 11/4/21 10:06, Daniel P. Berrangé wrote:
> On Wed, Nov 03, 2021 at 09:31:46PM +0100, Philippe Mathieu-Daudé wrote:
>> Use Qualcomm License Identifier tool [*] to classify linux-user/
>> files per license type, filtering those matching at least 60% the
>> "GPLv2 or later" type, and manually add the GPL-2.0-or-later SPDX
>> comment.
> When you say  "manually add", can you confirm that you explicitly
> inspected every license header yourself to validate that the reported
> match truely is GPL-2.0+, before adding the SPDX tag.
> Adding licenses tags based merely on a 60% match, without doing
> human inspection would be questionable.

Yes I explicitly inspected every license error before manually add
the tag, but I am human, so prone to errors, which is why peer
review is important :)

This diff is generated using 'git-format-patch --unified=4' because
I thought it would ease review, however I just realize reviewers can't
check from the diff whether there is another license in the file.

In v2 I moved the tag to the first line and generated the diff using
'git-format-patch --unified=20' which is better because in almost all
cases the full header is displayed. I might use '--function-context'
in eventual v3, but it is slightly more verbose (it displays everything
until the first function, so all #include / #define after the header).

reply via email to

[Prev in Thread] Current Thread [Next in Thread]