Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/1] package/gstreamer1/gst1-libav: fix license
Date: Sat, 17 Oct 2020 11:37:58 +0200	[thread overview]
Message-ID: <20201017093758.GH3466@scaer> (raw)
In-Reply-To: <20201017092653.2605549-1-fontaine.fabrice@gmail.com>

Fabrice, All,

On 2020-10-17 11:26 +0200, Fabrice Fontaine spake thusly:
> Commit 15dc48ca9becafec6ba11b08af18453bc0e40c9e forgot to change
> the license which is LGPL-2.1+ since version 1.17.1 and
> https://gitlab.freedesktop.org/gstreamer/gst-libav/-/commit/c437e175ad9c82732ac46e36ccc398ad041bf13b
> 
> Fixes:
>  - http://autobuild.buildroot.org/results/848d7c49a01f719397cb2e8a8f8e3bd73bd03df4
> 
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> ---
>  package/gstreamer1/gst1-libav/gst1-libav.hash | 2 +-
>  package/gstreamer1/gst1-libav/gst1-libav.mk   | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/package/gstreamer1/gst1-libav/gst1-libav.hash b/package/gstreamer1/gst1-libav/gst1-libav.hash
> index 9a74465e09..9e4b9b1c27 100644
> --- a/package/gstreamer1/gst1-libav/gst1-libav.hash
> +++ b/package/gstreamer1/gst1-libav/gst1-libav.hash
> @@ -1,3 +1,3 @@
>  # From https://gstreamer.freedesktop.org/src/gst-libav/gst-libav-1.18.0.tar.xz.sha256sum
>  sha256  42f93f5ce9a3fc22051e5f783a4574b56ebf213f331f75dcbc3552459bd3a06a  gst-libav-1.18.0.tar.xz
> -sha256  8177f97513213526df2cf6184d8ff986c675afb514d4e68a404010521b880643  COPYING
> +sha256  f445dc78b88496f7e20c7a2a461b95baba5865c8919b8289ac24ac0a80c6ce7a  COPYING
> diff --git a/package/gstreamer1/gst1-libav/gst1-libav.mk b/package/gstreamer1/gst1-libav/gst1-libav.mk
> index 3fd23b8185..ba296844aa 100644
> --- a/package/gstreamer1/gst1-libav/gst1-libav.mk
> +++ b/package/gstreamer1/gst1-libav/gst1-libav.mk
> @@ -7,7 +7,7 @@
>  GST1_LIBAV_VERSION = 1.18.0
>  GST1_LIBAV_SOURCE = gst-libav-$(GST1_LIBAV_VERSION).tar.xz
>  GST1_LIBAV_SITE = https://gstreamer.freedesktop.org/src/gst-libav
> -GST1_LIBAV_LICENSE = GPL-2.0+
> +GST1_LIBAV_LICENSE = LGPL-2.1+

The upstream commit indeed mentions LGPL-2.1+, but the license text is
LGPL-2, and the soruce fiels mention LGPL-2+, e.g.:

    https://gitlab.freedesktop.org/gstreamer/gst-libav/-/blob/master/ext/libav/gstav.c
    https://gitlab.freedesktop.org/gstreamer/gst-libav/-/blob/master/ext/libav/gstavcfg.c
    https://gitlab.freedesktop.org/gstreamer/gst-libav/-/blob/master/ext/libav/gstavvidenc.c
    https://gitlab.freedesktop.org/gstreamer/gst-libav/-/blob/master/tests/check/elements/avaudenc.c

which all have:

     * This library is free software; you can redistribute it and/or
     * modify it under the terms of the GNU Library General Public
     * License as published by the Free Software Foundation; either
     * version 2 of the License, or (at your option) any later version.

Could you check further if/where LGPL-2.1+ is really applicalbe, please?

Maybe check with upstream whether they really meant 2+ in that commit?

Regards,
Yann E. MORIN.

>  GST1_LIBAV_LICENSE_FILES = COPYING
>  GST1_LIBAV_DEPENDENCIES =  host-pkgconf ffmpeg gstreamer1 gst1-plugins-base
>  GST1_LIBAV_CONF_OPTS = -Ddoc=disabled
> -- 
> 2.28.0
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

      reply	other threads:[~2020-10-17  9:37 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-17  9:26 [Buildroot] [PATCH 1/1] package/gstreamer1/gst1-libav: fix license Fabrice Fontaine
2020-10-17  9:37 ` Yann E. MORIN [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20201017093758.GH3466@scaer \
    --to=yann.morin.1998@free.fr \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox