[oe] [OE-core] SRC_URI computing order

[Richard Purdie]

On Wed, 2013-11-06 at 09:45 +0100, Andrea Adami wrote:
> On Mon, Nov 4, 2013 at 11:13 PM, Andrea Adami <andrea.adami at gmail.com> wrote:
> > On Mon, Nov 4, 2013 at 12:10 AM, Richard Purdie
> > <richard.purdie at linuxfoundation.org> wrote:
> >> On Sun, 2013-11-03 at 23:16 +0100, Andrea Adami wrote:
> >>> I have to report an undesiderate behavior:
> >>>
> >>> the formfactor files in our .bbappend are not considered :/
> >>> DEBUG: Searching for machconfig in paths:....
> >>>   /oe/oe-core/meta/recipes-bsp/formfactor/formfactor-0.0/
> >>>   /oe/oe-core/meta/recipes-bsp/formfactor/formfactor/
> >>>   /oe/oe-core/meta/recipes-bsp/formfactor/files/
> >>>   /oe/meta-handheld/recipes-bsp/formfactor/files/poodle
> >>>
> >>> so we end up with the empty machconfig of
> >>> /oe/oe-core/meta/recipes-bsp/formfactor/files/
> >>>
> >>> Surely this didn't happen when we tested the recipe.
> >>
> >> With which revision of OE-Core? Was this with the dora release tag,
> >> current dora head or master?
> >>
> >> Cheers,
> >>
> >> Richard
> >>
> >>
> >
> > This was with fresh master:
> >
> > Build Configuration:
> > BB_VERSION        = "1.21.0"
> > BUILD_SYS         = "i686-linux"
> > NATIVELSBSTRING   = "Gentoo"
> > TARGET_SYS        = "arm-oe-linux-gnueabi"
> > MACHINE           = "poodle"
> > DISTRO_VERSION    = "oe-core.0"
> > TUNE_FEATURES     = "armv5 thumb dsp"
> > TARGET_FPU        = "soft"
> > meta              = "master:511b4194165ed7a5645169e09c27db280d5a5316"
> > meta-initramfs    = "master:4d62e7f575e2a87197c74ab4639561b45eec0e60"
> > meta-handheld     = "master:55a310666b543e6beca47fa3c197492d5a6cf8ff"
> >
> > Cheers
> FYI
> In the hurry for a solution for fixing formfactor and ipaq-boot-params
> my quick hack was to revert
> 
> http://cgit.openembedded.org/openembedded-core/commit/meta?id=92cbf7eeea553bfa24c7081473fa8bc4ebc1f552
> 
> That appears to fix the specific issue...

Its a hacky workaround. The real issue is that you don't set DISTRO and
this puts a :: into FILESOVERRIDES (and OVERRIDES). I've sent a patch
out which sets a default for DISTRO since its the nicer way I could find
to fix this. The behaviour in the above patch is correct.

Cheers,

Richard