diff mbox series

[v3] zstd: bump to version 1.3.1

Message ID 7c859af274e2f31bbcc7db64b9096d2d7e0e7733.1505479199.git.baruch@tkos.co.il
State Accepted
Commit b34d7b7674b31fcfc227ed516c7a4d5c1bb87d64
Headers show
Series [v3] zstd: bump to version 1.3.1 | expand

Commit Message

Baruch Siach Sept. 15, 2017, 12:39 p.m. UTC
In this release zstd added the GPLv2 license and removed the PATENTS
clause (upstream commit 4f73b3b55d83b).

Cc: Andrey Smirnov <andrew.smirnov@gmail.com>
Signed-off-by: Baruch Siach <baruch@tkos.co.il>
---
v3: Remove PATENTS from LICENSE_FILES as well
v2: Remove the PATENTS hash, update commit log
---
 package/zstd/zstd.hash | 4 ++--
 package/zstd/zstd.mk   | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)

Comments

Andrey Smirnov Sept. 16, 2017, 12:22 a.m. UTC | #1
On Fri, Sep 15, 2017 at 5:39 AM, Baruch Siach <baruch@tkos.co.il> wrote:
> In this release zstd added the GPLv2 license and removed the PATENTS
> clause (upstream commit 4f73b3b55d83b).
>

Hah! You beat me to the punch, I have/had similar patch sitting in my
own repo that I was going to submit this week.

Builds and runs fine on my setup, so:

Tested-by: Andrey Smirnov <andrew.smirnov@gmail.com>

NOTE: I did notice that my original patch that added zstd has a bug
where it install tools to "/usr/local/bin" instead of "/usr/bin". I'll
submit a patch to fix that shortly.

Thanks,
Andrey Smirnov.
Baruch Siach Sept. 16, 2017, 7:26 p.m. UTC | #2
Hi Andrey,

On Fri, Sep 15, 2017 at 05:22:50PM -0700, Andrey Smirnov wrote:
> On Fri, Sep 15, 2017 at 5:39 AM, Baruch Siach <baruch@tkos.co.il> wrote:
> > In this release zstd added the GPLv2 license and removed the PATENTS
> > clause (upstream commit 4f73b3b55d83b).
> 
> Hah! You beat me to the punch, I have/had similar patch sitting in my
> own repo that I was going to submit this week.
> 
> Builds and runs fine on my setup, so:
> 
> Tested-by: Andrey Smirnov <andrew.smirnov@gmail.com>
> 
> NOTE: I did notice that my original patch that added zstd has a bug
> where it install tools to "/usr/local/bin" instead of "/usr/bin". I'll
> submit a patch to fix that shortly.

One other issue that I found while testing the build is that the lz4 
dependency is not actually used even when enabled. You might want to look into 
that as well.

baruch
Bernd Kuhls Sept. 19, 2017, 6:36 p.m. UTC | #3
Am Fri, 15 Sep 2017 15:39:59 +0300 schrieb Baruch Siach:

> In this release zstd added the GPLv2 license and removed the PATENTS
> clause (upstream commit 4f73b3b55d83b).
> 
> Cc: Andrey Smirnov
> <andrew.smirnov@gmail.com>
> Signed-off-by: Baruch Siach
> <baruch@tkos.co.il>

Tested-by: Bernd Kuhls <bernd.kuhls@t-online.de>
Peter Korsgaard Sept. 20, 2017, 5:35 p.m. UTC | #4
>>>>> "Baruch" == Baruch Siach <baruch@tkos.co.il> writes:

 > In this release zstd added the GPLv2 license and removed the PATENTS
 > clause (upstream commit 4f73b3b55d83b).

 > Cc: Andrey Smirnov <andrew.smirnov@gmail.com>
 > Signed-off-by: Baruch Siach <baruch@tkos.co.il>
 > ---
 > v3: Remove PATENTS from LICENSE_FILES as well
 > v2: Remove the PATENTS hash, update commit log

Committed, thanks.
diff mbox series

Patch

diff --git a/package/zstd/zstd.hash b/package/zstd/zstd.hash
index 60641aa8ab6c..7fa5f7bf6ada 100644
--- a/package/zstd/zstd.hash
+++ b/package/zstd/zstd.hash
@@ -1,6 +1,6 @@ 
 # Locally computed
-sha256 0fdba643b438b7cbce700dcc0e7b3e3da6d829088c63757a5984930e2f70b348 zstd-v1.3.0.tar.gz
+sha256 312fb9dc75668addbc9c8f33c7fa198b0fc965c576386b8451397e06256eadc6 zstd-v1.3.1.tar.gz
 
 # License files (locally computed as well)
 sha256 2c1a7fa704df8f3a606f6fc010b8b5aaebf403f3aeec339a12048f1ba7331a0b LICENSE
-sha256 3a508245dd3c486bb9d78a8269091322359443ea54612c3648327a0b3512f23f PATENTS
+sha256 f9c375a1be4a41f7b70301dd83c91cb89e41567478859b77eef375a52d782505 COPYING
diff --git a/package/zstd/zstd.mk b/package/zstd/zstd.mk
index cad80684f307..475079dbfe2b 100644
--- a/package/zstd/zstd.mk
+++ b/package/zstd/zstd.mk
@@ -4,10 +4,10 @@ 
 #
 ################################################################################
 
-ZSTD_VERSION = v1.3.0
+ZSTD_VERSION = v1.3.1
 ZSTD_SITE = $(call github,facebook,zstd,$(ZSTD_VERSION))
-ZSTD_LICENSE = BSD-3-Clause
-ZSTD_LICENSE_FILES = LICENSE PATENTS
+ZSTD_LICENSE = BSD-3-Clause, GPL-2.0
+ZSTD_LICENSE_FILES = LICENSE COPYING
 
 ifeq ($(BR2_TOOLCHAIN_HAS_THREADS),y)
 ZSTD_OPTS += HAVE_THREAD=1