[oe] [meta-oe][PATCH] talloc: rename to libtalloc and cleanup dependency mess

[Christopher Larson]

On Fri, Nov 13, 2015 at 10:08 AM, Jens Rehsack <rehsack at gmail.com> wrote:

> > Am 13.11.2015 um 18:01 schrieb Christopher Larson <kergoth at gmail.com>:
> >
> > On Fri, Nov 13, 2015 at 9:56 AM, Christopher Larson <kergoth at gmail.com>
> wrote:
> > On Fri, Nov 13, 2015 at 8:53 AM, Jens Rehsack <rehsack at gmail.com> wrote:
> > To avoid errors when building dev-images (talloc-dev is missing), and to
> avoid
> > insane empty packages for that, rename talloc -> libtalloc as libtevent
> shows.
> >
> > With that, remove dependencies to attr/xattr and libbsd - unless
> explicitely
> > enabled via PACKAGECONFIG.
> >
> > Signed-off-by: Jens Rehsack <sno at netbsd.org>
> >
> > Our recipe names should be named based on the upstream project name, not
> the library they happen to ship.
> >
> > Instead, either we could we should kill all the libtalloc bits in the
> PACKAGES var, and let the debian shlib renaming handle renaming talloc* to
> libtalloc*, or add a talloc-dev that pulls in libtalloc-dev & pytalloc-dev.
> >
> > I just realized this may sound critical, but that wasn't intended. The
> other changes look great, thanks for your work on making those dependencies
> optional, it's good stuff.
>
> In that case - please take me by the hand and explain a bit more detailed
>
> > Instead, either we could we should kill all the libtalloc bits in the
> PACKAGES var, and let the debian shlib renaming handle renaming talloc* to
> libtalloc*
>
> since I have no clue what you try to guide me.


I'm not able to reproduce any error when building an image. Add libtalloc
and dev-pkgs, and it builds just fine. You can't add talloc, as there is no
talloc package emitted.

That said, what I was suggesting was something along the lines of
https://gist.github.com/kergoth/ead0e1d2682a9e9fcd59 -- debian.bbclass will
automatically rename the ${PN}, ${PN}-dev, and ${PN}-dbg packages based on
the library soname, so will end up named libtalloc* anyway if you use it.
Since the main packages weren't emitted at all, it was pointless complexity
in the packaging to add libtalloc* explicitly.
-- 
Christopher Larson
kergoth at gmail dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Maintainer - Tslib
Senior Software Engineer, Mentor Graphics