diff mbox

[5/6] package/gnu-efi: fix legal-info

Message ID 41ea946a815b2214044827c73815d8a171a3a939.1428252467.git.yann.morin.1998@free.fr
State Changes Requested
Headers show

Commit Message

Yann E. MORIN April 5, 2015, 4:48 p.m. UTC
Fix a bunch of deficiencies:
  - 'debian/copyright' does not exist.
  - gnuefi is not GPLv2+. Only one AArch64 file has GPLv2+ headers,
    all others have BSD-3c
  - efilib is BSD-2c

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 package/gnu-efi/gnu-efi.mk | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Thomas Petazzoni April 5, 2015, 5:17 p.m. UTC | #1
Dear Yann E. MORIN,

On Sun,  5 Apr 2015 18:48:26 +0200, Yann E. MORIN wrote:
> Fix a bunch of deficiencies:
>   - 'debian/copyright' does not exist.
>   - gnuefi is not GPLv2+. Only one AArch64 file has GPLv2+ headers,
>     all others have BSD-3c
>   - efilib is BSD-2c

I don't quite agree here: some ARM files are also under GPLv2+, so I
believe the whole thing really is under GPLv2+.

This is also apparently the opinion of Debian:
http://changelogs.ubuntu.com/changelogs/pool/main/g/gnu-efi/gnu-efi_3.0i-2ubuntu1/copyright.

Thomas
Yann E. MORIN April 5, 2015, 5:31 p.m. UTC | #2
Thomas, All,

On 2015-04-05 19:17 +0200, Thomas Petazzoni spake thusly:
> On Sun,  5 Apr 2015 18:48:26 +0200, Yann E. MORIN wrote:
> > Fix a bunch of deficiencies:
> >   - 'debian/copyright' does not exist.
> >   - gnuefi is not GPLv2+. Only one AArch64 file has GPLv2+ headers,
> >     all others have BSD-3c
> >   - efilib is BSD-2c
> 
> I don't quite agree here: some ARM files are also under GPLv2+, so I
> believe the whole thing really is under GPLv2+.

The reasoning for changing that is:
  - there are two files with a GPLv2+ header;
    - gnuefi/crt0-efi-aarch64.S -> GPLv2+ or something like a BSD-2c
    - gnuefi/crt0-efi-arm.S     -> GPLv2+
  - all other files are BSD-3c (or have no licensing info)
  - we don't build gnuefi for ARM or AArch64, only for x86 or x86_64

However, re-reading README.gnuefi, we can read this:

---8<---
 gnuefi: This directory contains the glue necessary to convert ELF64
        binaries to EFI binaries.  Various runtime code bits, such as
        a self-relocator are included as well.  This code has been
        contributed by the Hewlett-Packard Company and is distributed
        under the GNU GPL.
---8<---

This is very confusing.Waht about changing to:

GNU_EFI_LICENSE = BSD-3c and/or GPLv2+ (gnuefi), BSD-3c (efilib)

> This is also apparently the opinion of Debian:
> http://changelogs.ubuntu.com/changelogs/pool/main/g/gnu-efi/gnu-efi_3.0i-2ubuntu1/copyright.

Well, I mostly disagree there, if only by looking at the files we *do*
build (others we don't care).

Regards,
Yann E. MORIN.
Thomas Petazzoni April 5, 2015, 5:36 p.m. UTC | #3
Dear Yann E. MORIN,

On Sun, 5 Apr 2015 19:31:34 +0200, Yann E. MORIN wrote:

> The reasoning for changing that is:
>   - there are two files with a GPLv2+ header;
>     - gnuefi/crt0-efi-aarch64.S -> GPLv2+ or something like a BSD-2c
>     - gnuefi/crt0-efi-arm.S     -> GPLv2+
>   - all other files are BSD-3c (or have no licensing info)
>   - we don't build gnuefi for ARM or AArch64, only for x86 or x86_64
> 
> However, re-reading README.gnuefi, we can read this:
> 
> ---8<---
>  gnuefi: This directory contains the glue necessary to convert ELF64
>         binaries to EFI binaries.  Various runtime code bits, such as
>         a self-relocator are included as well.  This code has been
>         contributed by the Hewlett-Packard Company and is distributed
>         under the GNU GPL.
> ---8<---
> 
> This is very confusing.Waht about changing to:
> 
> GNU_EFI_LICENSE = BSD-3c and/or GPLv2+ (gnuefi), BSD-3c (efilib)

I would find this to be a safer value.

Thomas
Yann E. MORIN April 5, 2015, 5:40 p.m. UTC | #4
Thomas, All,

On 2015-04-05 19:36 +0200, Thomas Petazzoni spake thusly:
> On Sun, 5 Apr 2015 19:31:34 +0200, Yann E. MORIN wrote:
> > The reasoning for changing that is:
> >   - there are two files with a GPLv2+ header;
> >     - gnuefi/crt0-efi-aarch64.S -> GPLv2+ or something like a BSD-2c
> >     - gnuefi/crt0-efi-arm.S     -> GPLv2+
> >   - all other files are BSD-3c (or have no licensing info)
> >   - we don't build gnuefi for ARM or AArch64, only for x86 or x86_64
> > 
> > However, re-reading README.gnuefi, we can read this:
> > 
> > ---8<---
> >  gnuefi: This directory contains the glue necessary to convert ELF64
> >         binaries to EFI binaries.  Various runtime code bits, such as
> >         a self-relocator are included as well.  This code has been
> >         contributed by the Hewlett-Packard Company and is distributed
> >         under the GNU GPL.
> > ---8<---
> > 
> > This is very confusing.Waht about changing to:
> > 
> > GNU_EFI_LICENSE = BSD-3c and/or GPLv2+ (gnuefi), BSD-3c (efilib)
> 
> I would find this to be a safer value.

OK, I'll respin.

Thanks!

Regards,
Yann E. MORIN.
diff mbox

Patch

diff --git a/package/gnu-efi/gnu-efi.mk b/package/gnu-efi/gnu-efi.mk
index 0e067a6..cdbfe8e 100644
--- a/package/gnu-efi/gnu-efi.mk
+++ b/package/gnu-efi/gnu-efi.mk
@@ -8,8 +8,8 @@  GNU_EFI_VERSION = 3.0.1
 GNU_EFI_SOURCE = gnu-efi-$(GNU_EFI_VERSION).tar.bz2
 GNU_EFI_SITE = http://downloads.sourceforge.net/project/gnu-efi
 GNU_EFI_INSTALL_STAGING = YES
-GNU_EFI_LICENSE = GPLv2+ (gnuefi), BSD (efilib)
-GNU_EFI_LICENSE_FILES = debian/copyright
+GNU_EFI_LICENSE = BSD-3c (gnuefi), BSD-2c (efilib)
+GNU_EFI_LICENSE_FILES = README.efilib
 
 # gnu-efi is a set of library and header files used to build
 # standalone EFI applications such as bootloaders. There is no point