Skip to contentSkip to conversation

buildstats-summary: add eventmask

PATCH New Chen Qi 1 participant 2 messages May 20, 2015

lore
Share & tools
lore

Chen Qi

author
#0 lore raw

v2 cover buildstats-summary: add eventmask

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(-)
Signature · 1 line
1.9.1
author
#1 lore raw

PATCH v2 1/1 buildstats-summary: add eventmask

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.

---
 meta/classes/buildstats-summary.bbclass | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)
meta/classes/buildstats-summary.bbclass +1 -3
index c8fbb2f..05ead9f 100644
@@ -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"
Signature · 1 line
1.9.1