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

Previous Next

Package: guix-patches;

Reported by: Philip McGrath <philip <at> philipmcgrath.com>

Date: Wed, 19 Oct 2022 04:56:01 UTC

Severity: normal

Tags: patch

Done: Ludovic Courtès <ludo <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


Message #22 received at 58621-done <at> debbugs.gnu.org (full text, mbox):

From: Philip McGrath <philip <at> philipmcgrath.com>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: 58621-done <at> debbugs.gnu.org
Subject: Re: bug#58621: [PATCH 0/3] import/utils: spdx-string->license: Match
 case-insensitively and support '+' operator.
Date: Fri, 18 Nov 2022 15:21:52 -0500
[Message part 1 (text/plain, inline)]
Hi,

On Friday, November 18, 2022 8:45:26 AM EST Ludovic Courtès wrote:
> Hi,
> 
> Applied all three patches.  I added trivial tests for
> ‘spdx-string->license’ and changed ‘substring’ to ‘string-drop-right’,
> which I find clearer.
> 

Thanks!

> Philip McGrath <philip <at> 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".

Some upstreams probably wrote "GPL-2.0" out of confusion even though they 
intended to allow "any later version" (and maybe even said so in prose), which 
is why the "+" operator was deprecated for GNU licenses.

If upstream wrote "GPL-2.0+", though, that does communicate "or, at your 
option, any later version"; since that's the more compatible case, it seemed 
useful to retain that information when we have it.

Ideally, upstream projects should move away from the deprecated identifiers 
(the Racket tooling I've written will complain), but they still seem to come 
up often enough in the wild to be worth handling as special cases.

-Philip
[signature.asc (application/pgp-signature, inline)]

This bug report was last modified 2 years and 247 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.