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

Unified Diff: build/android/gyp/gcc_preprocess.py

Issue 132463007: Enable presubmit pylint in build/android. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase vs tot and only disabling F0401 in specific spots Created 6 years, 10 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « build/android/gyp/finalize_apk.py ('k') | build/android/gyp/generate_v14_compatible_resources.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/gyp/gcc_preprocess.py
diff --git a/build/android/gyp/gcc_preprocess.py b/build/android/gyp/gcc_preprocess.py
index 25192546053e181d35322a0d34887dd4dbe280a6..a5d4ba558488e85d42b078fcefef1ff9c6858464 100755
--- a/build/android/gyp/gcc_preprocess.py
+++ b/build/android/gyp/gcc_preprocess.py
@@ -6,10 +6,9 @@
import optparse
import os
-import subprocess
import sys
-from util import build_utils
+from util import build_utils # pylint: disable=F0401
def DoGcc(options):
build_utils.MakeDirectory(os.path.dirname(options.output))
@@ -30,7 +29,7 @@ def DoGcc(options):
build_utils.CheckOutput(gcc_cmd)
-def main(argv):
+def main():
parser = optparse.OptionParser()
parser.add_option('--include-path', help='Include path for gcc.')
parser.add_option('--template', help='Path to template.')
@@ -50,4 +49,4 @@ def main(argv):
if __name__ == '__main__':
- sys.exit(main(sys.argv))
+ sys.exit(main())
« no previous file with comments | « build/android/gyp/finalize_apk.py ('k') | build/android/gyp/generate_v14_compatible_resources.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698