commit 25f309abfacfeaadb205dad2b01597edd151408b
author Don Garrett <dgarrett@google.com> Wed Mar 19 14:02:12 2014 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Thu Mar 20 23:52:56 2014 +0000
tree 93a71c02f9b26eda1910ace6bfc4957ffcd4a7d9
parent 0e03049082c78dd0b80a4ff30cb10778675e3c72

Fix cros lint issues in chromite/scripts.

Fix or disable all cros lint warnings in scripts.

BUG=None
TEST=Ran pylint.

Change-Id: Iedd2f2e361ea82c082f1c95c9054ac15d19193ff
Reviewed-on: https://chromium-review.googlesource.com/190730
Reviewed-by: Don Garrett <dgarrett@chromium.org>
Tested-by: Don Garrett <dgarrett@chromium.org>
Commit-Queue: Don Garrett <dgarrett@chromium.org>

scripts/cros_setup_toolchains.py

diff --git a/scripts/cros_setup_toolchains.py b/scripts/cros_setup_toolchains.py
index dce7e50..7735ed3 100644
--- a/scripts/cros_setup_toolchains.py
+++ b/scripts/cros_setup_toolchains.py
@@ -25,6 +25,7 @@
   # Only import portage after we've checked that we're inside the chroot.
   # Outside may not have portage, in which case the above may not happen.
   # We'll check in main() if the operation needs portage.
+  # pylint: disable=F0401
   import portage
 
 
@@ -110,9 +111,9 @@
     """Calls crossdev to initialize a cross target.
 
     Args:
-      targets - the list of targets to initialize using crossdev
-      usepkg - copies the commandline opts
-      config_only - Just update
+      targets: The list of targets to initialize using crossdev.
+      usepkg: Copies the commandline opts.
+      config_only: Just update.
     """
     configured_targets = cls._CACHE.setdefault('configured_targets', [])
 
@@ -906,8 +907,9 @@
   Tarballs (one per target) will be created in $PWD.
 
   Args:
-    targets_wanted: The targets to package up
-    root: The root path to pull all packages/files from
+    targets_wanted: The targets to package up.
+    output_dir: The directory to put the packages in.
+    root: The root path to pull all packages/files from.
   """
   osutils.SafeMakedirs(output_dir)
   ldpaths = lddtree.LoadLdpaths(root)