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

Unified Diff: tools/foozzie/v8_foozzie.py

Issue 2598323002: [foozzie] Fix paths and resources (Closed)
Patch Set: Created 4 years 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 | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/foozzie/v8_foozzie.py
diff --git a/tools/foozzie/v8_foozzie.py b/tools/foozzie/v8_foozzie.py
index aeb1b4e7518b20492f3135999080b34b4f5743a7..222a3d8cfb2600817275ad9617a56d77dd547a31 100755
--- a/tools/foozzie/v8_foozzie.py
+++ b/tools/foozzie/v8_foozzie.py
@@ -117,17 +117,9 @@ def parse_args():
os.path.isfile(options.testcase)), (
'Test case %s doesn\'t exist' % options.testcase)
- resources_path = os.path.join(
+ options.meta_data_path = os.path.join(
os.path.dirname(options.testcase),
- 'resources' + os.path.basename(options.testcase)[len('fuzz'):])
- assert os.path.exists(resources_path), (
- 'Resources file %s doesn\'t exist' % resources_path)
-
- with open(resources_path) as f:
- resources = f.read().strip().splitlines()
- assert len(resources) == 1
- options.meta_data_path = os.path.join(
- os.path.dirname(resources_path), resources[0])
+ 'meta' + os.path.basename(options.testcase)[len('fuzz'):])
assert os.path.exists(options.meta_data_path), (
'Metadata %s doesn\'t exist' % options.meta_data_path)
@@ -135,8 +127,10 @@ def parse_args():
options.second_d8 = options.second_d8 or options.first_d8
# Ensure absolute paths.
- options.first_d8 = os.path.abspath(options.first_d8)
- options.second_d8 = os.path.abspath(options.second_d8)
+ if not os.path.isabs(options.first_d8):
+ options.first_d8 = os.path.join(BASE_PATH, options.first_d8)
+ if not os.path.isabs(options.second_d8):
+ options.second_d8 = os.path.join(BASE_PATH, options.second_d8)
# Ensure executables exist.
assert os.path.exists(options.first_d8)
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698