Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/6] gstreamer: add legal info
@ 2015-11-18 12:39 gustavo.zacarias at free-electrons.com
  2015-11-18 12:39 ` [Buildroot] [PATCH 2/6] gst-plugins-base: " gustavo.zacarias at free-electrons.com
                   ` (5 more replies)
  0 siblings, 6 replies; 15+ messages in thread
From: gustavo.zacarias at free-electrons.com @ 2015-11-18 12:39 UTC (permalink / raw)
  To: buildroot

From: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>

Even though COPYING is LGPLv2+ some code files state v2.1+

Signed-off-by: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>
---
 package/gstreamer/gstreamer/gstreamer.mk | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/package/gstreamer/gstreamer/gstreamer.mk b/package/gstreamer/gstreamer/gstreamer.mk
index 1dbd66f..156afab 100644
--- a/package/gstreamer/gstreamer/gstreamer.mk
+++ b/package/gstreamer/gstreamer/gstreamer.mk
@@ -8,6 +8,8 @@ GSTREAMER_VERSION = 0.10.36
 GSTREAMER_SOURCE = gstreamer-$(GSTREAMER_VERSION).tar.xz
 GSTREAMER_SITE = http://gstreamer.freedesktop.org/src/gstreamer
 GSTREAMER_INSTALL_STAGING = YES
+GSTREAMER_LICENSE = LGPLv2+ LGPLv2.1+
+GSTREAMER_LICENSE_FILES = COPYING
 
 # Checking if unaligned memory access works correctly cannot be done when cross
 # compiling. For the following architectures there is no information available
-- 
2.4.10

^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [Buildroot] [PATCH 2/6] gst-plugins-base: add legal info
  2015-11-18 12:39 [Buildroot] [PATCH 1/6] gstreamer: add legal info gustavo.zacarias at free-electrons.com
@ 2015-11-18 12:39 ` gustavo.zacarias at free-electrons.com
  2015-11-18 21:49   ` Arnout Vandecappelle
  2015-11-18 12:39 ` [Buildroot] [PATCH 3/6] gst-plugins-good: " gustavo.zacarias at free-electrons.com
                   ` (4 subsequent siblings)
  5 siblings, 1 reply; 15+ messages in thread
From: gustavo.zacarias at free-electrons.com @ 2015-11-18 12:39 UTC (permalink / raw)
  To: buildroot

From: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>

Signed-off-by: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>
---
 package/gstreamer/gst-plugins-base/gst-plugins-base.mk | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/package/gstreamer/gst-plugins-base/gst-plugins-base.mk b/package/gstreamer/gst-plugins-base/gst-plugins-base.mk
index 08733d1..43e4df0 100644
--- a/package/gstreamer/gst-plugins-base/gst-plugins-base.mk
+++ b/package/gstreamer/gst-plugins-base/gst-plugins-base.mk
@@ -8,6 +8,8 @@ GST_PLUGINS_BASE_VERSION = 0.10.36
 GST_PLUGINS_BASE_SOURCE = gst-plugins-base-$(GST_PLUGINS_BASE_VERSION).tar.xz
 GST_PLUGINS_BASE_SITE = http://gstreamer.freedesktop.org/src/gst-plugins-base
 GST_PLUGINS_BASE_INSTALL_STAGING = YES
+GST_PLUGINS_BASE_LICENSE = GPLv2+ LGPLv2+
+GST_PLUGINS_BASE_LICENSE_FILES = COPYING COPYING.LIB
 
 # freetype is only used by examples, but if it is not found
 # and the host has a freetype-config script, then the host
-- 
2.4.10

