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

Side by Side Diff: components/cronet/tools/cr_cronet.py

Issue 1945023004: [Cronet] run pylint in Cronet presubmit check (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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 | « components/cronet/pylintrc ('k') | components/cronet/tools/extract_from_jars.py » ('j') | 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/python 1 #!/usr/bin/python
2 # Copyright 2014 The Chromium Authors. All rights reserved. 2 # Copyright 2014 The Chromium 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 """ 6 """
7 cr_cronet.py - cr - like helper tool for cronet developers 7 cr_cronet.py - cr - like helper tool for cronet developers
8 """ 8 """
9 9
10 import argparse 10 import argparse
(...skipping 111 matching lines...) Expand 10 before | Expand all | Expand 10 after
122 if (options.command=='test'): 122 if (options.command=='test'):
123 return install(release_arg) or test(out_dir, extra_options) 123 return install(release_arg) or test(out_dir, extra_options)
124 if (options.command=='build-test'): 124 if (options.command=='build-test'):
125 return build(out_dir, test_target) or install(release_arg) or \ 125 return build(out_dir, test_target) or install(release_arg) or \
126 test(out_dir, extra_options) 126 test(out_dir, extra_options)
127 if (options.command=='stack'): 127 if (options.command=='stack'):
128 return stack(out_dir) 128 return stack(out_dir)
129 if (options.command=='debug'): 129 if (options.command=='debug'):
130 return install(release_arg) or debug(extra_options) 130 return install(release_arg) or debug(extra_options)
131 if (options.command=='build-debug'): 131 if (options.command=='build-debug'):
132 return build(out_dir) or install(release_arg) or debug(extra_options) 132 return build(out_dir, test_target) or install(release_arg) or \
133 debug(extra_options)
133 else: 134 else:
134 if (options.command=='test'): 135 if (options.command=='test'):
135 return test_ios(out_dir, extra_options) 136 return test_ios(out_dir, extra_options)
136 if (options.command=='build-test'): 137 if (options.command=='build-test'):
137 return build(out_dir, test_target) or test_ios(out_dir, extra_options) 138 return build(out_dir, test_target) or test_ios(out_dir, extra_options)
138 139
139 parser.print_help() 140 parser.print_help()
140 return 1 141 return 1
141 142
142 143
143 if __name__ == '__main__': 144 if __name__ == '__main__':
144 sys.exit(main()) 145 sys.exit(main())
OLDNEW
« no previous file with comments | « components/cronet/pylintrc ('k') | components/cronet/tools/extract_from_jars.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698