Re: [PATCH] headers: Remove some left-over license text in include/uapi/rdma/

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

 



Le 26/09/2022 à 20:21, Jason Gunthorpe a écrit :
On Sun, Sep 25, 2022 at 03:39:58PM +0200, Christophe JAILLET wrote:
There is already a SPDX-License-Identifier tag, so the corresponding
license text can be removed.

Signed-off-by: Christophe JAILLET <christophe.jaillet-39ZsbGIQGT5GWvitb5QawA@xxxxxxxxxxxxxxxx>
---
Changes in the 2 files under rdma/hfi/ is a bit more than just removing
the license text. There were some Copyright(c) for GPL and BSD licence.
I have simplified it to what look logical to me.
But in case it matters, review with care.

Did you use a script or something to verify that the text being
removed is word for word identical to the text in LICENSES/ ?

Hi,

Short answer:
============

You are likely right, and patches for BSD like license should certainly be left as-is to keep as-is.



Longer answer:
=============

I spot potential candidate with:
grep -r --include=*.[ch] -A50 SPDX | grep -v SPDX | grep -v staging | grep -v usr\/include\/ | grep -C10 License

Then I manually check that the included license is consistent with the SPDX-License-Identifier tag.

If they don't match, I propose to change the SPDX tag (see [1] for an example)

My grep is not optimal. Looking for GNU or GPL or BSD or... also gives some decent results. But all what is done is done.


Honestly, I don't look word by word the text itself.
My point is that if a file states that it is usable under a given license, then it should use the term of this license. Otherwise it looks pointless to refer to it, no?

Moreover, in the BSD license, some parts of the text can be adjusted to the context. (in red in [3]

In "bnxt_re-abi.h", the BSD-2-Clause "THE COPYRIGHT HOLDER OR CONTRIBUTORS" has been replaced by "THE AUTHOR AND CONTRIBUTORS".


Giving a 2nd look at it (see [4]), when boilerplate have been removed, it was mostly (only?) for GPL-2, which don't have such "red" parts in the license text.


So you are likely right, and patches for BSD like license should certainly be left as-is to keep this "red places" as-is.


I had understood this was the reason the original SPDX conversion left
the license text around?

I don't know. It looks to me that Greg has done some mass update, especially in the uapi directory, only adding the SPDX-License-Identifier tag.
See [2].


It looks OK to me, but I didn't check it word for word :)

Is there a reason this series is coming up now around the uapi
headers? Are we doing them all or something?

No specific reason for it to happen now.

I chose uapi directory because it is .h files. This is my own contribution to the Ingo Molnar's "Fast Kernel Headers".
Smaller header files, faster compilation :)
(even if my clean-ups should be just noise in the overall building time)

I try to send patches in small hunks of related files. This avoids the "please split the patch in a serie by maintainer" answer.

I've send ~10 or 15 patches yet. I'll see the enthusiasms (or lack of) for such patches before continuing.


Jason


CJ

[1]: https://lore.kernel.org/all/a57505bf6c8ddbd68ff5a320aaf5922ec431e00e.1664117249.git.christophe.jaillet@xxxxxxxxxx/

[2]: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=e2be04c7f9958dde770eeb8b30e829ca969b37bb

[3]: https://spdx.org/licenses/BSD-2-Clause.html

[4]: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/log/?qt=grep&q=boilerplate%2Freference



[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux