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

Side by Side Diff: build/android/gyp/lint.py

Issue 2241973002: Reland of Fail builds when lint warnings are generated. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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 | build/android/lint/suppressions.xml » ('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/env python 1 #!/usr/bin/env python
2 # 2 #
3 # Copyright (c) 2013 The Chromium Authors. All rights reserved. 3 # Copyright (c) 2013 The Chromium Authors. All rights reserved.
4 # Use of this source code is governed by a BSD-style license that can be 4 # Use of this source code is governed by a BSD-style license that can be
5 # found in the LICENSE file. 5 # found in the LICENSE file.
6 6
7 """Runs Android's lint tool.""" 7 """Runs Android's lint tool."""
8 8
9 9
10 import argparse 10 import argparse
(...skipping 313 matching lines...) Expand 10 before | Expand all | Expand 10 after
324 324
325 for gyp_list in args.resource_sources: 325 for gyp_list in args.resource_sources:
326 resource_sources += build_utils.ParseGypList(gyp_list) 326 resource_sources += build_utils.ParseGypList(gyp_list)
327 327
328 for resource_source in resource_sources: 328 for resource_source in resource_sources:
329 if os.path.isdir(resource_source): 329 if os.path.isdir(resource_source):
330 input_paths.extend(build_utils.FindInDirectory(resource_source, '*')) 330 input_paths.extend(build_utils.FindInDirectory(resource_source, '*'))
331 else: 331 else:
332 input_paths.append(resource_source) 332 input_paths.append(resource_source)
333 333
334 input_strings = [] 334 input_strings = [
335 args.can_fail_build,
336 args.silent,
337 ]
335 if args.android_sdk_version: 338 if args.android_sdk_version:
336 input_strings.append(args.android_sdk_version) 339 input_strings.append(args.android_sdk_version)
337 if args.processed_config_path: 340 if args.processed_config_path:
338 input_strings.append(args.processed_config_path) 341 input_strings.append(args.processed_config_path)
339 342
340 output_paths = [ args.result_path ] 343 output_paths = [ args.result_path ]
341 344
342 build_utils.CallAndWriteDepfileIfStale( 345 build_utils.CallAndWriteDepfileIfStale(
343 lambda: _OnStaleMd5(args.lint_path, 346 lambda: _OnStaleMd5(args.lint_path,
344 args.config_path, 347 args.config_path,
345 args.processed_config_path, 348 args.processed_config_path,
346 args.manifest_path, args.result_path, 349 args.manifest_path, args.result_path,
347 args.product_dir, sources, 350 args.product_dir, sources,
348 args.jar_path, 351 args.jar_path,
349 args.cache_dir, 352 args.cache_dir,
350 args.android_sdk_version, 353 args.android_sdk_version,
351 resource_sources, 354 resource_sources,
352 classpath=classpath, 355 classpath=classpath,
353 can_fail_build=args.can_fail_build, 356 can_fail_build=args.can_fail_build,
354 silent=args.silent), 357 silent=args.silent),
355 args, 358 args,
356 input_paths=input_paths, 359 input_paths=input_paths,
357 input_strings=input_strings, 360 input_strings=input_strings,
358 output_paths=output_paths, 361 output_paths=output_paths,
359 depfile_deps=classpath) 362 depfile_deps=classpath)
360 363
361 364
362 if __name__ == '__main__': 365 if __name__ == '__main__':
363 sys.exit(main()) 366 sys.exit(main())
OLDNEW
« no previous file with comments | « no previous file | build/android/lint/suppressions.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698