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

Side by Side Diff: tools/release/update_node.py

Issue 2751513003: Small fixes to update_node.py. (Closed)
Patch Set: 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 unified diff | Download patch
« no previous file with comments | « tools/release/testdata/v8/.gitignore ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright 2017 the V8 project authors. All rights reserved. 2 # Copyright 2017 the V8 project authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 import argparse 6 import argparse
7 import os 7 import os
8 import shutil 8 import shutil
9 import subprocess 9 import subprocess
10 import sys 10 import sys
11 11
12 TARGET_SUBDIR = os.path.join("deps", "v8") 12 TARGET_SUBDIR = os.path.join("deps", "v8")
13 13
14 SUB_REPOSITORIES = [ ["testing", "gtest"], 14 SUB_REPOSITORIES = [ ["testing", "gtest"],
15 ["third_party", "jinja2"], 15 ["third_party", "jinja2"],
16 ["third_party", "markupsafe"] ] 16 ["third_party", "markupsafe"] ]
17 17
18 DELETE_FROM_GITIGNORE = [ "/base", 18 DELETE_FROM_GITIGNORE = [ "/base",
19 "/testing/gtest" ] 19 "/testing/gtest",
20 "/third_party/jinja2",
21 "/third_party/markupsafe" ]
20 22
21 # Node.js requires only a single header file from gtest to build V8. 23 # Node.js requires only a single header file from gtest to build V8.
22 # Both jinja2 and markupsafe are required to generate part of the inspector. 24 # Both jinja2 and markupsafe are required to generate part of the inspector.
23 ADD_TO_GITIGNORE = [ "/testing/gtest/*", 25 ADD_TO_GITIGNORE = [ "/testing/gtest/*",
24 "!/testing/gtest/include", 26 "!/testing/gtest/include",
25 "/testing/gtest/include/*", 27 "/testing/gtest/include/*",
26 "!/testing/gtest/include/gtest", 28 "!/testing/gtest/include/gtest",
27 "/testing/gtest/include/gtest/*", 29 "/testing/gtest/include/gtest/*",
28 "!/testing/gtest/include/gtest/gtest_prod.h", 30 "!/testing/gtest/include/gtest/gtest_prod.h",
29 "!/third_party/jinja2", 31 "!/third_party/jinja2",
(...skipping 74 matching lines...) Expand 10 before | Expand all | Expand 10 after
104 RunGclient(options.v8_path) 106 RunGclient(options.v8_path)
105 # Update main V8 repository. 107 # Update main V8 repository.
106 UpdateTarget([""], options) 108 UpdateTarget([""], options)
107 # Patch .gitignore before updating sub-repositories. 109 # Patch .gitignore before updating sub-repositories.
108 UpdateGitIgnore(options) 110 UpdateGitIgnore(options)
109 for repo in SUB_REPOSITORIES: 111 for repo in SUB_REPOSITORIES:
110 UpdateTarget(repo, options) 112 UpdateTarget(repo, options)
111 113
112 if __name__ == "__main__": 114 if __name__ == "__main__":
113 Main(sys.argv[1:]) 115 Main(sys.argv[1:])
OLDNEW
« no previous file with comments | « tools/release/testdata/v8/.gitignore ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698