lint: Enable logging checker for cros_logging module
The 'logging-modules' pylint option allows us to apply the logging
checker to the cros_logging module.
Fixed all logging checker errors found by this new coverage.
BUG=none
TEST=`cros lint`: no more logging errors
Change-Id: I231f6f7bf376b7ebc433de3a23b2b1e73dd5fa9f
Reviewed-on: https://chromium-review.googlesource.com/1194187
Commit-Ready: Lann Martin <lannm@chromium.org>
Tested-by: Lann Martin <lannm@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>
diff --git a/scripts/crosfw.py b/scripts/crosfw.py
index 14e2208..35f022b 100644
--- a/scripts/crosfw.py
+++ b/scripts/crosfw.py
@@ -473,12 +473,12 @@
# Get the correct board for cros_write_firmware
config_mk = '%s/include/autoconf.mk' % outdir
if not os.path.exists(config_mk):
- logging.warning('No build found for %s - dropping -i' % board)
+ logging.warning('No build found for %s - dropping -i', board)
options.incremental = False
config_mk = 'include/autoconf.mk'
if os.path.exists(config_mk):
- logging.warning("Warning: '%s' exists, try 'make distclean'" % config_mk)
+ logging.warning("Warning: '%s' exists, try 'make distclean'", config_mk)
# For when U-Boot supports ccache
# See http://patchwork.ozlabs.org/patch/245079/
@@ -692,12 +692,12 @@
cros_build_lib.Die('cros_bundle_firmware failed')
if not dest or not result.returncode:
- logging.info('Image is available at %s/out/image.bin' % outdir)
+ logging.info('Image is available at %s/out/image.bin', outdir)
else:
if result.returncode:
cros_build_lib.Die('Failed to write image to board')
else:
- logging.info('Image written to board with %s' % ' '.join(dest + servo))
+ logging.info('Image written to board with %s', ' '.join(dest + servo))
def main(argv):