^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [Buildroot] [PATCH 3/6] gst-plugins-good: add legal info
  2015-11-18 12:39 [Buildroot] [PATCH 1/6] gstreamer: add legal info gustavo.zacarias at free-electrons.com
  2015-11-18 12:39 ` [Buildroot] [PATCH 2/6] gst-plugins-base: " gustavo.zacarias at free-electrons.com
@ 2015-11-18 12:39 ` gustavo.zacarias at free-electrons.com
  2015-11-18 21:53   ` Arnout Vandecappelle
  2015-11-18 12:39 ` [Buildroot] [PATCH 4/6] gst-plugins-bad: " gustavo.zacarias at free-electrons.com
                   ` (3 subsequent siblings)
  5 siblings, 1 reply; 15+ messages in thread
From: gustavo.zacarias at free-electrons.com @ 2015-11-18 12:39 UTC (permalink / raw)
  To: buildroot

From: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>

Signed-off-by: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>
---
 package/gstreamer/gst-plugins-good/gst-plugins-good.mk | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/package/gstreamer/gst-plugins-good/gst-plugins-good.mk b/package/gstreamer/gst-plugins-good/gst-plugins-good.mk
index fee83a1..a686caf 100644
--- a/package/gstreamer/gst-plugins-good/gst-plugins-good.mk
+++ b/package/gstreamer/gst-plugins-good/gst-plugins-good.mk
@@ -7,6 +7,8 @@
 GST_PLUGINS_GOOD_VERSION = 0.10.31
 GST_PLUGINS_GOOD_SOURCE = gst-plugins-good-$(GST_PLUGINS_GOOD_VERSION).tar.xz
 GST_PLUGINS_GOOD_SITE = http://gstreamer.freedesktop.org/src/gst-plugins-good
+GST_PLUGINS_GOOD_LICENSE = LGPLv2.1+
+GST_PLUGINS_GOOD_LICENSE_FILES = COPYING
 
 GST_PLUGINS_GOOD_CONF_OPTS = \
 	--disable-examples \
-- 
2.4.10

^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [Buildroot] [PATCH 4/6] gst-plugins-bad: add legal info
  2015-11-18 12:39 [Buildroot] [PATCH 1/6] gstreamer: add legal info gustavo.zacarias at free-electrons.com
  2015-11-18 12:39 ` [Buildroot] [PATCH 2/6] gst-plugins-base: " gustavo.zacarias at free-electrons.com
  2015-11-18 12:39 ` [Buildroot] [PATCH 3/6] gst-plugins-good: " gustavo.zacarias at free-electrons.com
@ 2015-11-18 12:39 ` gustavo.zacarias at free-electrons.com
  2015-11-18 22:03   ` Arnout Vandecappelle
  2015-11-18 12:39 ` [Buildroot] [PATCH 5/6] gst-plugins-ugly: " gustavo.zacarias at free-electrons.com
                   ` (2 subsequent siblings)
  5 siblings, 1 reply; 15+ messages in thread
From: gustavo.zacarias at free-electrons.com @ 2015-11-18 12:39 UTC (permalink / raw)
  To: buildroot

From: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>

Even though COPYING.LIB is LGPLv2+ some code files state v2.1+

Signed-off-by: Gustavo Zacarias <gustavo@zacarias@free-electrons.com>
---
 package/gstreamer/gst-plugins-bad/gst-plugins-bad.mk | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/package/gstreamer/gst-plugins-bad/gst-plugins-bad.mk b/package/gstreamer/gst-plugins-bad/gst-plugins-bad.mk
index 0a85603..28bcc1d 100644
--- a/package/gstreamer/gst-plugins-bad/gst-plugins-bad.mk
+++ b/package/gstreamer/gst-plugins-bad/gst-plugins-bad.mk
@@ -7,6 +7,8 @@
 GST_PLUGINS_BAD_VERSION = 0.10.23
 GST_PLUGINS_BAD_SOURCE = gst-plugins-bad-$(GST_PLUGINS_BAD_VERSION).tar.xz
 GST_PLUGINS_BAD_SITE = http://gstreamer.freedesktop.org/src/gst-plugins-bad
+GST_PLUGINS_BAD_LICENSE = GPLv2+ LGPLv2+ LGPLv2.1+
+GST_PLUGINS_BAD_LICENSE_FILES = COPYING COPYING.LIB
 
 GST_PLUGINS_BAD_CONF_OPTS = \
 	--disable-examples
-- 
2.4.10

^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [Buildroot] [PATCH 5/6] gst-plugins-ugly: add legal info
  2015-11-18 12:39 [Buildroot] [PATCH 1/6] gstreamer: add legal info gustavo.zacarias at free-electrons.com
                   ` (2 preceding siblings ...)
  2015-11-18 12:39 ` [Buildroot] [PATCH 4/6] gst-plugins-bad: " gustavo.zacarias at free-electrons.com
@ 2015-11-18 12:39 ` gustavo.zacarias at free-electrons.com
  2015-11-18 22:12   ` Arnout Vandecappelle
  2015-11-18 12:39 ` [Buildroot] [PATCH 6/6] gst-ffmpeg: " gustavo.zacarias at free-electrons.com
  2015-11-18 21:40 ` [Buildroot] [PATCH 1/6] gstreamer: " Arnout Vandecappelle
  5 siblings, 1 reply; 15+ messages in thread
From: gustavo.zacarias at free-electrons.com @ 2015-11-18 12:39 UTC (permalink / raw)
  To: buildroot

From: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>

Signed-off-by: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>
---
 package/gstreamer/gst-plugins-ugly/gst-plugins-ugly.mk | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/package/gstreamer/gst-plugins-ugly/gst-plugins-ugly.mk b/package/gstreamer/gst-plugins-ugly/gst-plugins-ugly.mk
index e8f2362..9662623 100644
--- a/package/gstreamer/gst-plugins-ugly/gst-plugins-ugly.mk
+++ b/package/gstreamer/gst-plugins-ugly/gst-plugins-ugly.mk
@@ -7,6 +7,8 @@
 GST_PLUGINS_UGLY_VERSION = 0.10.19
 GST_PLUGINS_UGLY_SOURCE = gst-plugins-ugly-$(GST_PLUGINS_UGLY_VERSION).tar.xz
 GST_PLUGINS_UGLY_SITE = http://gstreamer.freedesktop.org/src/gst-plugins-ugly
+GST_PLUGINS_UGLY_LICENSE = LGPLv2.1+
+GST_PLUGINS_UGLY_LICENSE_FILES = COPYING
 
 GST_PLUGINS_UGLY_CONF_OPTS = \
 	--disable-examples
-- 
2.4.10

^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [Buildroot] [PATCH 6/6] gst-ffmpeg: add legal info
  2015-11-18 12:39 [Buildroot] [PATCH 1/6] gstreamer: add legal info gustavo.zacarias at free-electrons.com
                   ` (3 preceding siblings ...)
  2015-11-18 12:39 ` [Buildroot] [PATCH 5/6] gst-plugins-ugly: " gustavo.zacarias at free-electrons.com
@ 2015-11-18 12:39 ` gustavo.zacarias at free-electrons.com
  2015-11-18 22:15   ` Arnout Vandecappelle
  2015-11-18 21:40 ` [Buildroot] [PATCH 1/6] gstreamer: " Arnout Vandecappelle
  5 siblings, 1 reply; 15+ messages in thread
