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

Unified Diff: test/test262/list.py

Issue 2733843002: test262 roll (Closed)
Patch Set: in a comment, s/gn/gyp Created 3 years, 9 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 | « test/test262/harness-agent.js ('k') | test/test262/test262.status » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/test262/list.py
diff --git a/test/test262/list.py b/test/test262/list.py
index 9b36ce789c5baa217d118ecb5e5294c86336ffa7..e99be68cd7fe9b5d41ab7ad3b704f3d09e1f38aa 100755
--- a/test/test262/list.py
+++ b/test/test262/list.py
@@ -18,4 +18,9 @@ for root, dirs, files in chain(os.walk("data"), os.walk("harness")):
# For gyp, quote the name in case it includes spaces
if len(sys.argv) > 1 and sys.argv[1] == '--quoted':
pathname = '"' + pathname + '"'
+ # Temporary hack until we upgrade to gn swarming:
+ # gyp doesn't handle files containing $ in the name very well, so we just
+ # exclude them from the 'sources' list and hope that other changes cause
+ # the archive to be rebuilt.
+ if '$' in pathname: continue
print(pathname)
« no previous file with comments | « test/test262/harness-agent.js ('k') | test/test262/test262.status » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698