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>
diff --git a/scripts/autotest_quickmerge.py b/scripts/autotest_quickmerge.py
index a6a1fa7..09542d7 100644
--- a/scripts/autotest_quickmerge.py
+++ b/scripts/autotest_quickmerge.py
@@ -25,6 +25,7 @@
if cros_build_lib.IsInsideChroot():
+ # pylint: disable=F0401
# Only import portage after we've checked that we're inside the chroot.
import portage
@@ -231,8 +232,8 @@
considered owned by that package.
Args:
- changereport: ItemizedChangeReport object for the changes to be
- made to the package.
+ change_report: ItemizedChangeReport object for the changes to be
+ made to the package.
package_cp: A string similar to 'chromeos-base/autotest-tests' giving
the package category and name of the package to be altered.
portage_root: Portage root path, corresponding to the board that
@@ -271,10 +272,13 @@
Command will take form `rsync -a [options] --exclude=**.pyc
--exclude=**.pyo
- [optional --include-from argument]
+ [optional --include-from include_pattern_file]
--exclude=* [source_path] [sysroot_autotest_path]`
Args:
+ source_path: Directory to rsync from.
+ sysroot_autotest_path: Directory to rsync too.
+ include_pattern_file: Optional pattern of files to include in rsync.
pretend: True to use the '-n' option to rsync, to perform dry run.
overwrite: True to omit '-u' option, overwrite all files in sysroot,
not just older files.