[PATCH 0/2] update lz4 and add maintainer

[PATCH 0/2] update lz4 and add maintainer

[Armin Kuster]

Update SRC_URI to github so we can use less complicated fetch
Update to version 119

Add self to maintainers list. If inappropriate, then drop second patch

Armin Kuster (2):
  lz4: update to latest version 119
  lz4: Add self to maintainers list

 meta-yocto/conf/distro/include/maintainers.inc       |  1 +
 meta/recipes-support/lz4/{lz4_svn.bb => lz4_r119.bb} | 12 ++++++------
 2 files changed, 7 insertions(+), 6 deletions(-)
 rename meta/recipes-support/lz4/{lz4_svn.bb => lz4_r119.bb} (63%)

-- 
1.9.1

[PATCH 1/2] lz4: update to latest version 119

[Armin Kuster]

Update SRC_URI to github as googlecode does not have this version
Minor clean to reflect non-svn fetch

Signed-off-by: Armin Kuster <akuster@mvista.com>
---
 meta/recipes-support/lz4/{lz4_svn.bb => lz4_r119.bb} | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)
 rename meta/recipes-support/lz4/{lz4_svn.bb => lz4_r119.bb} (63%)

diff --git a/meta/recipes-support/lz4/lz4_svn.bb b/meta/recipes-support/lz4/lz4_r119.bb
similarity index 63%
rename from meta/recipes-support/lz4/lz4_svn.bb
rename to meta/recipes-support/lz4/lz4_r119.bb
index 38e36d8..dfc155e 100644
--- a/meta/recipes-support/lz4/lz4_svn.bb
+++ b/meta/recipes-support/lz4/lz4_r119.bb
@@ -2,15 +2,15 @@ SUMMARY = "Extremely Fast Compression algorithm"
 DESCRIPTION = "LZ4 is a very fast lossless compression algorithm, providing compression speed at 400 MB/s per core, scalable with multi-cores CPU. It also features an extremely fast decoder, with speed in multiple GB/s per core, typically reaching RAM speed limits on multi-core systems."
 
 LICENSE = "BSD"
-LIC_FILES_CHKSUM = "file://LICENSE;md5=2008d2325e11691e17fcaa3a6046f850"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=0b0d063f37a4477b54af2459477dcafd"
 
-# Upstream names releases after SVN revs
-SRCREV = "112"
-PV = "r${SRCREV}"
 
-SRC_URI = "svn://lz4.googlecode.com/svn/;module=trunk;protocol=http"
+SRC_URI = "https://github.com/Cyan4973/lz4/archive/${PV}.tar.gz"
 
-S = "${WORKDIR}/trunk"
+SRC_URI[md5sum] = "3fc4ed20cb98dba909896f485b25d0f1"
+SRC_URI[sha256sum] = "1deb77e5cc7b41422c02dd3d56e7e0f0deb20723a54cad4ad5a65fb1f4805b1c"
+
+S = "${WORKDIR}/${BPN}-${PV}"
 
 EXTRA_OEMAKE = "PREFIX=${prefix} CC='${CC}' DESTDIR=${D} LIBDIR=${libdir} INCLUDEDIR=${includedir}"
 
-- 
1.9.1

[PATCH 2/2] lz4: Add self to maintainers list

[Armin Kuster]

Signed-off-by: Armin Kuster <akuster@mvista.com>
---
 meta-yocto/conf/distro/include/maintainers.inc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta-yocto/conf/distro/include/maintainers.inc b/meta-yocto/conf/distro/include/maintainers.inc
index 4d2f46b..4c566d1 100644
--- a/meta-yocto/conf/distro/include/maintainers.inc
+++ b/meta-yocto/conf/distro/include/maintainers.inc
@@ -419,6 +419,7 @@ RECIPE_MAINTAINER_pn-lttng-tools = "Richard Purdie <richard.purdie@linuxfoundati
 RECIPE_MAINTAINER_pn-lttng-ust = "Richard Purdie <richard.purdie@linuxfoundation.org>"
 RECIPE_MAINTAINER_pn-lzop = "Saul Wold <sgw@linux.intel.com>"
 RECIPE_MAINTAINER_pn-lzo = "Saul Wold <sgw@linux.intel.com>"
