All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Chen Qi <Qi.Chen@windriver.com>, bitbake-devel@lists.openembedded.org
Subject: Re: [PATCH] bitbake: clean up stamp-base related codes
Date: Wed, 06 Jan 2016 11:22:42 +0000	[thread overview]
Message-ID: <1452079362.7598.67.camel@linuxfoundation.org> (raw)
In-Reply-To: <1452069127-13245-1-git-send-email-Qi.Chen@windriver.com>

On Wed, 2016-01-06 at 16:32 +0800, Chen Qi wrote:
> The 'stamp-base' related codes are no longer useful, clean them up.
> 
> [YOCTO #8468]
> 
> Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
> ---
>  bitbake/lib/bb/build.py | 4 ++--
>  bitbake/lib/bb/cache.py | 3 ---
>  2 files changed, 2 insertions(+), 5 deletions(-)
> 
> diff --git a/bitbake/lib/bb/build.py b/bitbake/lib/bb/build.py
> index 22428a6..b84338b 100644
> --- a/bitbake/lib/bb/build.py
> +++ b/bitbake/lib/bb/build.py
> @@ -582,10 +582,10 @@ def stamp_internal(taskname, d, file_name,
> baseonly=False):
>          taskflagname = taskname.replace("_setscene", "")
>  
>      if file_name:
> -        stamp = d.stamp_base[file_name].get(taskflagname) or
> d.stamp[file_name]
> +        stamp = d.stamp[file_name]
>          extrainfo = d.stamp_extrainfo[file_name].get(taskflagname)
> or ""
>      else:
> -        stamp = d.getVarFlag(taskflagname, 'stamp-base', True) or
> d.getVar('STAMP', True)
> +        stamp = d.getVar('STAMP', True)
>          file_name = d.getVar('BB_FILENAME', True)
>          extrainfo = d.getVarFlag(taskflagname, 'stamp-extra-info',
> True) or ""
>  
> diff --git a/bitbake/lib/bb/cache.py b/bitbake/lib/bb/cache.py
> index 521f758..0ec4057 100644
> --- a/bitbake/lib/bb/cache.py
> +++ b/bitbake/lib/bb/cache.py
> @@ -129,7 +129,6 @@ class CoreRecipeInfo(RecipeInfoCommon):
>          self.not_world = self.getvar('EXCLUDE_FROM_WORLD', metadata)
>          self.stamp = self.getvar('STAMP', metadata)
>          self.stampclean = self.getvar('STAMPCLEAN', metadata)       
> -        self.stamp_base = self.flaglist('stamp-base', self.tasks,
> metadata)
>          self.stamp_base_clean = self.flaglist('stamp-base-clean',


Ok, but what about stamp_base_clean?

Cheers,

Richard


> self.tasks, metadata)
>          self.stamp_extrainfo = self.flaglist('stamp-extra-info',
> self.tasks, metadata)
>          self.file_checksums = self.flaglist('file-checksums',
> self.tasks, metadata, True)
> @@ -158,7 +157,6 @@ class CoreRecipeInfo(RecipeInfoCommon):
>  
>          cachedata.stamp = {}
>          cachedata.stampclean = {}
> -        cachedata.stamp_base = {}
>          cachedata.stamp_base_clean = {}
>          cachedata.stamp_extrainfo = {}
>          cachedata.file_checksums = {}
> @@ -192,7 +190,6 @@ class CoreRecipeInfo(RecipeInfoCommon):
>          cachedata.pkg_dp[fn] = self.defaultpref
>          cachedata.stamp[fn] = self.stamp
>          cachedata.stampclean[fn] = self.stampclean
> -        cachedata.stamp_base[fn] = self.stamp_base
>          cachedata.stamp_base_clean[fn] = self.stamp_base_clean
>          cachedata.stamp_extrainfo[fn] = self.stamp_extrainfo
>          cachedata.file_checksums[fn] = self.file_checksums
> -- 
> 1.9.1
> 


  reply	other threads:[~2016-01-06 11:22 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-06  8:32 [PATCH] bitbake: clean up stamp-base related codes Chen Qi
2016-01-06 11:22 ` Richard Purdie [this message]
  -- strict thread matches above, loose matches on Subject: below --
2016-01-06  8:20 Chen Qi
2016-01-06  8:34 ` ChenQi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1452079362.7598.67.camel@linuxfoundation.org \
    --to=richard.purdie@linuxfoundation.org \
    --cc=Qi.Chen@windriver.com \
    --cc=bitbake-devel@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.