Re: [oe-commits] Richard Purdie : gcc-common.inc: Ensure WORKDIR doesn' t adversly affect sstate checksums

Re: [oe-commits] Richard Purdie : gcc-common.inc: Ensure WORKDIR doesn' t adversly affect sstate checksums

[Andreas Müller]

On Tue, Jan 24, 2012 at 1:29 PM, <git@git.openembedded.org> wrote:
>
> Module: openembedded-core.git
> Branch: master
> Commit: 0de85d2ca3bc9674323302a35995dfddb6044a71
> URL:    http://git.openembedded.org/?p=openembedded-core.git&a=commit;h=0de85d2ca3bc9674323302a35995dfddb6044a71
>
> Author: Richard Purdie <richard.purdie@linuxfoundation.org>
> Date:   Tue Jan 24 12:14:19 2012 +0000
>
> gcc-common.inc: Ensure WORKDIR doesn't adversly affect sstate checksums
>
> Changes in WORKDIR were changing the do_patch stamps between gcc-crosssdk
> and gcc-cross. This excludes the variable since in this case, we don't
> need want those changes to affect the signatures.
>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
>
> ---
>
>  meta/recipes-devtools/gcc/gcc-common.inc |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
>
> diff --git a/meta/recipes-devtools/gcc/gcc-common.inc b/meta/recipes-devtools/gcc/gcc-common.inc
> index ed41b0f..5ba68a0 100644
> --- a/meta/recipes-devtools/gcc/gcc-common.inc
> +++ b/meta/recipes-devtools/gcc/gcc-common.inc
> @@ -105,3 +105,7 @@ do_headerfix () {
>  addtask headerfix after do_unpack before do_patch
>
>  do_headerfix[vardepvalue] = "PATH"
> +
> +# We need to ensure that for the shared work directory, the do_patch singatures match
> +The real WORKDIR location isn't a dependency for the shared workdir.
Oops^^
> +src_patches[vardepsexclude] = "WORKDIR"

Aargh: sent to wrong list again :(

Andreas

Re: [oe-commits] Richard Purdie : gcc-common.inc: Ensure WORKDIR doesn' t adversly affect sstate checksums

[Richard Purdie]

On Tue, 2012-01-24 at 15:10 +0100, Andreas Müller wrote:
> On Tue, Jan 24, 2012 at 1:29 PM, <git@git.openembedded.org> wrote:
> >
> > Module: openembedded-core.git
> > Branch: master
> > Commit: 0de85d2ca3bc9674323302a35995dfddb6044a71
> > URL:    http://git.openembedded.org/?p=openembedded-core.git&a=commit;h=0de85d2ca3bc9674323302a35995dfddb6044a71
> >
> > Author: Richard Purdie <richard.purdie@linuxfoundation.org>
> > Date:   Tue Jan 24 12:14:19 2012 +0000
> >
> > gcc-common.inc: Ensure WORKDIR doesn't adversly affect sstate checksums
> >
> > Changes in WORKDIR were changing the do_patch stamps between gcc-crosssdk
> > and gcc-cross. This excludes the variable since in this case, we don't
> > need want those changes to affect the signatures.
> >
> > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> >
> > ---
> >
> >  meta/recipes-devtools/gcc/gcc-common.inc |    4 ++++
> >  1 files changed, 4 insertions(+), 0 deletions(-)
> >
> > diff --git a/meta/recipes-devtools/gcc/gcc-common.inc b/meta/recipes-devtools/gcc/gcc-common.inc
> > index ed41b0f..5ba68a0 100644
> > --- a/meta/recipes-devtools/gcc/gcc-common.inc
> > +++ b/meta/recipes-devtools/gcc/gcc-common.inc
> > @@ -105,3 +105,7 @@ do_headerfix () {
> >  addtask headerfix after do_unpack before do_patch
> >
> >  do_headerfix[vardepvalue] = "PATH"
> > +
> > +# We need to ensure that for the shared work directory, the do_patch singatures match
> > +The real WORKDIR location isn't a dependency for the shared workdir.
> Oops^^
> > +src_patches[vardepsexclude] = "WORKDIR"

Sorry about this, I pushed a fix earlier on.

Cheers,

Richard