+RECIPE_MAINTAINER_pn-lz4 = "Armin Kuster <akuster@mvista.com>"
 RECIPE_MAINTAINER_pn-m4 = "Robert Yang <liezhi.yang@windriver.com>"
 RECIPE_MAINTAINER_pn-mailx = "Kai Kang <kai.kang@windriver.com>"
 RECIPE_MAINTAINER_pn-make = "Robert Yang <liezhi.yang@windriver.com>"
-- 
1.9.1

Re: [PATCH 1/2] lz4: update to latest version 119

[Burton, Ross]

On 7 July 2014 22:33, Armin Kuster <akuster@mvista.com> wrote:
> -LIC_FILES_CHKSUM = "file://LICENSE;md5=2008d2325e11691e17fcaa3a6046f850"
> +LIC_FILES_CHKSUM = "file://LICENSE;md5=0b0d063f37a4477b54af2459477dcafd"

Please add a comment in the commit message explaining why the checksum
has changed.

> -SRC_URI = "svn://lz4.googlecode.com/svn/;module=trunk;protocol=http"
> +SRC_URI = "https://github.com/Cyan4973/lz4/archive/${PV}.tar.gz"

Github archives can't be trusted as they can be re-generated at any
time by github (and the checksums will change).  If upstream doesn't
provide static tarballs then use the git fetcher and checkout the
release tag.

> +S = "${WORKDIR}/${BPN}-${PV}"

This is the default value.

Ross

Re: [PATCH 2/2] lz4: Add self to maintainers list

[Saul Wold]

On 07/07/2014 02:33 PM, Armin Kuster wrote:
> Signed-off-by: Armin Kuster <akuster@mvista.com>
> ---
>   meta-yocto/conf/distro/include/maintainers.inc | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/meta-yocto/conf/distro/include/maintainers.inc b/meta-yocto/conf/distro/include/maintainers.inc
> index 4d2f46b..4c566d1 100644
> --- a/meta-yocto/conf/distro/include/maintainers.inc
> +++ b/meta-yocto/conf/distro/include/maintainers.inc
> @@ -419,6 +419,7 @@ RECIPE_MAINTAINER_pn-lttng-tools = "Richard Purdie <richard.purdie@linuxfoundati
>   RECIPE_MAINTAINER_pn-lttng-ust = "Richard Purdie <richard.purdie@linuxfoundation.org>"
>   RECIPE_MAINTAINER_pn-lzop = "Saul Wold <sgw@linux.intel.com>"
>   RECIPE_MAINTAINER_pn-lzo = "Saul Wold <sgw@linux.intel.com>"
> +RECIPE_MAINTAINER_pn-lz4 = "Armin Kuster <akuster@mvista.com>"
>   RECIPE_MAINTAINER_pn-m4 = "Robert Yang <liezhi.yang@windriver.com>"
>   RECIPE_MAINTAINER_pn-mailx = "Kai Kang <kai.kang@windriver.com>"
>   RECIPE_MAINTAINER_pn-make = "Robert Yang <liezhi.yang@windriver.com>"
>
This as an independant patch should go to poky@yoctoproject.org as this 
is part of the poky distro files.

Thanks
	Sau!

Re: [PATCH 1/2] lz4: update to latest version 119

[akuster]

On 07/08/2014 03:15 AM, Burton, Ross wrote:
> On 7 July 2014 22:33, Armin Kuster <akuster@mvista.com> wrote:
>> -LIC_FILES_CHKSUM = "file://LICENSE;md5=2008d2325e11691e17fcaa3a6046f850"
>> +LIC_FILES_CHKSUM = "file://LICENSE;md5=0b0d063f37a4477b54af2459477dcafd"
>
> Please add a comment in the commit message explaining why the checksum
> has changed.

ok, will do.

>
>> -SRC_URI = "svn://lz4.googlecode.com/svn/;module=trunk;protocol=http"
>> +SRC_URI = "https://github.com/Cyan4973/lz4/archive/${PV}.tar.gz"
>
> Github archives can't be trusted as they can be re-generated at any
> time by github (and the checksums will change).  If upstream doesn't
> provide static tarballs then use the git fetcher and checkout the
> release tag.
>
Thanks for the info. I will look for a different solution.

- Armin

>> +S = "${WORKDIR}/${BPN}-${PV}"
>
> This is the default value.
>
> Ross
>