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
                   ` (5 more replies)
  0 siblings, 6 replies; 7+ 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] 7+ 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
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 7+ 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] 7+ 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
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 7+ 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] 7+ 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
  2024-09-12  1:56 ` github-actions
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 7+ 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] 7+ 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
                   ` (2 preceding siblings ...)
  2024-06-13 19:49 ` classabbyamp
@ 2024-09-12  1:56 ` github-actions
  2024-09-24 18:26 ` classabbyamp
  2024-09-25 22:01 ` [PR PATCH] [Merged]: " classabbyamp
  5 siblings, 0 replies; 7+ messages in thread
From: github-actions @ 2024-09-12  1:56 UTC (permalink / raw)
  To: ml

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

New comment by github-actions[bot] on void-packages repository

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

Comment:
Pull Requests become stale 90 days after last activity and are closed 14 days after that.  If this pull request is still relevant bump it or assign it.

^ permalink raw reply	[flat|nested] 7+ 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
                   ` (3 preceding siblings ...)
  2024-09-12  1:56 ` github-actions
@ 2024-09-24 18:26 ` classabbyamp
  2024-09-25 22:01 ` [PR PATCH] [Merged]: " classabbyamp
  5 siblings, 0 replies; 7+ messages in thread
From: classabbyamp @ 2024-09-24 18:26 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 231 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~~ done

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

* Re: [PR PATCH] [Merged]: [RFC] Manual.md: use SPDX expressions for license=
  2024-06-13 18:03 [PR PATCH] [RFC] Manual.md: use SPDX expressions for license= classabbyamp
                   ` (4 preceding siblings ...)
  2024-09-24 18:26 ` classabbyamp
@ 2024-09-25 22:01 ` classabbyamp
  5 siblings, 0 replies; 7+ messages in thread
From: classabbyamp @ 2024-09-25 22:01 UTC (permalink / raw)
  To: ml

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

There's a merged pull request on the void-packages repository

[RFC] Manual.md: use SPDX expressions for license=
https://github.com/void-linux/void-packages/pull/50819

Description:
see also:
- https://github.com/void-linux/void-packages/issues/48303
- https://github.com/void-linux/void-packages/issues/11240

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

there are also a few `qt6-*` packages that already use this notation

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

end of thread, other threads:[~2024-09-25 22:01 UTC | newest]

Thread overview: 7+ 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
2024-09-12  1:56 ` github-actions
2024-09-24 18:26 ` classabbyamp
2024-09-25 22:01 ` [PR PATCH] [Merged]: " 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).