aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHumberto Ibarra <humberto.ibarra.lopez@intel.com>2016-04-05 22:15:54 -0500
committerRichard Purdie <richard.purdie@linuxfoundation.org>2016-04-05 23:12:41 +0100
commit03d1711350895e941a476bd2d2a6cd389be07509 (patch)
tree9805a03c85a4dd88b60856a4db64ca38165339f8
parent7157fd3bfbf2d5ae07fd4e726884baf7138bb24f (diff)
downloadopenembedded-core-contrib-03d1711350895e941a476bd2d2a6cd389be07509.tar.gz
scripts/oe-selftest: avoid the creation of coverage file when coverage not installed
Coverage subprocessing file is being created even when coverage is not installed, which causes errors of "module not found" to be send to the oe-selftest output. This patch adds indent to the block of code creating this coverage file, so it can only be executed when coverage is actually installed. [Yocto #9334] Signed-off-by: Humberto Ibarra <humberto.ibarra.lopez@intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rwxr-xr-xscripts/oe-selftest28
1 files changed, 14 insertions, 14 deletions
diff --git a/scripts/oe-selftest b/scripts/oe-selftest
index 375ba5c415..3d2dffb5c2 100755
--- a/scripts/oe-selftest
+++ b/scripts/oe-selftest
@@ -584,20 +584,20 @@ def buildResultClass(args):
if self.coverage_installed:
log.info("Coverage is enabled")
- # In case the user has not set the variable COVERAGE_PROCESS_START,
- # create a default one and export it. The COVERAGE_PROCESS_START
- # value indicates where the coverage configuration file resides
- # More info on https://pypi.python.org/pypi/coverage
- if not os.environ.get('COVERAGE_PROCESS_START'):
- os.environ['COVERAGE_PROCESS_START'] = coverage_setup(args.coverage_source, args.coverage_include, args.coverage_omit)
-
- # Use default site.USER_SITE and write corresponding config file
- site.ENABLE_USER_SITE = True
- if not os.path.exists(site.USER_SITE):
- os.makedirs(site.USER_SITE)
- self.coveragepth = os.path.join(site.USER_SITE, "coverage.pth")
- with open(self.coveragepth, 'w') as cps:
- cps.write('import sys,site; sys.path.extend(site.getsitepackages()); import coverage; coverage.process_startup();')
+ # In case the user has not set the variable COVERAGE_PROCESS_START,
+ # create a default one and export it. The COVERAGE_PROCESS_START
+ # value indicates where the coverage configuration file resides
+ # More info on https://pypi.python.org/pypi/coverage
+ if not os.environ.get('COVERAGE_PROCESS_START'):
+ os.environ['COVERAGE_PROCESS_START'] = coverage_setup(args.coverage_source, args.coverage_include, args.coverage_omit)
+
+ # Use default site.USER_SITE and write corresponding config file
+ site.ENABLE_USER_SITE = True
+ if not os.path.exists(site.USER_SITE):
+ os.makedirs(site.USER_SITE)
+ self.coveragepth = os.path.join(site.USER_SITE, "coverage.pth")
+ with open(self.coveragepth, 'w') as cps:
+ cps.write('import sys,site; sys.path.extend(site.getsitepackages()); import coverage; coverage.process_startup();')
def stopTestRun(self):
""" Report coverage data after the testcases are run """