Re: [PATCH v2] lzop-1.03: add recipe

[Saul Wold <sgw@linux.intel.com>]

On 12/05/2011 02:13 PM, Christopher Larson wrote:
> From: Christopher Larson<chris_larson@mentor.com>
>
> This is needed by some kernels when CONFIG_KERNEL_LZO=y (specifically, given
> the current defconfig, this affects linux-omap4 2.6 in the meta-ti layer).
>
> Signed-off-by: Christopher Larson<chris_larson@mentor.com>
> ---
>   meta/recipes-support/lzop/lzop/acinclude.m4 |  390 +++++++++++++++++++++++++++
>   meta/recipes-support/lzop/lzop_1.03.bb      |   25 ++
>   2 files changed, 415 insertions(+), 0 deletions(-)
>   create mode 100644 meta/recipes-support/lzop/lzop/acinclude.m4
>   create mode 100644 meta/recipes-support/lzop/lzop_1.03.bb
>

<SNIP>

> diff --git a/meta/recipes-support/lzop/lzop_1.03.bb b/meta/recipes-support/lzop/lzop_1.03.bb
> new file mode 100644
> index 0000000..d0cb842
> --- /dev/null
> +++ b/meta/recipes-support/lzop/lzop_1.03.bb
> @@ -0,0 +1,25 @@
> +SUMMARY = "Real-time file compressor"
> +DESCRIPTION = "lzop is a compression utility which is designed to be a companion to gzip. \n\
> +It is based on the LZO data compression library and its main advantages over \n\
> +gzip are much higher compression and decompression speed at the cost of some \n\
> +compression ratio. The lzop compression utility was designed with the goals \n\
> +of reliability, speed, portability and with reasonable drop-in compatibility \n\
> +to gzip."
> +DEPENDS += "lzo"
> +PR = "r0"
> +
> +LICENSE = "GPLv2+"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=dfeaf3dc4beef4f5a7bdbc35b197f39e \
> +                    file://src/lzop.c;beginline=5;endline=21;md5=6797bd3ed0a1a49327b7ebf9366ebd86"
> +
> +SRC_URI = "http://www.lzop.org/download/${BP}.tar.gz \
I think this ------------------------------- ^^
should really be ${BPN}-${PV} which make is much easier to do updates 
with by just changing the file name.


> +           file://acinclude.m4"
> +
NO SRC_URI Checksums!

> +inherit autotools
> +
> +do_configure () {
> +    ln -sf ../acinclude.m4
> +    autotools_do_configure
> +}
> +
> +BBCLASSEXTEND += "native nativesdk"