Re: [meta-networking][PATCH] iscsi-initiator-utils: Update to 2.0.874

[Joe MacDonald <Joe_MacDonald@mentor.com>]

[-- Attachment #1: Type: text/plain, Size: 3421 bytes --]

[Re: [oe] [meta-networking][PATCH] iscsi-initiator-utils: Update to 2.0.874] On 16.11.29 (Tue 21:00) Martin Jansa wrote:

> On Wed, Nov 23, 2016 at 10:08:29AM -0500, Joe MacDonald wrote:
> > The latest iscsi-initiator-utils has been available for a while now,
> > update our recipe.  The new version no longer bundles its own ISNS server,
> > so this update brings in open-isns as a dependency.
> > 
> > Additionally, open-iscsi.org is defunct, the new home is at
> > open-iscsi.com, update the SRC_URI and homepage accordingly.
> > 
> > Signed-off-by: Joe MacDonald <joe_macdonald@mentor.com>
[...]
> > diff --git a/meta-networking/recipes-support/open-isns/open-isns_0.97.bb b/meta-networking/recipes-support/open-isns/open-isns_0.97.bb
> > new file mode 100644
> > index 0000000..fe41093
> > --- /dev/null
> > +++ b/meta-networking/recipes-support/open-isns/open-isns_0.97.bb
> > @@ -0,0 +1,33 @@
> > +# Copyright (C) 2016 Joe MacDonald <joe_macdonald@mentor.com>
> > +# Released under the MIT license (see COPYING.MIT for the terms)
> > +
> > +SUMMARY = "iSNS daemon and utility programs"
> > +DESCRIPTION = "This is a partial implementation of RFC4171, the Internet \
> > +Storage Name Service (iSNS).  The distribution includes the iSNS server, \
> > +supporting persisten storage of registrations, isnsadm, a command line \
> > +utility for managing nodes, and isnsdd, a corresponding discovery daemon."
> > +HOMEPAGE = "http://github.com/gonzoleeman/open-isns/"
> > +LICENSE = "GPLv2+ & LGPLv2+"
> > +LIC_FILES_CHKSUM = "file://COPYING;md5=321bf41f280cf805086dd5a720b37785"
> > +SECTION = "net"
> > +
> > +DEPENDS = "openssl systemd"
> > +
> > +SRC_URI = " \
> > +           git://github.com/open-iscsi/open-isns \
> > +"
> > +
> > +SRCREV ?= "09954404e948e41eb0fce8e28836018b4ce3d20d"
> > +
> > +S = "${WORKDIR}/git"
> > +
> > +inherit systemd autotools-brokensep
> > +
> > +EXTRA_OECONF = " --prefix=${prefix} --enable-shared"
> > +
> > +do_install_append () {
> > +    oe_runmake -C ${B} INCDIR=${STAGING_INCDIR}/libisns/ install_hdrs
> > +    oe_runmake -C ${B} LIBDIR=${D}${libdir} install_lib
> 
> This seems to bypass sstate and install it directly to sysroot, why did you use STAGING_INCDIR?

I'm not sure, some brain-fart from before I added the brokensep
dependency, I think.  I've pointed it at the appropriate place (and
dropped the unnecessary chdir) in v2, coming out now.

-J.

> ERROR: Following files are installed in sysroot, but not tracked by
> sstate:
> qemuarm/usr/include/guid.h
> qemuarm/usr/include/libisns
> qemuarm/usr/include/libisns/attrs.h
> qemuarm/usr/include/libisns/buffer.h
> qemuarm/usr/include/libisns/isns.h
> qemuarm/usr/include/libisns/isns-proto.h
> qemuarm/usr/include/libisns/message.h
> qemuarm/usr/include/libisns/paths.h
> qemuarm/usr/include/libisns/source.h
> qemuarm/usr/include/libisns/types.h
> qemuarm/usr/include/libisns/util.h
> qemuarm/usr/lib/libcrossguid.a
> INFO: Output written in:
> /home/jenkins/oe/world/shr-core/tmp-glibc/sysroot.cruft.1480448660
> 
> 
> > +}
> > +
> > +FILES_${PN} += "${libdir}"
> > -- 
> > 1.9.1
> > 
> > -- 
> > _______________________________________________
> > Openembedded-devel mailing list
> > Openembedded-devel@lists.openembedded.org
> > http://lists.openembedded.org/mailman/listinfo/openembedded-devel
> 
-- 
-Joe MacDonald.
:wq

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 484 bytes --]