From: gustavo.zacarias at free-electrons.com @ 2015-11-18 12:39 UTC (permalink / raw)
  To: buildroot

From: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>

Signed-off-by: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>
---
 package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk b/package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk
index b1d8f8b..b728128 100644
--- a/package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk
+++ b/package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk
@@ -9,6 +9,8 @@ GST_FFMPEG_SOURCE = gst-ffmpeg-$(GST_FFMPEG_VERSION).tar.bz2
 GST_FFMPEG_SITE = http://gstreamer.freedesktop.org/src/gst-ffmpeg
 GST_FFMPEG_INSTALL_STAGING = YES
 GST_FFMPEG_DEPENDENCIES = host-pkgconf gstreamer gst-plugins-base
+GST_FFMPEG_LICENSE = GPLv2+ LGPLv2+ LGPLv2.1+
+GST_FFMPEG_LICENSE_FILES = COPYING COPYING.LIB
 
 ifeq ($(BR2_PACKAGE_GST_FFMPEG_GPL),y)
 GST_FFMPEG_CONF_OPTS += --disable-lgpl
-- 
2.4.10

^ permalink raw reply related	[flat|nested] 15+ messages in thread

* [Buildroot] [PATCH 1/6] gstreamer: add legal info
  2015-11-18 12:39 [Buildroot] [PATCH 1/6] gstreamer: add legal info gustavo.zacarias at free-electrons.com
                   ` (4 preceding siblings ...)
  2015-11-18 12:39 ` [Buildroot] [PATCH 6/6] gst-ffmpeg: " gustavo.zacarias at free-electrons.com
@ 2015-11-18 21:40 ` Arnout Vandecappelle
  5 siblings, 0 replies; 15+ messages in thread
From: Arnout Vandecappelle @ 2015-11-18 21:40 UTC (permalink / raw)
  To: buildroot

On 18-11-15 13:39, gustavo.zacarias at free-electrons.com wrote:
> From: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>
> 
> Even though COPYING is LGPLv2+ some code files state v2.1+

 Correct.

> 
> Signed-off-by: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>
> ---
>  package/gstreamer/gstreamer/gstreamer.mk | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/package/gstreamer/gstreamer/gstreamer.mk b/package/gstreamer/gstreamer/gstreamer.mk
> index 1dbd66f..156afab 100644
> --- a/package/gstreamer/gstreamer/gstreamer.mk
> +++ b/package/gstreamer/gstreamer/gstreamer.mk
> @@ -8,6 +8,8 @@ GSTREAMER_VERSION = 0.10.36
>  GSTREAMER_SOURCE = gstreamer-$(GSTREAMER_VERSION).tar.xz
>  GSTREAMER_SITE = http://gstreamer.freedesktop.org/src/gstreamer
>  GSTREAMER_INSTALL_STAGING = YES
> +GSTREAMER_LICENSE = LGPLv2+ LGPLv2.1+

 We separate these things by commas. Otherwise looks good so
 Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>

 I was thinking, why not simply claim it is LGPLv2.1+, given that it's + anyway?
The binaries will anyway need to be LGPLv2.1+ because there's always some 2.1
code in there. So I would actually propose

# Some files are LGPLv2+, but the whole will be 2.1+
GSTREAMER_LICENSE = LGPLv2.1+
# COPYING file is actually for v2
GSTREAMER_LICENSE_FILES = COPYING

 Regards,
 Arnout

> +GSTREAMER_LICENSE_FILES = COPYING
>  
>  # Checking if unaligned memory access works correctly cannot be done when cross
>  # compiling. For the following architectures there is no information available
> 


-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [Buildroot] [PATCH 2/6] gst-plugins-base: add legal info
  2015-11-18 12:39 ` [Buildroot] [PATCH 2/6] gst-plugins-base: " gustavo.zacarias at free-electrons.com
@ 2015-11-18 21:49   ` Arnout Vandecappelle
  0 siblings, 0 replies; 15+ messages in thread
From: Arnout Vandecappelle @ 2015-11-18 21:49 UTC (permalink / raw)
  To: buildroot

On 18-11-15 13:39, gustavo.zacarias at free-electrons.com wrote:
> From: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>
> 
> Signed-off-by: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>
> ---
>  package/gstreamer/gst-plugins-base/gst-plugins-base.mk | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/package/gstreamer/gst-plugins-base/gst-plugins-base.mk b/package/gstreamer/gst-plugins-base/gst-plugins-base.mk
> index 08733d1..43e4df0 100644
> --- a/package/gstreamer/gst-plugins-base/gst-plugins-base.mk
> +++ b/package/gstreamer/gst-plugins-base/gst-plugins-base.mk
> @@ -8,6 +8,8 @@ GST_PLUGINS_BASE_VERSION = 0.10.36
>  GST_PLUGINS_BASE_SOURCE = gst-plugins-base-$(GST_PLUGINS_BASE_VERSION).tar.xz
>  GST_PLUGINS_BASE_SITE = http://gstreamer.freedesktop.org/src/gst-plugins-base
>  GST_PLUGINS_BASE_INSTALL_STAGING = YES
> +GST_PLUGINS_BASE_LICENSE = GPLv2+ LGPLv2+

 comma-separated, otherwise

 Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>

> +GST_PLUGINS_BASE_LICENSE_FILES = COPYING COPYING.LIB
>  
>  # freetype is only used by examples, but if it is not found
>  # and the host has a freetype-config script, then the host
> 


-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [Buildroot] [PATCH 3/6] gst-plugins-good: add legal info
  2015-11-18 12:39 ` [Buildroot] [PATCH 3/6] gst-plugins-good: " gustavo.zacarias at free-electrons.com
