[oe] [PATCH v3] libroxml: Add version 2.1.1

[Paul Menzel]

Dear Tristan,


thank you for your patience.

Am Mittwoch, den 21.09.2011, 14:40 +0200 schrieb Tristan Lelong:
> libroxml is a small fast and powerful xml library
> build successfully using org.openembedded.dev with DISTRO=minimal and MACHINE = "i686-generic"
> build successfully using org.openembedded.dev with DISTRO=minimal and MACHINE = "ppce500v2"
> tested using projects unit tests on powerpc binaries
> 
> Signed-off-by: Tristan Lelong <tristan.lelong at blunderer.org>
> ---
>  recipes/libroxml/libroxml.inc      |   21 +++++++++++++++++++++
>  recipes/libroxml/libroxml_2.1.1.bb |    5 +++++
>  2 files changed, 26 insertions(+), 0 deletions(-)
>  create mode 100644 recipes/libroxml/libroxml.inc
>  create mode 100644 recipes/libroxml/libroxml_2.1.1.bb
> 
> diff --git a/recipes/libroxml/libroxml.inc b/recipes/libroxml/libroxml.inc
> new file mode 100644
> index 0000000..eed21d1
> --- /dev/null
> +++ b/recipes/libroxml/libroxml.inc
> @@ -0,0 +1,21 @@
> +DESCRIPTION = "Small, fast and powerful xml library"
> +AUTHOR = "Tristan Lelong <tristan.lelong at libroxml.net>"
> +HOMEPAGE = "http://www.libroxml.net"
> +SECTION = "libs"
> +PRIORITY = "optional"
> +LICENSE = "LGPL"

I checked again. This should be `LGPLv2.1+`, I think.

> +INC_PR = "r1"

Please start with `r0` when submitting recipes.

> +
> +SRC_URI = "http://libroxml.googlecode.com/files/${P}.tar.gz"
> +
> +do_install() {
> +	install -D -m 755 ${S}/roxml ${D}/${bindir}/roxml
> +	install -D -m 755 ${S}/libroxml.so.0 ${D}/${libdir}/libroxml.so.0
> +	install -D -m 644 ${S}/inc/roxml.h ${D}/${includedir}/roxml.h
> +	ln -s /usr/lib/libroxml.so.0 ${D}/${libdir}/libroxml.so
> +}
> +
> +PACKAGES = ${PN} ${PN}-dev roxml

Is that intended? Using `=` you override the variable and for example no
debug package is created and the following files are not packaged.

        WARNING: the following files were installed but not shipped in any package:
        WARNING:   /usr/lib/.debug/libroxml.so.0
        WARNING:   /usr/bin/.debug/roxml

> +FILES_${PN} = ${libdir}/libroxml.so.0
> +FILES_${PN}-dev = ${libdir}/libroxml.so ${includedir}/roxml.h
> +FILES_roxml = ${bindir}/roxml

Does

PACKAGES += "roxml"
FILES_roxml = ${bindir}/roxml

work for? I do not know if it is done automatically. Maybe you also have
to add `roxml-dbg` then. I am not sure.

> diff --git a/recipes/libroxml/libroxml_2.1.1.bb b/recipes/libroxml/libroxml_2.1.1.bb
> new file mode 100644
> index 0000000..0852930
> --- /dev/null
> +++ b/recipes/libroxml/libroxml_2.1.1.bb
> @@ -0,0 +1,5 @@
> +require libroxml.inc
> +PR = "${INC_PR}.1"

Please also start with 0.

> +
> +SRC_URI[md5sum] = "dc8342d0490bff5a507dbf1de3426102"
> +SRC_URI[sha256sum] = "e4e91b610614d6cb1204ac9abbb417368fe84654c7de58134b436cfd00f1a244"

Thank you! I guess the next iteration will be ready to be committed.


Thanks,

Paul
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://lists.openembedded.org/pipermail/openembedded-devel/attachments/20110921/77284e81/attachment-0002.sig>