🌇 lint.py - print original exception for empty results.xml

build/android/gyp/lint.py

 #!/usr/bin/env python
 #
 # Copyright (c) 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Runs Android's lint tool."""
 
 
 import argparse
@@ skipping 118 matching lines @@
     if os.path.exists(result_path):
       os.remove(result_path)
 
     env = {}
     if cache_dir:
       env['_JAVA_OPTIONS'] = '-Duser.home=%s' % _RelativizePath(cache_dir)
 
     try:
       build_utils.CheckOutput(cmd, cwd=_SRC_ROOT, env=env or None)
     except build_utils.CalledProcessError:
-      if can_fail_build:
-        traceback.print_exc()
-
       # There is a problem with lint usage
       if not os.path.exists(result_path):
         raise
 
+      # Sometimes produces empty (almost) files:
+      if os.path.getsize(result_path) < 10:
+        if can_fail_build:
+          raise
+        elif not silent:
+          traceback.print_exc()
+          return

[jbudorick, 2016/03/22 23:46:12]

I don't think this should be in the elif.

[agrieve, 2016/03/23 00:03:07]

Done.

+
+      if can_fail_build and not silent:
+        traceback.print_exc()
+
       # There are actual lint issues
-      else:
-        try:
-          num_issues = _ParseAndShowResultFile()
-        except Exception: # pylint: disable=broad-except
-          if not silent:
-            print 'Lint created unparseable xml file...'
-            print 'File contents:'
-            with open(result_path) as f:
-              print f.read()
-          raise
+      try:
+        num_issues = _ParseAndShowResultFile()
+      except Exception: # pylint: disable=broad-except
+        if not silent:
+          print 'Lint created unparseable xml file...'
+          print 'File contents:'
+          with open(result_path) as f:
+            print f.read()
+        raise
 
-        _ProcessResultFile()
-        msg = ('\nLint found %d new issues.\n'
-               ' - For full explanation refer to %s\n' %
-               (num_issues,
-                _RelativizePath(result_path)))
-        if config_path:
-          msg += (' - Wanna suppress these issues?\n'
-                  '    1. Read comment in %s\n'
-                  '    2. Run "python %s %s"\n' %
-                  (_RelativizePath(config_path),
-                   _RelativizePath(os.path.join(_SRC_ROOT, 'build', 'android',
-                                                'lint', 'suppress.py')),
-                   _RelativizePath(result_path)))
-        if not silent:
-          print >> sys.stderr, msg
-        if can_fail_build:
-          raise Exception('Lint failed.')
+      _ProcessResultFile()
+      msg = ('\nLint found %d new issues.\n'
+             ' - For full explanation refer to %s\n' %
+             (num_issues,
+              _RelativizePath(result_path)))
+      if config_path:
+        msg += (' - Wanna suppress these issues?\n'
+                '    1. Read comment in %s\n'
+                '    2. Run "python %s %s"\n' %
+                (_RelativizePath(config_path),
+                 _RelativizePath(os.path.join(_SRC_ROOT, 'build', 'android',
+                                              'lint', 'suppress.py')),
+                 _RelativizePath(result_path)))
+      if not silent:
+        print >> sys.stderr, msg
+      if can_fail_build:
+        raise Exception('Lint failed.')
 
 
 def main():
   parser = argparse.ArgumentParser()
   build_utils.AddDepfileOption(parser)
 
   parser.add_argument('--lint-path', required=True,
                       help='Path to lint executable.')
   parser.add_argument('--product-dir', required=True,
                       help='Path to product dir.')
@@ skipping 87 matching lines @@
                                     silent=args.silent),
         args,
         input_paths=input_paths,
         input_strings=input_strings,
         output_paths=output_paths,
         pass_changes=True)
 
 
 if __name__ == '__main__':
   sys.exit(main())