@ 2015-11-18 21:53   ` Arnout Vandecappelle
  0 siblings, 0 replies; 15+ messages in thread
From: Arnout Vandecappelle @ 2015-11-18 21:53 UTC (permalink / raw)
  To: buildroot

On 18-11-15 13:39, gustavo.zacarias at free-electrons.com wrote:
> From: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>
> 
> Signed-off-by: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>
> ---
>  package/gstreamer/gst-plugins-good/gst-plugins-good.mk | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/package/gstreamer/gst-plugins-good/gst-plugins-good.mk b/package/gstreamer/gst-plugins-good/gst-plugins-good.mk
> index fee83a1..a686caf 100644
> --- a/package/gstreamer/gst-plugins-good/gst-plugins-good.mk
> +++ b/package/gstreamer/gst-plugins-good/gst-plugins-good.mk
> @@ -7,6 +7,8 @@
>  GST_PLUGINS_GOOD_VERSION = 0.10.31
>  GST_PLUGINS_GOOD_SOURCE = gst-plugins-good-$(GST_PLUGINS_GOOD_VERSION).tar.xz
>  GST_PLUGINS_GOOD_SITE = http://gstreamer.freedesktop.org/src/gst-plugins-good
> +GST_PLUGINS_GOOD_LICENSE = LGPLv2.1+

 Here there are also some files marked as LGPLv2+, but as I commented on patch
1, it may be better like that. So

Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>


 Regards,
 Arnout

> +GST_PLUGINS_GOOD_LICENSE_FILES = COPYING
>  
>  GST_PLUGINS_GOOD_CONF_OPTS = \
>  	--disable-examples \
> 


-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [Buildroot] [PATCH 4/6] gst-plugins-bad: add legal info
  2015-11-18 12:39 ` [Buildroot] [PATCH 4/6] gst-plugins-bad: " gustavo.zacarias at free-electrons.com
@ 2015-11-18 22:03   ` Arnout Vandecappelle
  2015-11-19 22:08     ` Gustavo Zacarias
  0 siblings, 1 reply; 15+ messages in thread
From: Arnout Vandecappelle @ 2015-11-18 22:03 UTC (permalink / raw)
  To: buildroot

On 18-11-15 13:39, gustavo.zacarias at free-electrons.com wrote:
> From: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>
> 
> Even though COPYING.LIB is LGPLv2+ some code files state v2.1+
> 
> Signed-off-by: Gustavo Zacarias <gustavo@zacarias@free-electrons.com>
> ---
>  package/gstreamer/gst-plugins-bad/gst-plugins-bad.mk | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/package/gstreamer/gst-plugins-bad/gst-plugins-bad.mk b/package/gstreamer/gst-plugins-bad/gst-plugins-bad.mk
> index 0a85603..28bcc1d 100644
> --- a/package/gstreamer/gst-plugins-bad/gst-plugins-bad.mk
> +++ b/package/gstreamer/gst-plugins-bad/gst-plugins-bad.mk
> @@ -7,6 +7,8 @@
>  GST_PLUGINS_BAD_VERSION = 0.10.23
>  GST_PLUGINS_BAD_SOURCE = gst-plugins-bad-$(GST_PLUGINS_BAD_VERSION).tar.xz
>  GST_PLUGINS_BAD_SITE = http://gstreamer.freedesktop.org/src/gst-plugins-bad
> +GST_PLUGINS_BAD_LICENSE = GPLv2+ LGPLv2+ LGPLv2.1+

 comma-separated, I would drop the v2, I would also put LGPL first because it is
dominant, and perhaps explicitly state which ones are GPLv2+:

GST_PLUGINS_BAD_LICENSE = LGPLv2.1+, GPLv2+ (debugutils,mpegpsmux, mpegtsmux, \
	librfb,  audiovisualizers, tta, dvb, vdpau)

but maybe that's too much...

 Regards,
 Arnout

> +GST_PLUGINS_BAD_LICENSE_FILES = COPYING COPYING.LIB
>  
>  GST_PLUGINS_BAD_CONF_OPTS = \
>  	--disable-examples
> 


