Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(163)

Unified Diff: client/bin/job.py

Issue 3511003: Merge remote branch 'cros/upstream' into tempbranch2 (Closed) Base URL: http://git.chromium.org/git/autotest.git
Patch Set: Created 10 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | client/bin/kernel.py » ('j') | client/common_lib/utils.py » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: client/bin/job.py
diff --git a/client/bin/job.py b/client/bin/job.py
index 5adbc00e6e68a4b016c0ed0bcb76befc12c2cbc7..6dd19d18f94c420240277d9a21d0095fe9b28b14 100644
--- a/client/bin/job.py
+++ b/client/bin/job.py
@@ -503,7 +503,8 @@ class base_client_job(base_job.base_job):
raise error.TestError("Dependency %s does not exist" % dep)
os.chdir(dep_dir)
truty1 2010/09/28 22:59:03 I think this function should leave the os dir wher
- utils.system('./' + dep + '.py')
+ if execfile('%s.py' % dep, {}) is None:
+ logging.info('Dependency %s successfuly built', dep)
truty1 2010/09/28 22:59:03 execfile() always returns None - I think this logi
def _runtest(self, url, tag, args, dargs):
« no previous file with comments | « no previous file | client/bin/kernel.py » ('j') | client/common_lib/utils.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698