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

Unified Diff: tools/isolate_driver.py

Issue 1329783003: Revert of Fix sbox_validation_tests & Make isolate_driver.py include the executable itself (Closed) Base URL: https://chromium.googlesource.com/a/chromium/src.git@2_process
Patch Set: Created 5 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 | « tools/gn/gn_unittests.isolate ('k') | ui/accessibility/accessibility_unittests.isolate » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/isolate_driver.py
diff --git a/tools/isolate_driver.py b/tools/isolate_driver.py
index 3d2b7551979e765db14a8bad868837821457fa83..c5d80fc01a1a04aaa26e300ddaa5dbf1abc620db 100755
--- a/tools/isolate_driver.py
+++ b/tools/isolate_driver.py
@@ -118,9 +118,9 @@
if item.endswith('.isolated.gen.json'):
return True
IGNORED = (
- '.a', '.cc', '.css', '.dat', '.def', '.frag', '.h', '.html', '.isolate',
- '.js', '.json', '.manifest', '.o', '.obj', '.pak', '.png', '.pdb', '.py',
- '.strings', '.test', '.txt', '.vert',
+ '.a', '.cc', '.css', '.def', '.frag', '.h', '.html', '.js', '.json',
+ '.manifest', '.o', '.obj', '.pak', '.png', '.pdb', '.strings', '.test',
+ '.txt', '.vert',
)
# ninja files use native path format.
ext = os.path.splitext(item)[1]
@@ -180,12 +180,11 @@
return i[:-4]
return i
- def is_exe(i):
+ def f(i):
# This script is only for adding new binaries that are created as part of
# the component build.
ext = os.path.splitext(i)[1]
- # On POSIX, executables have no extension.
- if ext not in ('', '.dll', '.dylib', '.exe', '.nexe', '.so'):
+ if ext not in ['.dll', '.nexe', '.so', '.dylib']:
return False
# Check for execute access and strip directories. This gets rid of all the
@@ -193,7 +192,7 @@
p = os.path.join(build_dir, i)
return os.access(p, os.X_OK) and not os.path.isdir(p)
- return filter(is_exe, map(filter_item, dependencies))
+ return filter(f, map(filter_item, dependencies))
def create_wrapper(args, isolate_index, isolated_index):
« no previous file with comments | « tools/gn/gn_unittests.isolate ('k') | ui/accessibility/accessibility_unittests.isolate » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698