-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [Buildroot] [PATCH 5/6] gst-plugins-ugly: add legal info
  2015-11-18 12:39 ` [Buildroot] [PATCH 5/6] gst-plugins-ugly: " gustavo.zacarias at free-electrons.com
@ 2015-11-18 22:12   ` Arnout Vandecappelle
  2015-11-19 22:06     ` Gustavo Zacarias
  0 siblings, 1 reply; 15+ messages in thread
From: Arnout Vandecappelle @ 2015-11-18 22:12 UTC (permalink / raw)
  To: buildroot

On 18-11-15 13:39, gustavo.zacarias at free-electrons.com wrote:
> From: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>
> 
> Signed-off-by: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>
> ---
>  package/gstreamer/gst-plugins-ugly/gst-plugins-ugly.mk | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/package/gstreamer/gst-plugins-ugly/gst-plugins-ugly.mk b/package/gstreamer/gst-plugins-ugly/gst-plugins-ugly.mk
> index e8f2362..9662623 100644
> --- a/package/gstreamer/gst-plugins-ugly/gst-plugins-ugly.mk
> +++ b/package/gstreamer/gst-plugins-ugly/gst-plugins-ugly.mk
> @@ -7,6 +7,8 @@
>  GST_PLUGINS_UGLY_VERSION = 0.10.19
>  GST_PLUGINS_UGLY_SOURCE = gst-plugins-ugly-$(GST_PLUGINS_UGLY_VERSION).tar.xz
>  GST_PLUGINS_UGLY_SITE = http://gstreamer.freedesktop.org/src/gst-plugins-ugly
> +GST_PLUGINS_UGLY_LICENSE = LGPLv2.1+

 synaesthesia is GPLv2+ (and is always enabled)

 All the rest is LGPLv2+...

> +GST_PLUGINS_UGLY_LICENSE_FILES = COPYING

 ...except this file which is 2.1...

 I guess there's a reason it's called ugly :-)


 Regards,
 Arnout

>  
>  GST_PLUGINS_UGLY_CONF_OPTS = \
>  	--disable-examples
> 


-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [Buildroot] [PATCH 6/6] gst-ffmpeg: add legal info
  2015-11-18 12:39 ` [Buildroot] [PATCH 6/6] gst-ffmpeg: " gustavo.zacarias at free-electrons.com
@ 2015-11-18 22:15   ` Arnout Vandecappelle
  0 siblings, 0 replies; 15+ messages in thread
From: Arnout Vandecappelle @ 2015-11-18 22:15 UTC (permalink / raw)
  To: buildroot

On 18-11-15 13:39, gustavo.zacarias at free-electrons.com wrote:
> From: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>
> 
> Signed-off-by: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>

 NACK. Marked as rejected
> ---
>  package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk b/package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk
> index b1d8f8b..b728128 100644
> --- a/package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk
> +++ b/package/gstreamer/gst-ffmpeg/gst-ffmpeg.mk
> @@ -9,6 +9,8 @@ GST_FFMPEG_SOURCE = gst-ffmpeg-$(GST_FFMPEG_VERSION).tar.bz2
>  GST_FFMPEG_SITE = http://gstreamer.freedesktop.org/src/gst-ffmpeg
>  GST_FFMPEG_INSTALL_STAGING = YES
>  GST_FFMPEG_DEPENDENCIES = host-pkgconf gstreamer gst-plugins-base
> +GST_FFMPEG_LICENSE = GPLv2+ LGPLv2+ LGPLv2.1+
> +GST_FFMPEG_LICENSE_FILES = COPYING COPYING.LIB
>  
>  ifeq ($(BR2_PACKAGE_GST_FFMPEG_GPL),y)
>  GST_FFMPEG_CONF_OPTS += --disable-lgpl

 The license is already specified in the lines just below:

ifeq ($(BR2_PACKAGE_GST_FFMPEG_GPL),y)
GST_FFMPEG_CONF_OPTS += --disable-lgpl
GST_FFMPEG_LICENSE = GPLv2+ (gst-ffmpeg), GPLv2+/GPLv3+ (libav)
GST_FFMPEG_LICENSE_FILES = COPYING gst-libs/ext/libav/COPYING.GPLv2
gst-libs/ext/libav/COPYING.GPLv3
else
GST_FFMPEG_CONF_OPTS += --enable-lgpl
GST_FFMPEG_LICENSE = LGPLv2+ (gst-ffmpeg), LGPLv2.1+/LGPLv3+ (libav)
GST_FFMPEG_LICENSE_FILES = COPYING.LIB gst-libs/ext/libav/COPYING.LGPLv2.1
gst-libs/ext/libav/COPYING.LGPLv3
endif


 Regards,
 Arnout



-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [Buildroot] [PATCH 5/6] gst-plugins-ugly: add legal info
  2015-11-18 22:12   ` Arnout Vandecappelle
