[oe] [meta-networking] [PATCH] unbound: this patch adds unbound to meta-networking

[Khem Raj]

On Wed, Apr 5, 2017 at 6:40 AM, Oliver Graute <oliver.graute at gmail.com> wrote:
> Unbound is a validating, recursive, and caching DNS resolver
>
> Signed-off-by: Oliver Graute <oliver.graute at neuhaus.de>
> ---
>  .../recipes-support/unbound/unbound_1.6.1.bb       | 57 ++++++++++++++++++++++
>  1 file changed, 57 insertions(+)
>  create mode 100644 meta-networking/recipes-support/unbound/unbound_1.6.1.bb
>
> diff --git a/meta-networking/recipes-support/unbound/unbound_1.6.1.bb b/meta-networking/recipes-support/unbound/unbound_1.6.1.bb
> new file mode 100644
> index 0000000..b6e0a42
> --- /dev/null
> +++ b/meta-networking/recipes-support/unbound/unbound_1.6.1.bb
> @@ -0,0 +1,57 @@
> +SUMMARY = "Unbound is a validating, recursive, and caching DNS resolver"
> +DESCRIPTION = "\ Unbound is designed as a set of modular components, so that \
> +                also DNSSEC (secure DNS) validation and stub-resolvers \
> +                (that do not run as a server, but are linked into an applica \
> +                tion) are easily possible."
> +
> +HOMEPAGE = "https://www.unbound.net/"
> +SECTION = "net"
> +LICENSE = "BSD"

Can you see which clause it is ? it will make it more clear

> +LIC_FILES_CHKSUM = "file://LICENSE;md5=5308494bc0590c0cb036afd781d78f06"
> +DEPENDS = "openssl libtool libevent"
> +RDEPENDS_${PN} += "bash"
> +inherit autotools pkgconfig
> +PR = "r0"

drop it

> +
> +SRC_URI = "https://www.unbound.net/downloads/${BP}.tar.gz"
> +SRC_URI[md5sum] = "aa808f33d94a36c9312d1b8ad8805e14"
> +SRC_URI[sha256sum] = "42df63f743c0fe8424aeafcf003ad4b880b46c14149d696057313f5c1ef51400"
> +
> +S = "${WORKDIR}/unbound-${PV}"
> +
> +EXTRA_OECONF = " \
> +        --host=${TARGET_SYS} \
> +        --sysconfdir=/etc/ \
> +        --localstatedir=/var \
> +        --sbindir=/usr/bin \

these look redundant.

> +        --with-conf-file=/etc/unbound/unbound.conf \
> +        --with-pidfile=/run/unbound.pid \
> +        --with-rootkey-file=/etc/trusted-key.key \
> +"
> +
> +do_configure () {
> +        ${S}/configure ${EXTRA_OECONF}
> +}

do we really need it? can't be reconfigure ?

> +
> +do_compile () {
> +        oe_runmake
> +}

redundant remove this function

> +
> +do_install () {
> +        install -d ${D}${bindir}
> +        install -d ${D}${libdir}
> +        install -d ${D}${sysconfdir}
> +        install -m 644 -D ${S}/libunbound/unbound.h ${D}${includedir}/unbound.h
> +        install -m 644 -D ${S}/doc/example.conf.in ${D}${sysconfdir}/unbound/unbound.conf
> +        install -m 644 -D ${S}/contrib/unbound.service.in ${D}${libdir}/systemd/unbound.service
> +        install -m 755 unbound ${D}${bindir}
> +        install -m 755 unbound-anchor ${D}${bindir}
> +        install -m 755 unbound-checkconf ${D}${bindir}
> +        install -m 755 unbound ${D}${bindir}
> +        install -m 755 unbound-control-setup ${D}${bindir}
> +        install -m 755 unbound-host ${D}${bindir}
> +        install -m 755 ${WORKDIR}/build/.libs/libunbound.so* ${D}${libdir}
> +}

do this in a do_install_append()

> +
> +FILES_${PN} ="${sysconfdir}/unbound/unbound.conf ${bindir}/unbound* ${libdir}/systemd/unbound.service ${includedir}/unbound.h ${libdir}/libunbound.so*"
> +FILES_${PN}-dev = "${includedir}/*"
> --
> 1.9.1
>
> --
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel at lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel