[Buildroot] [PATCH 1/1] package/cryptodev: fix build

[Buildroot] [PATCH 1/1] package/cryptodev: fix build

[Sébastien Szymanski]

cryptodev-linux-1.8 build is broken with kernel >= 4.6.0 because of
APIs changes in the kernel. Upstream already fixed this:

https://github.com/cryptodev-linux/cryptodev-linux/commit/2b29be8ac41414ed19cb4f5d5626d9bd0d7b11a8
https://github.com/cryptodev-linux/cryptodev-linux/commit/f126e4837e6334d0464540995df7426fedf6b175
https://github.com/cryptodev-linux/cryptodev-linux/commit/f14b4706b0d04988e7e5bc8c4d2aefef9f029d9d
https://github.com/cryptodev-linux/cryptodev-linux/commit/cb186f682679383e8b5806240927903730ce85d9

Bump to latest commit to fix cryptodev-linux build.

Signed-off-by: S?bastien Szymanski <sebastien.szymanski@armadeus.com>
---
 package/cryptodev-linux/cryptodev-linux.hash | 4 ++--
 package/cryptodev-linux/cryptodev-linux.mk   | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/package/cryptodev-linux/cryptodev-linux.hash b/package/cryptodev-linux/cryptodev-linux.hash
index d7195ac..6d9efca 100644
--- a/package/cryptodev-linux/cryptodev-linux.hash
+++ b/package/cryptodev-linux/cryptodev-linux.hash
@@ -1,2 +1,2 @@
-# Locally calculated after checking php signature
-sha256	67fabde9fb67b286a96c4f45b594b0eccd0f761b495705c18f2ae9461b831376	cryptodev-linux-1.8.tar.gz
+# Locally calculated
+sha256 a87e354e1218e6a2e3083f7be6a3e351c42ec55515f1aff1b50bd301cee7d8dd cryptodev-linux-2b29be8ac41414ed19cb4f5d5626d9bd0d7b11a8.tar.gz
diff --git a/package/cryptodev-linux/cryptodev-linux.mk b/package/cryptodev-linux/cryptodev-linux.mk
index e23e1e3..709c6ed 100644
--- a/package/cryptodev-linux/cryptodev-linux.mk
+++ b/package/cryptodev-linux/cryptodev-linux.mk
@@ -4,8 +4,8 @@
 #
 ################################################################################
 
-CRYPTODEV_LINUX_VERSION = 1.8
-CRYPTODEV_LINUX_SITE = http://download.gna.org/cryptodev-linux
+CRYPTODEV_LINUX_VERSION = 2b29be8ac41414ed19cb4f5d5626d9bd0d7b11a8
+CRYPTODEV_LINUX_SITE = $(call github,cryptodev-linux,cryptodev-linux,$(CRYPTODEV_LINUX_VERSION))
 CRYPTODEV_LINUX_INSTALL_STAGING = YES
 CRYPTODEV_LINUX_LICENSE = GPLv2+
 CRYPTODEV_LINUX_LICENSE_FILES = COPYING
-- 
2.7.3

[Buildroot] [PATCH 1/1] package/cryptodev: fix build

[Thomas Petazzoni]

Hello,

On Thu,  9 Feb 2017 11:32:04 +0100, S?bastien Szymanski wrote:
> cryptodev-linux-1.8 build is broken with kernel >= 4.6.0 because of
> APIs changes in the kernel. Upstream already fixed this:
> 
> https://github.com/cryptodev-linux/cryptodev-linux/commit/2b29be8ac41414ed19cb4f5d5626d9bd0d7b11a8
> https://github.com/cryptodev-linux/cryptodev-linux/commit/f126e4837e6334d0464540995df7426fedf6b175
> https://github.com/cryptodev-linux/cryptodev-linux/commit/f14b4706b0d04988e7e5bc8c4d2aefef9f029d9d
> https://github.com/cryptodev-linux/cryptodev-linux/commit/cb186f682679383e8b5806240927903730ce85d9
> 
> Bump to latest commit to fix cryptodev-linux build.
> 
> Signed-off-by: S?bastien Szymanski <sebastien.szymanski@armadeus.com>
> ---
>  package/cryptodev-linux/cryptodev-linux.hash | 4 ++--
>  package/cryptodev-linux/cryptodev-linux.mk   | 4 ++--
>  2 files changed, 4 insertions(+), 4 deletions(-)

I've applied after tweaking the commit title: the package is
cryptodev-linux, not cryptodev, and "fix build" is very fuzzy. I've
changed it to:

	package/cryptodev-linux: fix with with recent kernels

Thanks a lot!

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

[Buildroot] [PATCH 1/1] package/cryptodev: fix build

[Peter Korsgaard]

>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

Hi,

 > I've applied after tweaking the commit title: the package is
 > cryptodev-linux, not cryptodev, and "fix build" is very fuzzy. I've
 > changed it to:

 > 	package/cryptodev-linux: fix with with recent kernels

Do you really find 'with with' so much clearer? :P

(Luckily your commit message was correct)

-- 
Bye, Peter Korsgaard

[Buildroot] [PATCH 1/1] package/cryptodev: fix build

[Thomas Petazzoni]

Hello,

On Fri, 10 Feb 2017 08:25:47 +0100, Peter Korsgaard wrote:

>  > I've applied after tweaking the commit title: the package is
>  > cryptodev-linux, not cryptodev, and "fix build" is very fuzzy. I've
>  > changed it to:  
> 
>  > 	package/cryptodev-linux: fix with with recent kernels  
> 
> Do you really find 'with with' so much clearer? :P

Still better than "fix build", but indeed not entirely perfect :-)

> (Luckily your commit message was correct)

Pffiou :)

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com