@ 2015-11-19 22:06     ` Gustavo Zacarias
  2015-11-20 21:27       ` Arnout Vandecappelle
  0 siblings, 1 reply; 15+ messages in thread
From: Gustavo Zacarias @ 2015-11-19 22:06 UTC (permalink / raw)
  To: buildroot

On 18/11/15 19:12, Arnout Vandecappelle wrote:

> On 18-11-15 13:39, gustavo.zacarias at free-electrons.com wrote:
>> From: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>
>>
>> Signed-off-by: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>
>> ---
>>   package/gstreamer/gst-plugins-ugly/gst-plugins-ugly.mk | 2 ++
>>   1 file changed, 2 insertions(+)
>>
>> diff --git a/package/gstreamer/gst-plugins-ugly/gst-plugins-ugly.mk b/package/gstreamer/gst-plugins-ugly/gst-plugins-ugly.mk
>> index e8f2362..9662623 100644
>> --- a/package/gstreamer/gst-plugins-ugly/gst-plugins-ugly.mk
>> +++ b/package/gstreamer/gst-plugins-ugly/gst-plugins-ugly.mk
>> @@ -7,6 +7,8 @@
>>   GST_PLUGINS_UGLY_VERSION = 0.10.19
>>   GST_PLUGINS_UGLY_SOURCE = gst-plugins-ugly-$(GST_PLUGINS_UGLY_VERSION).tar.xz
>>   GST_PLUGINS_UGLY_SITE = http://gstreamer.freedesktop.org/src/gst-plugins-ugly
>> +GST_PLUGINS_UGLY_LICENSE = LGPLv2.1+
>
>   synaesthesia is GPLv2+ (and is always enabled)
>
>   All the rest is LGPLv2+...
>
>> +GST_PLUGINS_UGLY_LICENSE_FILES = COPYING
>
>   ...except this file which is 2.1...
>
>   I guess there's a reason it's called ugly :-)
>
>
>   Regards,
>   Arnout

Hi Arnout.
Actually it's more ugly, in fact tests/check/elements/xingmux.c is 
LGPLv2.1+ :)
So i'm going for:

# COPYING is in fact LGPLv2.1, but all of the code is v2+
# (except for one test, xingmux)
GST_PLUGINS_UGLY_LICENSE = LGPLv2+, GPLv2+ (synaesthesia)

Since in general we don't ship/use tests.
Regards.

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [Buildroot] [PATCH 4/6] gst-plugins-bad: add legal info
  2015-11-18 22:03   ` Arnout Vandecappelle
@ 2015-11-19 22:08     ` Gustavo Zacarias
  0 siblings, 0 replies; 15+ messages in thread
From: Gustavo Zacarias @ 2015-11-19 22:08 UTC (permalink / raw)
  To: buildroot

On 18/11/15 19:03, Arnout Vandecappelle wrote:

>   comma-separated, I would drop the v2, I would also put LGPL first because it is
> dominant, and perhaps explicitly state which ones are GPLv2+:
>
> GST_PLUGINS_BAD_LICENSE = LGPLv2.1+, GPLv2+ (debugutils,mpegpsmux, mpegtsmux, \
> 	librfb,  audiovisualizers, tta, dvb, vdpau)
>
> but maybe that's too much...
>
>   Regards,
>   Arnout

Hi Arnout.
ACK, though i'll skip the gory details and go for simple.
Regards.

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [Buildroot] [PATCH 5/6] gst-plugins-ugly: add legal info
  2015-11-19 22:06     ` Gustavo Zacarias
@ 2015-11-20 21:27       ` Arnout Vandecappelle
  0 siblings, 0 replies; 15+ messages in thread
From: Arnout Vandecappelle @ 2015-11-20 21:27 UTC (permalink / raw)
  To: buildroot

