[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#58621] [PATCH 0/3] import/utils: spdx-string->license: Match case-i
From: |
Ludovic Courtès |
Subject: |
[bug#58621] [PATCH 0/3] import/utils: spdx-string->license: Match case-insensitively and support '+' operator. |
Date: |
Sun, 20 Nov 2022 11:49:12 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Hi,
Philip McGrath <philip@philipmcgrath.com> skribis:
> On Friday, November 18, 2022 8:45:26 AM EST Ludovic Courtès wrote:
[...]
>> Philip McGrath <philip@philipmcgrath.com> skribis:
>> > + (and (string-suffix? "+" str)
>> > + ;; We try the form with the + to support deprecated identifiers
>> > for + ;; GNU licenses (see above). Here, we handle other uses
>> > of +. + (spdx-string->license
>> > + (substring str 0 (- (string-length str) 1)))))))
>>
>> I guess we can remove the “+” forms from the alist now?
>>
>
> I think we still want the "+" forms in the alist so that we continue convert
> "GPL-2.0+" as though it were "GPL-2.0-or-later", not "GPL-2.0-only".
Oh right. Then I wonder why the code above (with ‘substring’) doesn’t
replace “+” with “-or-later”?
Ludo’.