[PATCH V2 0/1] buildstats-summary: add eventmask

[PATCH V2 0/1] buildstats-summary: add eventmask

[Chen Qi]

The following changes since commit 7ffe10df73cc20d10fcd41b121074445273bd60e:

  license_class: license_create_manifest improvment (2015-05-09 22:26:02 +0100)

are available in the git repository at:

  git://git.openembedded.org/openembedded-core-contrib ChenQi/buildstats-summary
  http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=ChenQi/buildstats-summary

Chen Qi (1):
  buildstats-summary: add eventmask

 meta/classes/buildstats-summary.bbclass | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

-- 
1.9.1

[PATCH V2 1/1] buildstats-summary: add eventmask

[Chen Qi]

Add eventmask 'bb.event.BuildCompleted' for buildstats_summary handler to avoid
running the codes in the handler everytime there's an event.

The codes in buildstats_summary only need to run for bb.event.BuildCompleted.
So add this event mask.

Also, we remove the redundant isinstance() statement in the handler.

Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
---
 meta/classes/buildstats-summary.bbclass | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/meta/classes/buildstats-summary.bbclass b/meta/classes/buildstats-summary.bbclass
index c8fbb2f..05ead9f 100644
--- a/meta/classes/buildstats-summary.bbclass
+++ b/meta/classes/buildstats-summary.bbclass
@@ -1,8 +1,5 @@
 # Summarize sstate usage at the end of the build
 python buildstats_summary () {
-    if not isinstance(e, bb.event.BuildCompleted):
-        return
-
     import collections
     import os.path
 
@@ -37,3 +34,4 @@ python buildstats_summary () {
             bb.note("  {0}: {1}% sstate reuse ({2} setscene, {3} scratch)".format(t, 100*len(sstate)/(len(sstate)+len(no_sstate)), len(sstate), len(no_sstate)))
 }
 addhandler buildstats_summary
+buildstats_summary[eventmask] = "bb.event.BuildCompleted"
-- 
1.9.1