[yocto] [PATCH 1/1] Doing uname without subprocess: buildstats.bbclass

Elizabeth Flanagan elizabeth.flanagan at intel.com
Tue Feb 15 18:44:13 PST 2011


Instead of using subprocess which is problematic, we should use platform
instead.

Signed-off-by: Beth Flanagan <elizabeth.flanagan at intel.com>
---
 meta/classes/buildstats.bbclass |   11 +++++++----
 1 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/meta/classes/buildstats.bbclass b/meta/classes/buildstats.bbclass
index f042dd3..0acf516 100644
--- a/meta/classes/buildstats.bbclass
+++ b/meta/classes/buildstats.bbclass
@@ -67,7 +67,7 @@ python run_buildstats () {
     import bb.build
     import bb.event
     import bb.data
-    import time, subprocess
+    import time, subprocess, platform

     if isinstance(e, bb.event.BuildStarted):
         ##############################################
@@ -90,9 +90,12 @@ python run_buildstats () {
         build_time = os.path.join(bsdir, "build_stats")
         # write start of build into build_time
         file = open(build_time,"a")
-        # We do this here because subprocess within BuildStarted is messy
-        #host_info = subprocess.Popen(["uname", "-a"], stdout=subprocess.PIPE).stdout.read()
-        #file.write("Host Info: %s" % host_info)
+        host_info = platform.uname()
+        file.write("Host Info: ")
+        for x in host_info:
+            if x:
+                file.write(x + " ")
+        file.write("\n")
         file.write("Build Started: %0.2f \n" % time.time())
         file.close()

-- 
1.7.1



More information about the yocto mailing list