Merge "pylint: handle missing pylint better" am: 97e923f9f0 am: 7e1147107a

am: dd79d2981e

Change-Id: I622d2ed470e0180c147b71051757ac8b35d3395f

1 files changed, 14 insertions, 1 deletions

diff --git a/tools/pylint.py b/tools/pylint.py
index 6091c1d..1c8e6f7 100755
--- a/tools/pylint.py
+++ b/tools/pylint.py
@@ -19,12 +19,15 @@
 from __future__ import print_function
 
 import argparse
+import errno
 import os
 import sys
 
+
 DEFAULT_PYLINTRC_PATH = os.path.join(
     os.path.dirname(os.path.realpath(__file__)), 'pylintrc')
 
+
 def get_parser():
     """Return a command line parser."""
     parser = argparse.ArgumentParser(description=__doc__)
@@ -62,7 +65,17 @@ def main(argv):
     if opts.init_hook:
         cmd += ['--init-hook', opts.init_hook]
 
-    os.execvp(cmd[0], cmd)
+    try:
+        os.execvp(cmd[0], cmd)
+    except OSError as e:
+        if e.errno == errno.ENOENT:
+            print('%s: unable to run `%s`: %s' % (__file__, cmd[0], e),
+                  file=sys.stderr)
+            print('%s: Try installing pylint: sudo apt-get install pylint' %
+                  (__file__,), file=sys.stderr)
+            return 1
+        else:
+            raise
 
 
 if __name__ == '__main__':