qemu-devel
[Top][All Lists]
Advanced

[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 23:14:00 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.2.0

On 11/4/21 16:17, Taylor Simpson wrote:
>> -----Original Message-----
>> From: Philippe Mathieu-Daudé <philippe.mathieu.daude@gmail.com> On
>> Behalf Of Philippe Mathieu-Daudé
>> Sent: Thursday, November 4, 2021 5:02 AM
>> To: Daniel P. Berrangé <berrange@redhat.com>
>> Cc: Taylor Simpson <tsimpson@quicinc.com>; Richard Henderson
>> <richard.henderson@linaro.org>; qemu-devel@nongnu.org; Laurent Vivier
>> <laurent@vivier.eu>
>> Subject: Re: [PATCH] linux-user: Add GPL-2.0+ SPDX license identifier
>>
>> On 11/4/21 10:06, Daniel P. Berrangé wrote:
>>>
>>> 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 seems like more of a question for a lawyer to review than technical 
> reviewers.
> 
> I remember discussing this when I made the initial submission.  At that time, 
> there wasn't a broad policy on this.  Has that changed?

On the policy side, nothing changed.

In an earlier version I posted I was replacing the license text by the
equivalent SPDX tag, but I've been told I can not *remove* a license
text without an Ack from all the contributors who modified a file;
however simply *adding* (although duplicating the information) would
be OK.
This is what I'm doing here (what 'technically' changed from earlier
patch).



reply via email to

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