* [PATCH 0/2] LICENSES improvements/corrections @ 2023-05-05 13:05 Andrew Cooper 2023-05-05 13:05 ` [PATCH 1/2] LICENSES: Improve the legibility of these files Andrew Cooper 2023-05-05 13:05 ` [PATCH 2/2] LICENSES: Remove the use of deprecated LGPL SPDX tags Andrew Cooper 0 siblings, 2 replies; 11+ messages in thread From: Andrew Cooper @ 2023-05-05 13:05 UTC (permalink / raw) To: Xen-devel Cc: Andrew Cooper, George Dunlap, Jan Beulich, Stefano Stabellini, Wei Liu, Julien Grall Noticed in light of the recent LGPL changes to libacpi, but sadly only after the fact. Andrew Cooper (2): LICENSES: Improve the legibility of these files LICENSES: Remove the use of deprecated LGPL SPDX tags LICENSES/BSD-2-Clause | 4 ++++ LICENSES/BSD-3-Clause | 4 ++++ LICENSES/BSD-3-Clause-Clear | 4 ++++ LICENSES/CC-BY-4.0 | 5 +++++ LICENSES/GPL-2.0 | 17 +++++++++++------ LICENSES/LGPL-2.0 | 14 ++++++++++---- LICENSES/LGPL-2.1 | 14 ++++++++++---- LICENSES/MIT | 4 ++++ tools/libacpi/Makefile | 2 +- tools/libacpi/acpi2_0.h | 2 +- tools/libacpi/build.c | 2 +- tools/libacpi/dsdt.asl | 2 +- tools/libacpi/dsdt_acpi_info.asl | 2 +- tools/libacpi/libacpi.h | 2 +- tools/libacpi/mk_dsdt.c | 2 +- tools/libacpi/ssdt_laptop_slate.asl | 2 +- tools/libacpi/ssdt_pm.asl | 2 +- tools/libacpi/ssdt_s3.asl | 2 +- tools/libacpi/ssdt_s4.asl | 2 +- tools/libacpi/ssdt_tpm.asl | 2 +- tools/libacpi/static_tables.c | 2 +- 21 files changed, 65 insertions(+), 27 deletions(-) -- 2.30.2 ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 1/2] LICENSES: Improve the legibility of these files 2023-05-05 13:05 [PATCH 0/2] LICENSES improvements/corrections Andrew Cooper @ 2023-05-05 13:05 ` Andrew Cooper 2023-05-05 13:06 ` Jan Beulich 2023-05-05 13:05 ` [PATCH 2/2] LICENSES: Remove the use of deprecated LGPL SPDX tags Andrew Cooper 1 sibling, 1 reply; 11+ messages in thread From: Andrew Cooper @ 2023-05-05 13:05 UTC (permalink / raw) To: Xen-devel Cc: Andrew Cooper, George Dunlap, Jan Beulich, Stefano Stabellini, Wei Liu, Julien Grall A few newlines go a very long way Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> --- CC: George Dunlap <George.Dunlap@eu.citrix.com> CC: Jan Beulich <JBeulich@suse.com> CC: Stefano Stabellini <sstabellini@kernel.org> CC: Wei Liu <wl@xen.org> CC: Julien Grall <julien@xen.org> --- LICENSES/BSD-2-Clause | 4 ++++ LICENSES/BSD-3-Clause | 4 ++++ LICENSES/BSD-3-Clause-Clear | 4 ++++ LICENSES/CC-BY-4.0 | 5 +++++ LICENSES/GPL-2.0 | 5 +++++ LICENSES/LGPL-2.0 | 6 ++++++ LICENSES/LGPL-2.1 | 6 ++++++ LICENSES/MIT | 4 ++++ 8 files changed, 38 insertions(+) diff --git a/LICENSES/BSD-2-Clause b/LICENSES/BSD-2-Clause index da366e2ce50b..694d8c93221c 100644 --- a/LICENSES/BSD-2-Clause +++ b/LICENSES/BSD-2-Clause @@ -1,10 +1,14 @@ Valid-License-Identifier: BSD-2-Clause + SPDX-URL: https://spdx.org/licenses/BSD-2-Clause.html + Usage-Guide: + To use the BSD 2-clause "Simplified" License put the following SPDX tag/value pair into a comment according to the placement guidelines in the licensing rules documentation: SPDX-License-Identifier: BSD-2-Clause + License-Text: Copyright (c) <year> <owner> . All rights reserved. diff --git a/LICENSES/BSD-3-Clause b/LICENSES/BSD-3-Clause index 34c7f057c8d5..1441947f92e0 100644 --- a/LICENSES/BSD-3-Clause +++ b/LICENSES/BSD-3-Clause @@ -1,10 +1,14 @@ Valid-License-Identifier: BSD-3-Clause + SPDX-URL: https://spdx.org/licenses/BSD-3-Clause.html + Usage-Guide: + To use the BSD 3-clause "New" or "Revised" License put the following SPDX tag/value pair into a comment according to the placement guidelines in the licensing rules documentation: SPDX-License-Identifier: BSD-3-Clause + License-Text: Copyright (c) <year> <owner> . All rights reserved. diff --git a/LICENSES/BSD-3-Clause-Clear b/LICENSES/BSD-3-Clause-Clear index e53b56092b90..2b27f24a65a0 100644 --- a/LICENSES/BSD-3-Clause-Clear +++ b/LICENSES/BSD-3-Clause-Clear @@ -1,10 +1,14 @@ Valid-License-Identifier: BSD-3-Clause-Clear + SPDX-URL: https://spdx.org/licenses/BSD-3-Clause-Clear.html + Usage-Guide: + To use the BSD 3-clause "Clear" License put the following SPDX tag/value pair into a comment according to the placement guidelines in the licensing rules documentation: SPDX-License-Identifier: BSD-3-Clause-Clear + License-Text: The Clear BSD License diff --git a/LICENSES/CC-BY-4.0 b/LICENSES/CC-BY-4.0 index 27dfefa95ccf..4197ceb180ff 100644 --- a/LICENSES/CC-BY-4.0 +++ b/LICENSES/CC-BY-4.0 @@ -1,15 +1,20 @@ Valid-License-Identifier: CC-BY-4.0 + SPDX-URL: https://spdx.org/licenses/CC-BY-4.0 + Usage-Guide: + Do NOT use this license for code, but it's acceptable for content like artwork or documentation. When using it for the latter, it's best to use it together with a GPL2 compatible license using "OR", as CC-BY-4.0 texts processed by the kernel's build system might combine it with content taken from more restrictive licenses. + To use the Creative Commons Attribution 4.0 International license put the following SPDX tag/value pair into a comment according to the placement guidelines in the licensing rules documentation: SPDX-License-Identifier: CC-BY-4.0 + License-Text: Creative Commons Attribution 4.0 International diff --git a/LICENSES/GPL-2.0 b/LICENSES/GPL-2.0 index 9f09528a8bce..0022a7c17788 100644 --- a/LICENSES/GPL-2.0 +++ b/LICENSES/GPL-2.0 @@ -2,11 +2,15 @@ Valid-License-Identifier: GPL-2.0 Valid-License-Identifier: GPL-2.0-only Valid-License-Identifier: GPL-2.0+ Valid-License-Identifier: GPL-2.0-or-later + SPDX-URL: https://spdx.org/licenses/GPL-2.0.html + Usage-Guide: + To use this license in source code, put one of the following SPDX tag/value pairs into a comment according to the placement guidelines in the licensing rules documentation. + For 'GNU General Public License (GPL) version 2 only' use: SPDX-License-Identifier: GPL-2.0-only or (now deprecated) @@ -15,6 +19,7 @@ Usage-Guide: SPDX-License-Identifier: GPL-2.0+ or SPDX-License-Identifier: GPL-2.0-or-later + License-Text: GNU GENERAL PUBLIC LICENSE diff --git a/LICENSES/LGPL-2.0 b/LICENSES/LGPL-2.0 index 957d798fe037..2fa16d72eabf 100644 --- a/LICENSES/LGPL-2.0 +++ b/LICENSES/LGPL-2.0 @@ -1,15 +1,21 @@ Valid-License-Identifier: LGPL-2.0 Valid-License-Identifier: LGPL-2.0+ + SPDX-URL: https://spdx.org/licenses/LGPL-2.0.html + Usage-Guide: + To use this license in source code, put one of the following SPDX tag/value pairs into a comment according to the placement guidelines in the licensing rules documentation. + For 'GNU Library General Public License (LGPL) version 2.0 only' use: SPDX-License-Identifier: LGPL-2.0 + For 'GNU Library General Public License (LGPL) version 2.0 or any later version' use: SPDX-License-Identifier: LGPL-2.0+ + License-Text: GNU LIBRARY GENERAL PUBLIC LICENSE diff --git a/LICENSES/LGPL-2.1 b/LICENSES/LGPL-2.1 index 27bb4342a3e8..b366c7e49199 100644 --- a/LICENSES/LGPL-2.1 +++ b/LICENSES/LGPL-2.1 @@ -1,15 +1,21 @@ Valid-License-Identifier: LGPL-2.1 Valid-License-Identifier: LGPL-2.1+ + SPDX-URL: https://spdx.org/licenses/LGPL-2.1.html + Usage-Guide: + To use this license in source code, put one of the following SPDX tag/value pairs into a comment according to the placement guidelines in the licensing rules documentation. + For 'GNU Lesser General Public License (LGPL) version 2.1 only' use: SPDX-License-Identifier: LGPL-2.1 + For 'GNU Lesser General Public License (LGPL) version 2.1 or any later version' use: SPDX-License-Identifier: LGPL-2.1+ + License-Text: GNU LESSER GENERAL PUBLIC LICENSE diff --git a/LICENSES/MIT b/LICENSES/MIT index f33a68ceb3ea..eba1549f93e4 100644 --- a/LICENSES/MIT +++ b/LICENSES/MIT @@ -1,10 +1,14 @@ Valid-License-Identifier: MIT + SPDX-URL: https://spdx.org/licenses/MIT.html + Usage-Guide: + To use the MIT License put the following SPDX tag/value pair into a comment according to the placement guidelines in the licensing rules documentation: SPDX-License-Identifier: MIT + License-Text: MIT License -- 2.30.2 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] LICENSES: Improve the legibility of these files 2023-05-05 13:05 ` [PATCH 1/2] LICENSES: Improve the legibility of these files Andrew Cooper @ 2023-05-05 13:06 ` Jan Beulich 2023-05-05 22:29 ` Stefano Stabellini 0 siblings, 1 reply; 11+ messages in thread From: Jan Beulich @ 2023-05-05 13:06 UTC (permalink / raw) To: Andrew Cooper Cc: George Dunlap, Stefano Stabellini, Wei Liu, Julien Grall, Xen-devel On 05.05.2023 15:05, Andrew Cooper wrote: > A few newlines go a very long way > > Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> Acked-by: Jan Beulich <jbeulich@suse.com> ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/2] LICENSES: Improve the legibility of these files 2023-05-05 13:06 ` Jan Beulich @ 2023-05-05 22:29 ` Stefano Stabellini 0 siblings, 0 replies; 11+ messages in thread From: Stefano Stabellini @ 2023-05-05 22:29 UTC (permalink / raw) To: Jan Beulich Cc: Andrew Cooper, George Dunlap, Stefano Stabellini, Wei Liu, Julien Grall, Xen-devel On Fri, 5 May 2023, Jan Beulich wrote: > On 05.05.2023 15:05, Andrew Cooper wrote: > > A few newlines go a very long way > > > > Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> > > Acked-by: Jan Beulich <jbeulich@suse.com> Although I don't mind either way it should be noted that the LICENSES files came from Linux, and Linux doesn't have the extra spaces. That said, these are licenses files not code, so we don't necessarily need to keep them in sync over time I think. So I am fine with this. I am also fine if you drop this patch. Acked-by: Stefano Stabellini <sstabellini@kernel.org> ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 2/2] LICENSES: Remove the use of deprecated LGPL SPDX tags 2023-05-05 13:05 [PATCH 0/2] LICENSES improvements/corrections Andrew Cooper 2023-05-05 13:05 ` [PATCH 1/2] LICENSES: Improve the legibility of these files Andrew Cooper @ 2023-05-05 13:05 ` Andrew Cooper 2023-05-05 13:13 ` Jan Beulich 2023-05-05 15:30 ` Anthony PERARD 1 sibling, 2 replies; 11+ messages in thread From: Andrew Cooper @ 2023-05-05 13:05 UTC (permalink / raw) To: Xen-devel Cc: Andrew Cooper, George Dunlap, Jan Beulich, Stefano Stabellini, Wei Liu, Julien Grall The SPDX forms without an explicit -only or -or-later suffix are deprecated and should not be used. The recent changes to libacpi are the only examples in tree, so fix them all up. For GPL, we have many examples using deprecated tags. For now, just identify them as such recommend that no new instances get added. Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> --- CC: George Dunlap <George.Dunlap@eu.citrix.com> CC: Jan Beulich <JBeulich@suse.com> CC: Stefano Stabellini <sstabellini@kernel.org> CC: Wei Liu <wl@xen.org> CC: Julien Grall <julien@xen.org> Unsure whether this should get some Fixes: tags or not. Note also that Jenny's "[PATCH v4 2/2] acpi: Add TPM2 interface definition." wants its SPDX tag correcting as per this patch. --- LICENSES/GPL-2.0 | 12 ++++++------ LICENSES/LGPL-2.0 | 8 ++++---- LICENSES/LGPL-2.1 | 8 ++++---- tools/libacpi/Makefile | 2 +- tools/libacpi/acpi2_0.h | 2 +- tools/libacpi/build.c | 2 +- tools/libacpi/dsdt.asl | 2 +- tools/libacpi/dsdt_acpi_info.asl | 2 +- tools/libacpi/libacpi.h | 2 +- tools/libacpi/mk_dsdt.c | 2 +- tools/libacpi/ssdt_laptop_slate.asl | 2 +- tools/libacpi/ssdt_pm.asl | 2 +- tools/libacpi/ssdt_s3.asl | 2 +- tools/libacpi/ssdt_s4.asl | 2 +- tools/libacpi/ssdt_tpm.asl | 2 +- tools/libacpi/static_tables.c | 2 +- 16 files changed, 27 insertions(+), 27 deletions(-) diff --git a/LICENSES/GPL-2.0 b/LICENSES/GPL-2.0 index 0022a7c17788..dcd969aa85b5 100644 --- a/LICENSES/GPL-2.0 +++ b/LICENSES/GPL-2.0 @@ -1,8 +1,9 @@ -Valid-License-Identifier: GPL-2.0 Valid-License-Identifier: GPL-2.0-only -Valid-License-Identifier: GPL-2.0+ Valid-License-Identifier: GPL-2.0-or-later +Deprecated-Identifier: GPL-2.0 +Deprecated-Identifier: GPL-2.0+ + SPDX-URL: https://spdx.org/licenses/GPL-2.0.html Usage-Guide: @@ -13,13 +14,12 @@ Usage-Guide: For 'GNU General Public License (GPL) version 2 only' use: SPDX-License-Identifier: GPL-2.0-only - or (now deprecated) - SPDX-License-Identifier: GPL-2.0 For 'GNU General Public License (GPL) version 2 or any later version' use: - SPDX-License-Identifier: GPL-2.0+ - or SPDX-License-Identifier: GPL-2.0-or-later + The deprecated tags should not be used for any new additions. Where + possible, their existing uses should be phased out. + License-Text: GNU GENERAL PUBLIC LICENSE diff --git a/LICENSES/LGPL-2.0 b/LICENSES/LGPL-2.0 index 2fa16d72eabf..c960ba3ce3b8 100644 --- a/LICENSES/LGPL-2.0 +++ b/LICENSES/LGPL-2.0 @@ -1,5 +1,5 @@ -Valid-License-Identifier: LGPL-2.0 -Valid-License-Identifier: LGPL-2.0+ +Valid-License-Identifier: LGPL-2.0-only +Valid-License-Identifier: LGPL-2.0-or-later SPDX-URL: https://spdx.org/licenses/LGPL-2.0.html @@ -10,11 +10,11 @@ Usage-Guide: guidelines in the licensing rules documentation. For 'GNU Library General Public License (LGPL) version 2.0 only' use: - SPDX-License-Identifier: LGPL-2.0 + SPDX-License-Identifier: LGPL-2.0-only For 'GNU Library General Public License (LGPL) version 2.0 or any later version' use: - SPDX-License-Identifier: LGPL-2.0+ + SPDX-License-Identifier: LGPL-2.0-or-later License-Text: diff --git a/LICENSES/LGPL-2.1 b/LICENSES/LGPL-2.1 index b366c7e49199..4553664b76bf 100644 --- a/LICENSES/LGPL-2.1 +++ b/LICENSES/LGPL-2.1 @@ -1,5 +1,5 @@ -Valid-License-Identifier: LGPL-2.1 -Valid-License-Identifier: LGPL-2.1+ +Valid-License-Identifier: LGPL-2.1-only +Valid-License-Identifier: LGPL-2.1-or-later SPDX-URL: https://spdx.org/licenses/LGPL-2.1.html @@ -10,11 +10,11 @@ Usage-Guide: guidelines in the licensing rules documentation. For 'GNU Lesser General Public License (LGPL) version 2.1 only' use: - SPDX-License-Identifier: LGPL-2.1 + SPDX-License-Identifier: LGPL-2.1-only For 'GNU Lesser General Public License (LGPL) version 2.1 or any later version' use: - SPDX-License-Identifier: LGPL-2.1+ + SPDX-License-Identifier: LGPL-2.1-or-later License-Text: diff --git a/tools/libacpi/Makefile b/tools/libacpi/Makefile index aa9c520cbe85..bcfcd852f92f 100644 --- a/tools/libacpi/Makefile +++ b/tools/libacpi/Makefile @@ -1,4 +1,4 @@ -# SPDX-License-Identifier: LGPL-2.1 +# SPDX-License-Identifier: LGPL-2.1-only # # Copyright (c) 2004, Intel Corporation. diff --git a/tools/libacpi/acpi2_0.h b/tools/libacpi/acpi2_0.h index 212f5ab64182..e00b29854be0 100644 --- a/tools/libacpi/acpi2_0.h +++ b/tools/libacpi/acpi2_0.h @@ -1,4 +1,4 @@ -/* SPDX-License-Identifier: LGPL-2.1 */ +/* SPDX-License-Identifier: LGPL-2.1-only */ /* * Copyright (c) 2004, Intel Corporation. */ diff --git a/tools/libacpi/build.c b/tools/libacpi/build.c index 830d37c61f03..3142e0ac84c0 100644 --- a/tools/libacpi/build.c +++ b/tools/libacpi/build.c @@ -1,4 +1,4 @@ -/* SPDX-License-Identifier: LGPL-2.1 */ +/* SPDX-License-Identifier: LGPL-2.1-only */ /* * Copyright (c) 2004, Intel Corporation. * Copyright (c) 2006, Keir Fraser, XenSource Inc. diff --git a/tools/libacpi/dsdt.asl b/tools/libacpi/dsdt.asl index c6691b56a986..32b42f85ae9f 100644 --- a/tools/libacpi/dsdt.asl +++ b/tools/libacpi/dsdt.asl @@ -1,4 +1,4 @@ -/* SPDX-License-Identifier: LGPL-2.1 */ +/* SPDX-License-Identifier: LGPL-2.1-only */ /****************************************************************************** * DSDT for Xen with Qemu device model * diff --git a/tools/libacpi/dsdt_acpi_info.asl b/tools/libacpi/dsdt_acpi_info.asl index c6e82f1fe6a7..6e114fa23404 100644 --- a/tools/libacpi/dsdt_acpi_info.asl +++ b/tools/libacpi/dsdt_acpi_info.asl @@ -1,4 +1,4 @@ -/* SPDX-License-Identifier: LGPL-2.1 */ +/* SPDX-License-Identifier: LGPL-2.1-only */ Scope (\_SB) { diff --git a/tools/libacpi/libacpi.h b/tools/libacpi/libacpi.h index acf012e35578..7ae28525f604 100644 --- a/tools/libacpi/libacpi.h +++ b/tools/libacpi/libacpi.h @@ -1,4 +1,4 @@ -/* SPDX-License-Identifier: LGPL-2.1 */ +/* SPDX-License-Identifier: LGPL-2.1-only */ /****************************************************************************** * libacpi.h * diff --git a/tools/libacpi/mk_dsdt.c b/tools/libacpi/mk_dsdt.c index c74b270c0c5d..34f6753f6193 100644 --- a/tools/libacpi/mk_dsdt.c +++ b/tools/libacpi/mk_dsdt.c @@ -1,4 +1,4 @@ -/* SPDX-License-Identifier: LGPL-2.1 */ +/* SPDX-License-Identifier: LGPL-2.1-only */ #include <stdio.h> #include <stdarg.h> diff --git a/tools/libacpi/ssdt_laptop_slate.asl b/tools/libacpi/ssdt_laptop_slate.asl index 494f2d048d0a..69fd504c19fc 100644 --- a/tools/libacpi/ssdt_laptop_slate.asl +++ b/tools/libacpi/ssdt_laptop_slate.asl @@ -1,4 +1,4 @@ -/* SPDX-License-Identifier: LGPL-2.1 */ +/* SPDX-License-Identifier: LGPL-2.1-only */ /* * ssdt_conv.asl * diff --git a/tools/libacpi/ssdt_pm.asl b/tools/libacpi/ssdt_pm.asl index e577e85c072b..db578d10ac3e 100644 --- a/tools/libacpi/ssdt_pm.asl +++ b/tools/libacpi/ssdt_pm.asl @@ -1,4 +1,4 @@ -/* SPDX-License-Identifier: LGPL-2.1 */ +/* SPDX-License-Identifier: LGPL-2.1-only */ /* * ssdt_pm.asl * diff --git a/tools/libacpi/ssdt_s3.asl b/tools/libacpi/ssdt_s3.asl index 8f3177ec5adc..f6e9636f4759 100644 --- a/tools/libacpi/ssdt_s3.asl +++ b/tools/libacpi/ssdt_s3.asl @@ -1,4 +1,4 @@ -/* SPDX-License-Identifier: LGPL-2.1 */ +/* SPDX-License-Identifier: LGPL-2.1-only */ /* * ssdt_s3.asl * diff --git a/tools/libacpi/ssdt_s4.asl b/tools/libacpi/ssdt_s4.asl index 979318eca1f5..8014f5fc9014 100644 --- a/tools/libacpi/ssdt_s4.asl +++ b/tools/libacpi/ssdt_s4.asl @@ -1,4 +1,4 @@ -/* SPDX-License-Identifier: LGPL-2.1 */ +/* SPDX-License-Identifier: LGPL-2.1-only */ /* * ssdt_s4.asl * diff --git a/tools/libacpi/ssdt_tpm.asl b/tools/libacpi/ssdt_tpm.asl index 6c3267218f3b..944658d25177 100644 --- a/tools/libacpi/ssdt_tpm.asl +++ b/tools/libacpi/ssdt_tpm.asl @@ -1,4 +1,4 @@ -/* SPDX-License-Identifier: LGPL-2.1 */ +/* SPDX-License-Identifier: LGPL-2.1-only */ /* * ssdt_tpm.asl * diff --git a/tools/libacpi/static_tables.c b/tools/libacpi/static_tables.c index 631fb911413b..715f46fee05c 100644 --- a/tools/libacpi/static_tables.c +++ b/tools/libacpi/static_tables.c @@ -1,4 +1,4 @@ -/* SPDX-License-Identifier: LGPL-2.1 */ +/* SPDX-License-Identifier: LGPL-2.1-only */ /* * Copyright (c) 2004, Intel Corporation. * Copyright (c) 2006, Keir Fraser, XenSource Inc. -- 2.30.2 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH 2/2] LICENSES: Remove the use of deprecated LGPL SPDX tags 2023-05-05 13:05 ` [PATCH 2/2] LICENSES: Remove the use of deprecated LGPL SPDX tags Andrew Cooper @ 2023-05-05 13:13 ` Jan Beulich 2023-05-05 13:22 ` Andrew Cooper 2023-05-05 15:30 ` Anthony PERARD 1 sibling, 1 reply; 11+ messages in thread From: Jan Beulich @ 2023-05-05 13:13 UTC (permalink / raw) To: Andrew Cooper Cc: George Dunlap, Stefano Stabellini, Wei Liu, Julien Grall, Xen-devel On 05.05.2023 15:05, Andrew Cooper wrote: > The SPDX forms without an explicit -only or -or-later suffix are deprecated > and should not be used. I guess this wants a reference to where this is specified. In particular ... > --- a/LICENSES/LGPL-2.1 > +++ b/LICENSES/LGPL-2.1 > @@ -1,5 +1,5 @@ > -Valid-License-Identifier: LGPL-2.1 > -Valid-License-Identifier: LGPL-2.1+ > +Valid-License-Identifier: LGPL-2.1-only > +Valid-License-Identifier: LGPL-2.1-or-later > > SPDX-URL: https://spdx.org/licenses/LGPL-2.1.html ... I can't spot anything like this under e.g. this URL. Also is there a reason you add Deprecated-Identifier: only to GPL-2.0? Enumerating them would seem reasonable to me, not just for completeness, but also in case we end up importing a file with a deprecated tag. Jan ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 2/2] LICENSES: Remove the use of deprecated LGPL SPDX tags 2023-05-05 13:13 ` Jan Beulich @ 2023-05-05 13:22 ` Andrew Cooper 2023-05-05 13:31 ` Jan Beulich 0 siblings, 1 reply; 11+ messages in thread From: Andrew Cooper @ 2023-05-05 13:22 UTC (permalink / raw) To: Jan Beulich Cc: George Dunlap, Stefano Stabellini, Wei Liu, Julien Grall, Xen-devel On 05/05/2023 2:13 pm, Jan Beulich wrote: > On 05.05.2023 15:05, Andrew Cooper wrote: >> The SPDX forms without an explicit -only or -or-later suffix are deprecated >> and should not be used. > I guess this wants a reference to where this is specified. In particular ... > >> --- a/LICENSES/LGPL-2.1 >> +++ b/LICENSES/LGPL-2.1 >> @@ -1,5 +1,5 @@ >> -Valid-License-Identifier: LGPL-2.1 >> -Valid-License-Identifier: LGPL-2.1+ >> +Valid-License-Identifier: LGPL-2.1-only >> +Valid-License-Identifier: LGPL-2.1-or-later >> >> SPDX-URL: https://spdx.org/licenses/LGPL-2.1.html > ... I can't spot anything like this under e.g. this URL. Hmm yeah, it is irritating. The statement is at https://spdx.org/licenses/ but only by virtue of two tables, the second of which is the list of deprecated identifiers. I'll put a paragraph about this in the commit message. > Also is there a reason you add Deprecated-Identifier: only to GPL-2.0? > Enumerating them would seem reasonable to me, not just for completeness, > but also in case we end up importing a file with a deprecated tag. We have problematic uses of GPL in tree, where we don't for LGPL. I'm considering a gitlab pass which will reject patches with use an identifier not in the permitted list, and reject in introduction of new uses of the deprecated ones. For this, the deprecated-but-tolerated tags need calling out in some machine-readable way, but I don't think it's helpful to express the tolerating of a tag we don't have any violations of. ~Andrew ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 2/2] LICENSES: Remove the use of deprecated LGPL SPDX tags 2023-05-05 13:22 ` Andrew Cooper @ 2023-05-05 13:31 ` Jan Beulich 2023-05-05 13:36 ` Andrew Cooper 0 siblings, 1 reply; 11+ messages in thread From: Jan Beulich @ 2023-05-05 13:31 UTC (permalink / raw) To: Andrew Cooper Cc: George Dunlap, Stefano Stabellini, Wei Liu, Julien Grall, Xen-devel On 05.05.2023 15:22, Andrew Cooper wrote: > On 05/05/2023 2:13 pm, Jan Beulich wrote: >> On 05.05.2023 15:05, Andrew Cooper wrote: >>> The SPDX forms without an explicit -only or -or-later suffix are deprecated >>> and should not be used. >> I guess this wants a reference to where this is specified. In particular ... >> >>> --- a/LICENSES/LGPL-2.1 >>> +++ b/LICENSES/LGPL-2.1 >>> @@ -1,5 +1,5 @@ >>> -Valid-License-Identifier: LGPL-2.1 >>> -Valid-License-Identifier: LGPL-2.1+ >>> +Valid-License-Identifier: LGPL-2.1-only >>> +Valid-License-Identifier: LGPL-2.1-or-later >>> >>> SPDX-URL: https://spdx.org/licenses/LGPL-2.1.html >> ... I can't spot anything like this under e.g. this URL. > > Hmm yeah, it is irritating. The statement is at > https://spdx.org/licenses/ but only by virtue of two tables, the second > of which is the list of deprecated identifiers. > > I'll put a paragraph about this in the commit message. > >> Also is there a reason you add Deprecated-Identifier: only to GPL-2.0? >> Enumerating them would seem reasonable to me, not just for completeness, >> but also in case we end up importing a file with a deprecated tag. > > We have problematic uses of GPL in tree, where we don't for LGPL. > > I'm considering a gitlab pass which will reject patches with use an > identifier not in the permitted list, and reject in introduction of new > uses of the deprecated ones. For this, the deprecated-but-tolerated > tags need calling out in some machine-readable way, but I don't think > it's helpful to express the tolerating of a tag we don't have any > violations of. Hmm, okay. With the expanded commit message Acked-by: Jan Beulich <jbeulich@suse.com> Jan ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 2/2] LICENSES: Remove the use of deprecated LGPL SPDX tags 2023-05-05 13:31 ` Jan Beulich @ 2023-05-05 13:36 ` Andrew Cooper 0 siblings, 0 replies; 11+ messages in thread From: Andrew Cooper @ 2023-05-05 13:36 UTC (permalink / raw) To: Jan Beulich Cc: George Dunlap, Stefano Stabellini, Wei Liu, Julien Grall, Xen-devel On 05/05/2023 2:31 pm, Jan Beulich wrote: > On 05.05.2023 15:22, Andrew Cooper wrote: >> On 05/05/2023 2:13 pm, Jan Beulich wrote: >>> On 05.05.2023 15:05, Andrew Cooper wrote: >>>> The SPDX forms without an explicit -only or -or-later suffix are deprecated >>>> and should not be used. >>> I guess this wants a reference to where this is specified. In particular ... >>> >>>> --- a/LICENSES/LGPL-2.1 >>>> +++ b/LICENSES/LGPL-2.1 >>>> @@ -1,5 +1,5 @@ >>>> -Valid-License-Identifier: LGPL-2.1 >>>> -Valid-License-Identifier: LGPL-2.1+ >>>> +Valid-License-Identifier: LGPL-2.1-only >>>> +Valid-License-Identifier: LGPL-2.1-or-later >>>> >>>> SPDX-URL: https://spdx.org/licenses/LGPL-2.1.html >>> ... I can't spot anything like this under e.g. this URL. >> Hmm yeah, it is irritating. The statement is at >> https://spdx.org/licenses/ but only by virtue of two tables, the second >> of which is the list of deprecated identifiers. >> >> I'll put a paragraph about this in the commit message. >> >>> Also is there a reason you add Deprecated-Identifier: only to GPL-2.0? >>> Enumerating them would seem reasonable to me, not just for completeness, >>> but also in case we end up importing a file with a deprecated tag. >> We have problematic uses of GPL in tree, where we don't for LGPL. >> >> I'm considering a gitlab pass which will reject patches with use an >> identifier not in the permitted list, and reject in introduction of new >> uses of the deprecated ones. For this, the deprecated-but-tolerated >> tags need calling out in some machine-readable way, but I don't think >> it's helpful to express the tolerating of a tag we don't have any >> violations of. > Hmm, okay. With the expanded commit message > Acked-by: Jan Beulich <jbeulich@suse.com> Thanks. FAOD, here's the full commit message with adjustment: LICENSES: Remove the use of deprecated LGPL SPDX tags The SPDX forms without an explicit -only or -or-later suffix are deprecated and should not be used. Somewhat unhelpfully at the time of writing, this only appears to be indicated by the separation of the two tables at https://spdx.org/licenses/ The recent changes to libacpi are the only examples of deprecated LGPL tags in tree, so fix them all up. For GPL, we have many examples using deprecated tags. For now, just identify them as such and recommend that no new instances get added. Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 2/2] LICENSES: Remove the use of deprecated LGPL SPDX tags 2023-05-05 13:05 ` [PATCH 2/2] LICENSES: Remove the use of deprecated LGPL SPDX tags Andrew Cooper 2023-05-05 13:13 ` Jan Beulich @ 2023-05-05 15:30 ` Anthony PERARD 2023-05-05 15:52 ` Andrew Cooper 1 sibling, 1 reply; 11+ messages in thread From: Anthony PERARD @ 2023-05-05 15:30 UTC (permalink / raw) To: Andrew Cooper Cc: Xen-devel, George Dunlap, Jan Beulich, Stefano Stabellini, Wei Liu, Julien Grall On Fri, May 05, 2023 at 02:05:33PM +0100, Andrew Cooper wrote: > --- > LICENSES/GPL-2.0 | 12 ++++++------ > LICENSES/LGPL-2.0 | 8 ++++---- > LICENSES/LGPL-2.1 | 8 ++++---- > > diff --git a/LICENSES/GPL-2.0 b/LICENSES/GPL-2.0 > index 0022a7c17788..dcd969aa85b5 100644 > --- a/LICENSES/GPL-2.0 > +++ b/LICENSES/GPL-2.0 > @@ -1,8 +1,9 @@ > -Valid-License-Identifier: GPL-2.0 > Valid-License-Identifier: GPL-2.0-only > -Valid-License-Identifier: GPL-2.0+ > Valid-License-Identifier: GPL-2.0-or-later > > +Deprecated-Identifier: GPL-2.0 > +Deprecated-Identifier: GPL-2.0+ > + > SPDX-URL: https://spdx.org/licenses/GPL-2.0.html You probably want to update the URLs as well, this one point to a page with a deprecated notice. New url is: https://spdx.org/licenses/GPL-2.0-only.html Same remark for the other licences file changed. Also, maybe they want to be renamed as well, to match the identifier, I'm not sure. Cheers, -- Anthony PERARD ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 2/2] LICENSES: Remove the use of deprecated LGPL SPDX tags 2023-05-05 15:30 ` Anthony PERARD @ 2023-05-05 15:52 ` Andrew Cooper 0 siblings, 0 replies; 11+ messages in thread From: Andrew Cooper @ 2023-05-05 15:52 UTC (permalink / raw) To: Anthony PERARD Cc: Xen-devel, George Dunlap, Jan Beulich, Stefano Stabellini, Wei Liu, Julien Grall On 05/05/2023 4:30 pm, Anthony PERARD wrote: > On Fri, May 05, 2023 at 02:05:33PM +0100, Andrew Cooper wrote: >> --- >> LICENSES/GPL-2.0 | 12 ++++++------ >> LICENSES/LGPL-2.0 | 8 ++++---- >> LICENSES/LGPL-2.1 | 8 ++++---- >> >> diff --git a/LICENSES/GPL-2.0 b/LICENSES/GPL-2.0 >> index 0022a7c17788..dcd969aa85b5 100644 >> --- a/LICENSES/GPL-2.0 >> +++ b/LICENSES/GPL-2.0 >> @@ -1,8 +1,9 @@ >> -Valid-License-Identifier: GPL-2.0 >> Valid-License-Identifier: GPL-2.0-only >> -Valid-License-Identifier: GPL-2.0+ >> Valid-License-Identifier: GPL-2.0-or-later >> >> +Deprecated-Identifier: GPL-2.0 >> +Deprecated-Identifier: GPL-2.0+ >> + >> SPDX-URL: https://spdx.org/licenses/GPL-2.0.html > You probably want to update the URLs as well, this one point to a page > with a deprecated notice. New url is: > https://spdx.org/licenses/GPL-2.0-only.html > > Same remark for the other licences file changed. > > Also, maybe they want to be renamed as well, to match the identifier, > I'm not sure. Hmm, all good points. I'll update the URLs because that's easy. For the files however, the GPL ones already discuss both the "only" or "or-later" forms, so the filename already doesn't match the SDPX tag specifically. I think I'll leave them as-are. ~Andrew ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2023-05-05 22:29 UTC | newest] Thread overview: 11+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-05-05 13:05 [PATCH 0/2] LICENSES improvements/corrections Andrew Cooper 2023-05-05 13:05 ` [PATCH 1/2] LICENSES: Improve the legibility of these files Andrew Cooper 2023-05-05 13:06 ` Jan Beulich 2023-05-05 22:29 ` Stefano Stabellini 2023-05-05 13:05 ` [PATCH 2/2] LICENSES: Remove the use of deprecated LGPL SPDX tags Andrew Cooper 2023-05-05 13:13 ` Jan Beulich 2023-05-05 13:22 ` Andrew Cooper 2023-05-05 13:31 ` Jan Beulich 2023-05-05 13:36 ` Andrew Cooper 2023-05-05 15:30 ` Anthony PERARD 2023-05-05 15:52 ` Andrew Cooper
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.