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

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

Issue 2757593002: Add missing sub-repository in 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 | « no previous file | 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 = [ ["base", "trace_event", "common"],
15 ["testing", "gtest"],
15 ["third_party", "jinja2"], 16 ["third_party", "jinja2"],
16 ["third_party", "markupsafe"] ] 17 ["third_party", "markupsafe"] ]
17 18
18 DELETE_FROM_GITIGNORE = [ "/base", 19 DELETE_FROM_GITIGNORE = [ "/base",
19 "/testing/gtest", 20 "/testing/gtest",
20 "/third_party/jinja2", 21 "/third_party/jinja2",
21 "/third_party/markupsafe" ] 22 "/third_party/markupsafe" ]
22 23
23 # Node.js requires only a single header file from gtest to build V8. 24 # Node.js requires only a single header file from gtest to build V8.
24 # Both jinja2 and markupsafe are required to generate part of the inspector. 25 # Both jinja2 and markupsafe are required to generate part of the inspector.
(...skipping 97 matching lines...) Expand 10 before | Expand all | Expand 10 after
122 UpdateTarget([""], options) 123 UpdateTarget([""], options)
123 # Patch .gitignore before updating sub-repositories. 124 # Patch .gitignore before updating sub-repositories.
124 UpdateGitIgnore(options) 125 UpdateGitIgnore(options)
125 for repo in SUB_REPOSITORIES: 126 for repo in SUB_REPOSITORIES:
126 UpdateTarget(repo, options) 127 UpdateTarget(repo, options)
127 if options.commit: 128 if options.commit:
128 CreateCommit(options) 129 CreateCommit(options)
129 130
130 if __name__ == "__main__": 131 if __name__ == "__main__":
131 Main(sys.argv[1:]) 132 Main(sys.argv[1:])
OLDNEW
« 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