commit f5318e1f391859c82aaa47b297429e50f41f6b3c
author kjellander <kjellander@webrtc.org> Thu Mar 09 02:26:46 2017 -0800
committer Commit bot <commit-bot@chromium.org> Thu Mar 09 10:26:46 2017 +0000
tree dc312f9f107d9f6f3d2cce70da5dc77f274fcad4
parent 735da3ad38b9387f3a2b4ef778a0c8c40a1e75ff

PyLint fixes for tools-webrtc and webrtc/tools

Fix a lot of errors before bringing in the new config in
https://codereview.webrtc.org/2737963003/

BUG=webrtc:7303
NOTRY=True

Review-Url: https://codereview.webrtc.org/2736233003
Cr-Commit-Position: refs/heads/master@{#17137}

tools-webrtc/get_landmines.py

diff --git a/tools-webrtc/get_landmines.py b/tools-webrtc/get_landmines.py
index dd0c9b0..95ee536 100755
--- a/tools-webrtc/get_landmines.py
+++ b/tools-webrtc/get_landmines.py
@@ -15,19 +15,16 @@
 import os
 import sys
 
-script_dir = os.path.dirname(os.path.realpath(__file__))
-checkout_root = os.path.abspath(os.path.join(script_dir, os.pardir))
-sys.path.insert(0, os.path.join(checkout_root, 'build'))
+SCRIPT_DIR = os.path.dirname(os.path.realpath(__file__))
+CHECKOUT_ROOT = os.path.abspath(os.path.join(SCRIPT_DIR, os.pardir))
+sys.path.insert(0, os.path.join(CHECKOUT_ROOT, 'build'))
 import landmine_utils
 
 
-distributor = landmine_utils.distributor
-gyp_defines = landmine_utils.gyp_defines
-gyp_msvs_version = landmine_utils.gyp_msvs_version
-platform = landmine_utils.platform
+platform = landmine_utils.platform  # pylint: disable=invalid-name
 
 
-def print_landmines():
+def print_landmines():  # pylint: disable=invalid-name
   """
   ALL LANDMINES ARE EMITTED FROM HERE.
   """
@@ -54,7 +51,7 @@
     # is no longer the case.
     print 'Clobber due to iOS compile errors (crbug.com/694721)'
     print 'Clobber to unblock https://codereview.webrtc.org/2709573003'
-    print 'Clobber to fix https://codereview.webrtc.org/2709573003 after landing'
+    print 'Clobber to fix after https://codereview.webrtc.org/2709573003'
 
 
 def main():