rpm: convert license to SPDX format

[libvirt PATCH] rpm: convert license to SPDX format

Posted by Daniel P. Berrangé 1 year, 3 months ago

Note that with the introduction of SPDX, Fedora no longer wants
maintainers to do effective license analysis, hence we no list
all the licenses that are applicable to the binary package
contents

  https://docs.fedoraproject.org/en-US/legal/license-field/#_no_effective_license_analysis

Note, we can still omit licenses that are only applicable to
the build system.

Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
 libvirt.spec.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libvirt.spec.in b/libvirt.spec.in
index 77b274b6f9..afd0b04f99 100644
--- a/libvirt.spec.in
+++ b/libvirt.spec.in
@@ -230,7 +230,7 @@ Summary: Library providing a simple virtualization API
 Name: libvirt
 Version: @VERSION@
 Release: 1%{?dist}
-License: LGPLv2+
+License: GPL-2.0-or-later and LGPL-2.1-only and LGPL-2.1-or-later and OFL-1.1
 URL: https://libvirt.org/
 
 %if %(echo %{version} | grep -q "\.0$"; echo $?) == 1
-- 
2.39.0

Re: [libvirt PATCH] rpm: convert license to SPDX format

Posted by Peter Krempa 1 year, 3 months ago

On Mon, Jan 16, 2023 at 08:19:28 -0500, Daniel P. Berrangé wrote:
> Note that with the introduction of SPDX, Fedora no longer wants
> maintainers to do effective license analysis, hence we no list

*we now list

> all the licenses that are applicable to the binary package
> contents
> 
>   https://docs.fedoraproject.org/en-US/legal/license-field/#_no_effective_license_analysis

In examples in this page ...

> 
> Note, we can still omit licenses that are only applicable to
> the build system.
> 
> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> ---
>  libvirt.spec.in | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libvirt.spec.in b/libvirt.spec.in
> index 77b274b6f9..afd0b04f99 100644
> --- a/libvirt.spec.in
> +++ b/libvirt.spec.in
> @@ -230,7 +230,7 @@ Summary: Library providing a simple virtualization API
>  Name: libvirt
>  Version: @VERSION@
>  Release: 1%{?dist}
> -License: LGPLv2+
> +License: GPL-2.0-or-later and LGPL-2.1-only and LGPL-2.1-or-later and OFL-1.1

... the logical operators are spelled all-caps.

>  URL: https://libvirt.org/

Reviewed-by: Peter Krempa <pkrempa@redhat.com>

Re: [libvirt PATCH] rpm: convert license to SPDX format

Posted by Daniel P. Berrangé 1 year, 3 months ago

On Mon, Jan 16, 2023 at 02:30:51PM +0100, Peter Krempa wrote:
> On Mon, Jan 16, 2023 at 08:19:28 -0500, Daniel P. Berrangé wrote:
> > Note that with the introduction of SPDX, Fedora no longer wants
> > maintainers to do effective license analysis, hence we no list
> 
> *we now list
> 
> > all the licenses that are applicable to the binary package
> > contents
> > 
> >   https://docs.fedoraproject.org/en-US/legal/license-field/#_no_effective_license_analysis
> 
> In examples in this page ...
> 
> > 
> > Note, we can still omit licenses that are only applicable to
> > the build system.
> > 
> > Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> > ---
> >  libvirt.spec.in | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/libvirt.spec.in b/libvirt.spec.in
> > index 77b274b6f9..afd0b04f99 100644
> > --- a/libvirt.spec.in
> > +++ b/libvirt.spec.in
> > @@ -230,7 +230,7 @@ Summary: Library providing a simple virtualization API
> >  Name: libvirt
> >  Version: @VERSION@
> >  Release: 1%{?dist}
> > -License: LGPLv2+
> > +License: GPL-2.0-or-later and LGPL-2.1-only and LGPL-2.1-or-later and OFL-1.1
> 
> ... the logical operators are spelled all-caps.

Yes, that's actually normal practice, so I'll change this.

> 
> >  URL: https://libvirt.org/
> 
> Reviewed-by: Peter Krempa <pkrempa@redhat.com>
> 

With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|