Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] [RFC] Manual.md: use SPDX expressions for license=
@ 2024-06-13 18:03 classabbyamp
  2024-06-13 18:03 ` classabbyamp
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: classabbyamp @ 2024-06-13 18:03 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 777 bytes --]

There is a new pull request by classabbyamp against master on the void-packages repository

https://github.com/classabbyamp/void-packages spdx
https://github.com/void-linux/void-packages/pull/50819

[RFC] Manual.md: use SPDX expressions for license=
see also: https://github.com/void-linux/void-packages/issues/48303

xlint supports parsing this since leahneukirchen/xtools#322

SPDX expressions are more expressive and accurate than the comma-separated format, and WITH has already been usable for a long time.

I propose that existing multi-licence strings be changed in batches (if someone wants to spend the time) or as templates are modified.

@void-linux/pkg-committers


A patch file from https://github.com/void-linux/void-packages/pull/50819.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-spdx-50819.patch --]
[-- Type: text/x-diff, Size: 1468 bytes --]

From 39fb1342dac13c24c2096b1925f25a8a971b9f08 Mon Sep 17 00:00:00 2001
From: classabbyamp <void@placeviolette.net>
Date: Thu, 13 Jun 2024 13:49:51 -0400
Subject: [PATCH] Manual.md: use SPDX expressions for license=

xlint supports parsing this since leahneukirchen/xtools#322

SPDX expressions are more expressive and accurate than the
comma-separated format, and WITH has already been usable for a long
time.
---
 Manual.md | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/Manual.md b/Manual.md
index 507a18729459bb..80cc8f2c0bdd8b 100644
--- a/Manual.md
+++ b/Manual.md
@@ -439,7 +439,12 @@ The list of mandatory variables for a template:
 - <a id="var_license"></a>
 `license` A string matching the license's [SPDX Short identifier](https://spdx.org/licenses),
 `Public Domain`, or string prefixed with `custom:` for other licenses.
-Multiple licenses should be separated by commas, Example: `GPL-3.0-or-later, custom:Hugware`.
+Multiple licenses should be listed as an
+[SPDX license expression](https://spdx.github.io/spdx-spec/v3.0/annexes/SPDX-license-expressions/)
+(examples: `MIT OR Apache-2.0`, `MIT AND (LGPL-2.1-or-later OR BSD-3-Clause)`).
+Usage of `AND`, `OR`, `WITH`, and `()` are supported by xlint.
+The legacy comma-separated format should be converted when encountered (example:
+`GPL-3.0-or-later, custom:Hugware`).
 
   Empty meta-packages that don't include any files
   and thus have and require no license should use

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [RFC] Manual.md: use SPDX expressions for license=
  2024-06-13 18:03 [PR PATCH] [RFC] Manual.md: use SPDX expressions for license= classabbyamp
@ 2024-06-13 18:03 ` classabbyamp
  2024-06-13 18:05 ` [PR PATCH] [Updated] " classabbyamp
  2024-06-13 19:49 ` classabbyamp
  2 siblings, 0 replies; 4+ messages in thread
From: classabbyamp @ 2024-06-13 18:03 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 222 bytes --]

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/50819#issuecomment-2166463730

Comment:
would also be nice to have an xtools release so users can lint this locally

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PR PATCH] [Updated] [RFC] Manual.md: use SPDX expressions for license=
  2024-06-13 18:03 [PR PATCH] [RFC] Manual.md: use SPDX expressions for license= classabbyamp
  2024-06-13 18:03 ` classabbyamp
@ 2024-06-13 18:05 ` classabbyamp
  2024-06-13 19:49 ` classabbyamp
  2 siblings, 0 replies; 4+ messages in thread
From: classabbyamp @ 2024-06-13 18:05 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 782 bytes --]

There is an updated pull request by classabbyamp against master on the void-packages repository

https://github.com/classabbyamp/void-packages spdx
https://github.com/void-linux/void-packages/pull/50819

[RFC] Manual.md: use SPDX expressions for license=
see also: https://github.com/void-linux/void-packages/issues/48303

xlint supports parsing this since leahneukirchen/xtools#322

SPDX expressions are more expressive and accurate than the comma-separated format, and WITH has already been usable for a long time.

I propose that existing multi-licence strings be changed in batches (if someone wants to spend the time) or as templates are modified.

@void-linux/pkg-committers


A patch file from https://github.com/void-linux/void-packages/pull/50819.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-spdx-50819.patch --]
[-- Type: text/x-diff, Size: 1468 bytes --]

From ac1f4952ecf34328f0186524a5066c4d06d00d61 Mon Sep 17 00:00:00 2001
From: classabbyamp <void@placeviolette.net>
Date: Thu, 13 Jun 2024 13:49:51 -0400
Subject: [PATCH] Manual.md: use SPDX expressions for license=

xlint supports parsing this since leahneukirchen/xtools#322

SPDX expressions are more expressive and accurate than the
comma-separated format, and WITH has already been usable for a long
time.
---
 Manual.md | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/Manual.md b/Manual.md
index 507a18729459bb..1cc0ea701b592c 100644
--- a/Manual.md
+++ b/Manual.md
@@ -439,7 +439,12 @@ The list of mandatory variables for a template:
 - <a id="var_license"></a>
 `license` A string matching the license's [SPDX Short identifier](https://spdx.org/licenses),
 `Public Domain`, or string prefixed with `custom:` for other licenses.
-Multiple licenses should be separated by commas, Example: `GPL-3.0-or-later, custom:Hugware`.
+Multiple licenses should be listed as an
+[SPDX license expression](https://spdx.github.io/spdx-spec/v3.0/annexes/SPDX-license-expressions/)
+(examples: `MIT OR Apache-2.0`, `MIT AND (LGPL-2.1-or-later OR BSD-3-Clause)`).
+Usage of `AND`, `OR`, `WITH`, and `()` are supported by xlint. The legacy
+comma-separated format should be converted when encountered (example:
+`GPL-3.0-or-later, custom:Hugware`).
 
   Empty meta-packages that don't include any files
   and thus have and require no license should use

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [RFC] Manual.md: use SPDX expressions for license=
  2024-06-13 18:03 [PR PATCH] [RFC] Manual.md: use SPDX expressions for license= classabbyamp
  2024-06-13 18:03 ` classabbyamp
  2024-06-13 18:05 ` [PR PATCH] [Updated] " classabbyamp
@ 2024-06-13 19:49 ` classabbyamp
  2 siblings, 0 replies; 4+ messages in thread
From: classabbyamp @ 2024-06-13 19:49 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 338 bytes --]

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/50819#issuecomment-2166648983

Comment:
I'm also considering writing a small `common/scripts/spdx-validate` tool that does more complete parsing of `license` for xlint, instead of just converting `AND`, `OR`, `WITH`, `()` to commas

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2024-06-13 19:49 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-06-13 18:03 [PR PATCH] [RFC] Manual.md: use SPDX expressions for license= classabbyamp
2024-06-13 18:03 ` classabbyamp
2024-06-13 18:05 ` [PR PATCH] [Updated] " classabbyamp
2024-06-13 19:49 ` classabbyamp

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).