Since DownloadBitbakeErrorReports is also downloading reports
generated by RunOESelftest it don't make sense to have a hard
reference of bitbake in the buildstep.

Signed-off-by: Aníbal Limón <anibal.li...@linux.intel.com>
---
 .../buildsteps/DownloadBitbakeErrorReports.py      | 42 ----------------------
 .../autobuilder/buildsteps/DownloadErrorReports.py | 42 ++++++++++++++++++++++
 2 files changed, 42 insertions(+), 42 deletions(-)
 delete mode 100644 
lib/python2.7/site-packages/autobuilder/buildsteps/DownloadBitbakeErrorReports.py
 create mode 100644 
lib/python2.7/site-packages/autobuilder/buildsteps/DownloadErrorReports.py

diff --git 
a/lib/python2.7/site-packages/autobuilder/buildsteps/DownloadBitbakeErrorReports.py
 
b/lib/python2.7/site-packages/autobuilder/buildsteps/DownloadBitbakeErrorReports.py
deleted file mode 100644
index fe20247..0000000
--- 
a/lib/python2.7/site-packages/autobuilder/buildsteps/DownloadBitbakeErrorReports.py
+++ /dev/null
@@ -1,42 +0,0 @@
-'''
-Created on Jan 1, 2016
-
-__author__ = "Anibal (alimon) Limon"
-__copyright__ = "Copyright 2016, Intel Corp."
-__credits__ = ["Anibal Limon"]
-__license__ = "GPL"
-__version__ = "2.0"
-__maintainer__ = "Anibal Limon"
-__email__ = "anibal.li...@linux.intel.com"
-'''
-
-import os
-from buildbot.steps.transfer import DirectoryDownload
-from buildbot.process.buildstep import SKIPPED
-
-from lib.ABTools import get_error_report_worker_dir, \
-        get_error_report_controller_dir
-
-class DownloadBitbakeErrorReports(DirectoryDownload):
-    """
-        Transfer from master to slave the error reports when bitbake
-        fails.
-    """
-    name = "DownloadBitbakeErrorReports"
-
-    def __init__(self, factory, argdict=None, **kwargs):
-        super(DownloadBitbakeErrorReports, self).__init__(
-                "", "")
-
-    def start(self):
-        buildername = self.getProperty('buildername')
-        buildnumber = self.getProperty('buildnumber')
-
-        self.mastersrc = get_error_report_controller_dir(buildername,
-                buildnumber)
-        self.slavedest = get_error_report_worker_dir(buildername)
-
-        if not os.path.exists(self.mastersrc):
-            return SKIPPED
-
-        super(DownloadBitbakeErrorReports, self).start()
diff --git 
a/lib/python2.7/site-packages/autobuilder/buildsteps/DownloadErrorReports.py 
b/lib/python2.7/site-packages/autobuilder/buildsteps/DownloadErrorReports.py
new file mode 100644
index 0000000..696f56c
--- /dev/null
+++ b/lib/python2.7/site-packages/autobuilder/buildsteps/DownloadErrorReports.py
@@ -0,0 +1,42 @@
+'''
+Created on Jan 1, 2016
+
+__author__ = "Anibal (alimon) Limon"
+__copyright__ = "Copyright 2016, Intel Corp."
+__credits__ = ["Anibal Limon"]
+__license__ = "GPL"
+__version__ = "2.0"
+__maintainer__ = "Anibal Limon"
+__email__ = "anibal.li...@linux.intel.com"
+'''
+
+import os
+from buildbot.steps.transfer import DirectoryDownload
+from buildbot.process.buildstep import SKIPPED
+
+from lib.ABTools import get_error_report_worker_dir, \
+        get_error_report_controller_dir
+
+class DownloadErrorReports(DirectoryDownload):
+    """
+        Transfer from master to slave the error reports when bitbake
+        fails.
+    """
+    name = "DownloadErrorReports"
+
+    def __init__(self, factory, argdict=None, **kwargs):
+        super(DownloadErrorReports, self).__init__(
+                "", "")
+
+    def start(self):
+        buildername = self.getProperty('buildername')
+        buildnumber = self.getProperty('buildnumber')
+
+        self.mastersrc = get_error_report_controller_dir(buildername,
+                buildnumber)
+        self.slavedest = get_error_report_worker_dir(buildername)
+
+        if not os.path.exists(self.mastersrc):
+            return SKIPPED
+
+        super(DownloadErrorReports, self).start()
-- 
2.1.4

-- 
_______________________________________________
yocto mailing list
yocto@yoctoproject.org
https://lists.yoctoproject.org/listinfo/yocto

Reply via email to