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

Unified Diff: mojo/tools/mojob.py

Issue 921053003: Add option to allow mojob to specify extra gn args. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Fix qsr's comments Created 5 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
Index: mojo/tools/mojob.py
diff --git a/mojo/tools/mojob.py b/mojo/tools/mojob.py
index 7f60900656af4c6251c0fde73c841b9bdae25b0c..817632842ef30bedac1217df5aa280bacf3fb83d 100755
--- a/mojo/tools/mojob.py
+++ b/mojo/tools/mojob.py
@@ -64,6 +64,9 @@ def _args_to_config(args):
if 'test_results_server' in args:
additional_args['test_results_server'] = args.test_results_server
+ if 'extra_args' in args:
+ additional_args['extra_args'] = args.extra_args
qsr 2015/03/10 10:02:38 Could you use gn_args instead of extra_args?
haltonhuo 2015/03/11 05:58:06 Done.
+
return Config(target_os=target_os, target_cpu=target_cpu,
is_debug=args.debug, dcheck_always_on=args.dcheck_always_on,
**additional_args)
@@ -215,6 +218,8 @@ def main():
gn_parser = subparsers.add_parser('gn', parents=[parent_parser],
help='Run gn for mojo (does not sync).')
gn_parser.set_defaults(func=gn)
+ gn_parser.add_argument('--args', help='Specify extra args',
+ default=None, dest='extra_args')
clang_group = gn_parser.add_mutually_exclusive_group()
clang_group.add_argument('--clang', help='Use Clang (default)', default=None,
action='store_true')
« no previous file with comments | « mojo/public/tools/BUILD.gn ('k') | mojo/tools/mopy/gn.py » ('j') | mojo/tools/mopy/gn.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698