[oe] [PATCH] ftplib: add initial version of recipe

[Paul Menzel]

Dear Vitaly,


thank you for your patch.

Am Sonntag, den 11.12.2011, 00:10 +0400 schrieb Vitaly Perov:
> From: Vitaly Perov <vit at vt-tech.eu>

Please add the recipe version to the commit summary (subject) and follow
our commit policy [1][2][3].

Especially your Signed-off-by line is missing. Please also add
information how you tested that recipe.

> ---
>  recipes/ftplib/ftplib_3.1.bb |   38 ++++++++++++++++++++++++++++++++++++++
>  1 files changed, 38 insertions(+), 0 deletions(-)
>  create mode 100644 recipes/ftplib/ftplib_3.1.bb
> 
> diff --git a/recipes/ftplib/ftplib_3.1.bb b/recipes/ftplib/ftplib_3.1.bb
> new file mode 100644
> index 0000000..1dcdb3d
> --- /dev/null
> +++ b/recipes/ftplib/ftplib_3.1.bb
> @@ -0,0 +1,38 @@
> +#! /bin/sh
> +

The shebang is not needed.

> +DESCRIPTION = "A set of routines that implement the FTP protocol"
> +HOMEPAGE = "http://nbpfaus.net/~pfau/ftplib/"
> +LICENSE = "LGPLv2"
> +
> +PR = "r1"

Please start with `r0` when submitting.

> +
> +######################################################################################

Please remove these “style” lines. They are not common. [4]

> +
> +TARGET_CC_ARCH += "${LDFLAGS}"

Why is that needed. Please add a comment and explain it in the commit
message.

> +
> +SRC_URI = "http://nbpfaus.net/~pfau/ftplib/ftplib-${PV}-src.tar.gz"

Can you talk with upstream, that they release versioned tarballs
otherwise that will become a mess when upgraded.

> +
> +S = "${WORKDIR}/ftplib-${PV}"
> +
> +######################################################################################
> +
> +SRC_URI[md5sum] = "c6af758ccbf806b28f022a587a41d9c5"
> +SRC_URI[sha256sum] = "be524e1086cfe9d86afa832e1413dde79a4e88ee8ea5325a87ffaaf6620a0a2b"
> +
> +######################################################################################
> +do_compile() {
> +    cd linux && make

Are you sure that works? At least `oe-runmake` should be used. This way
you should get not the correct target architecture since the build host
make is used. (I could be mistaken and maybe the Makefile figures it out
nevertheless.)

> +}
> +
> +do_install() {
> +    install -d ${D}/${libdir}
> +    install -d ${D}/${includedir}
> +    install -d ${D}/${bindir}
> +    install -m 755 ${S}/linux/libftp.so.3.1 ${D}/${libdir}
> +    install -m 755 ${S}/linux/libftp.a ${D}/${libdir}
> +    install -m 644 ${S}/linux/ftplib.h ${D}/${includedir}
> +    install -m 755 ${S}/linux/qftp ${D}/${bindir}
> +    (cd ${D}/${libdir} && \
> +     ln -sf libftp.so.3.1 libftp.so.3 && \
> +     ln -sf libftp.so.3 libftp.so)
> +}

I do not know if the last three lines are common. But if it works,
great!


Thanks,

Paul


[1] http://www.openembedded.org/wiki/Category:Policy
[2] http://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
[3] http://www.openembedded.org/wiki/Commit_Policy
[4] http://www.openembedded.org/wiki/Styleguide
-------------- 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/20111210/0530cb2c/attachment-0002.sig>