Re: [PATCH] bdwgc: use github repo for source location

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Mon, 2016-04-04 at 18:33 -0700, Saul Wold wrote:
> On Mon, 2016-04-04 at 09:18 -0700, Bill Randle wrote:
> > The Yocto Autobuilder has been unable to fetch the release package
> > from the developer's website at www.hboehm.info, so change recipe
> > to fetch from the developer's github repo instead.
> > 
> > Signed-off-by: Bill Randle <william.c.randle@intel.com>
> > ---
> >  meta/recipes-support/bdwgc/bdwgc_7.4.2.bb | 6 ++----
> >  1 file changed, 2 insertions(+), 4 deletions(-)
> > 
> > diff --git a/meta/recipes-support/bdwgc/bdwgc_7.4.2.bb
> > b/meta/recipes-support/bdwgc/bdwgc_7.4.2.bb
> > index 95e358f..7a4110d 100644
> > --- a/meta/recipes-support/bdwgc/bdwgc_7.4.2.bb
> > +++ b/meta/recipes-support/bdwgc/bdwgc_7.4.2.bb
> > @@ -21,12 +21,10 @@ SECTION = "devel"
> >  LICENSE = "MIT"
> >  LIC_FILES_CHKSUM =
> > "file://README.QUICK;md5=55f5088f90a982fed7af9a4897047ef7"
> >  
> > -SRC_URI = "http://www.hboehm.info/gc/gc_source/gc-${PV}.tar.gz \
> > +SRCREV = "e31cdc288ffbee5bf25f948e19deb5a7ce846971"
> > +SRC_URI = "git://github.com/ivmai/bdwgc.git;branch=release-7_4 \
> >             file://0001-fix-build-with-musl.patch \
> >            "
> 
> It appears that when you updated to the git location the source base
> might have changed and the MUSL patch is no longer applyng correctly.
> 
> Please verify that the patch correctly applies.
> 
> See the failure in master-next with your patch:
> 
> https://autobuilder.yoctoproject.org/main/builders/nightly-no-x11/bui
> ld
> s/76/steps/BuildImages/logs/stdio
> 
> This might cause more failures in the Autobuilder, this is the first
> that I saw.

The patch is ok, but ${S} wasn't set correctly. I've tweaked in the
version I merged into master.

Cheers,

Richard