On 19-11-15 23:06, Gustavo Zacarias wrote:
> On 18/11/15 19:12, Arnout Vandecappelle wrote:
> 
>> On 18-11-15 13:39, gustavo.zacarias at free-electrons.com wrote:
>>> From: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>
>>>
>>> Signed-off-by: Gustavo Zacarias <gustavo.zacarias@free-electrons.com>
>>> ---
>>>   package/gstreamer/gst-plugins-ugly/gst-plugins-ugly.mk | 2 ++
>>>   1 file changed, 2 insertions(+)
>>>
>>> diff --git a/package/gstreamer/gst-plugins-ugly/gst-plugins-ugly.mk
>>> b/package/gstreamer/gst-plugins-ugly/gst-plugins-ugly.mk
>>> index e8f2362..9662623 100644
>>> --- a/package/gstreamer/gst-plugins-ugly/gst-plugins-ugly.mk
>>> +++ b/package/gstreamer/gst-plugins-ugly/gst-plugins-ugly.mk
>>> @@ -7,6 +7,8 @@
>>>   GST_PLUGINS_UGLY_VERSION = 0.10.19
>>>   GST_PLUGINS_UGLY_SOURCE = gst-plugins-ugly-$(GST_PLUGINS_UGLY_VERSION).tar.xz
>>>   GST_PLUGINS_UGLY_SITE = http://gstreamer.freedesktop.org/src/gst-plugins-ugly
>>> +GST_PLUGINS_UGLY_LICENSE = LGPLv2.1+
>>
>>   synaesthesia is GPLv2+ (and is always enabled)
>>
>>   All the rest is LGPLv2+...
>>
>>> +GST_PLUGINS_UGLY_LICENSE_FILES = COPYING
>>
>>   ...except this file which is 2.1...
>>
>>   I guess there's a reason it's called ugly :-)
>>
>>
>>   Regards,
>>   Arnout
> 
> Hi Arnout.
> Actually it's more ugly, in fact tests/check/elements/xingmux.c is LGPLv2.1+ :)

 When checking licenses, I skip stuff in tests, docs, config, ... because we
don't build that.


 Regards,
 Arnout

> So i'm going for:
> 
> # COPYING is in fact LGPLv2.1, but all of the code is v2+
> # (except for one test, xingmux)
> GST_PLUGINS_UGLY_LICENSE = LGPLv2+, GPLv2+ (synaesthesia)
> 
> Since in general we don't ship/use tests.
> Regards.
> 


-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
Essensium/Mind                                http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium           BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

^ permalink raw reply	[flat|nested] 15+ messages in thread

end of thread, other threads:[~2015-11-20 21:27 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-11-18 12:39 [Buildroot] [PATCH 1/6] gstreamer: add legal info gustavo.zacarias at free-electrons.com
2015-11-18 12:39 ` [Buildroot] [PATCH 2/6] gst-plugins-base: " gustavo.zacarias at free-electrons.com
2015-11-18 21:49   ` Arnout Vandecappelle
2015-11-18 12:39 ` [Buildroot] [PATCH 3/6] gst-plugins-good: " gustavo.zacarias at free-electrons.com
2015-11-18 21:53   ` Arnout Vandecappelle
2015-11-18 12:39 ` [Buildroot] [PATCH 4/6] gst-plugins-bad: " gustavo.zacarias at free-electrons.com
2015-11-18 22:03   ` Arnout Vandecappelle
2015-11-19 22:08     ` Gustavo Zacarias
2015-11-18 12:39 ` [Buildroot] [PATCH 5/6] gst-plugins-ugly: " gustavo.zacarias at free-electrons.com
2015-11-18 22:12   ` Arnout Vandecappelle
2015-11-19 22:06     ` Gustavo Zacarias
2015-11-20 21:27       ` Arnout Vandecappelle
2015-11-18 12:39 ` [Buildroot] [PATCH 6/6] gst-ffmpeg: " gustavo.zacarias at free-electrons.com
2015-11-18 22:15   ` Arnout Vandecappelle
2015-11-18 21:40 ` [Buildroot] [PATCH 1/6] gstreamer: " Arnout Vandecappelle

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox