[OE-core] [PATCH v3 1/3] dbus: merge .bb and .inc

[Burton, Ross]

On 2 December 2015 at 16:27, Patrick Ohly <patrick.ohly at intel.com> wrote:

> I'm not sure about the "more complicated to do changes in external
> layers". Why is that?
>
> Quite the opposite, I found it useful that the version independent build
> rules (*) were in a .inc file and relied on including that .inc file in
> a "dbus-cynara" recipe which builds a specific fork of the source code
> [2].
>
> But I understand that this is an unusual usage of OE-core. As I didn't
> notice the change in time (it's in master now), I'll just copy the
> previous content of dbus.inc.
>

When a bb and an inc are split the inc has to be considered some sort of
ABI.  For example, you couldn't remove a configure option that was removed
in a new release from the .inc because older recipes that use the inc will
still want it.  Then of course on new recipes that triggers a QA warning.
New options need to be added to the relevant .bb and not the inc, but then
break if other users don't notice that they need to add options
explicitly.  Basically the idea of bb/inc seems good, and in some
situations is useful (multiple recipes maintained *in the same place*
sharing common code) but as a method of re-using packaging it's got many
pitfalls, especially as your recipe could include the .bb from oe-core
directly or be a bbappend.

Ross
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20151202/241a9685/attachment-0002.html>