Subject: Re: bug#58621: [PATCH 0/3] import/utils: spdx-string->license: Match case-insensitively and support '+' operator.

From: Ludovic Courtès <[email-masked]>
Date: Fri, 18 Nov 2022 14:45:26 +0100

Hi,

Applied all three patches.  I added trivial tests for
‘spdx-string->license’ and changed ‘substring’ to ‘string-drop-right’,
which I find clearer.

Philip McGrath <[email-masked]> 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?

Thanks,
Ludo’.