W1202 fix

Signed-off-by: Petr Hracek <phracek@redhat.com>
rebase-helper · Apr 9, 2015 · ea7cad5 · ea7cad5
1 parent c858fba
commit ea7cad5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/rebasehelper/application.py b/rebasehelper/application.py
@@ -358,10 +358,11 @@ def build_packages(self):
                         raise RebaseHelperError('Building package failed during patching. Check log %s', build_log_path)
 
                     if files['missing']:
-                        logger.info('Files not packaged in the SPEC file:\n{f}'.format(f='\n'.join(files['added'])))
+                        missing_files = '\n'.join(files['added'])
+                        logger.info('Files not packaged in the SPEC file:\n%s', missing_files)
                     elif files['deleted']:
-                        logger.warning('Removed files packaged in SPEC file:\n{f}'.format(
-                            f='\n'.join(files['deleted'])))
+                        deleted_files = '\n'.join(files['deleted'])
+                        logger.warning('Removed files packaged in SPEC file:\n%s', deleted_files)
                     else:
                         raise RebaseHelperError("Build failed, but no issues were found in the build log %s", build_log)
                     self.rebase_spec_file.modify_spec_files_section(files)

diff --git a/rebasehelper/output_tool.py b/rebasehelper/output_tool.py
@@ -58,7 +58,6 @@ def match(cls, cmd=None, *args, **kwargs):
         raise NotImplementedError()
 
     def print_summary(self, path, **kwargs):
-
         """
         Return list of files which has been changed against old version
         This will be used by checkers