diff options
| author | Bryn M. Reeves <bmr@redhat.com> | 2016-02-12 13:50:17 +0000 |
|---|---|---|
| committer | Bryn M. Reeves <bmr@redhat.com> | 2016-02-12 13:50:17 +0000 |
| commit | 7ea7917781b402fab287722cd1c7cd97ea39118c (patch) | |
| tree | 56fd5c785df2a9c8ca8f01a53d1a0b60775e059e | |
| parent | 184de3818abbf602d198a5030bdada7989c44b46 (diff) | |
| download | sos-7ea7917781b402fab287722cd1c7cd97ea39118c.tar.gz | |
[sosreport] push common code into _report_profiles_and_plugins
There's no point having both callers bracket the call with lines
that output a single blank line: do this all in the method itself.
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
| -rw-r--r-- | sos/sosreport.py | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/sos/sosreport.py b/sos/sosreport.py index 85093831..e192b938 100644 --- a/sos/sosreport.py +++ b/sos/sosreport.py @@ -1097,12 +1097,14 @@ class SoSReport(object): optparm)) def _report_profiles_and_plugins(self): + self.ui_log.info("") if len(self.loaded_plugins): self.ui_log.info(" %d profiles, %d plugins" % (len(self.profiles), len(self.loaded_plugins))) else: # no valid plugins for this profile self.ui_log.info(" %d profiles" % len(self.profiles)) + self.ui_log.info("") def list_plugins(self): if not self.loaded_plugins and not self.skipped_plugins: @@ -1154,9 +1156,7 @@ class SoSReport(object): lines = _format_list("Profiles: ", profiles, indent=True) for line in lines: self.ui_log.info(" %s" % line) - self.ui_log.info("") self._report_profiles_and_plugins() - self.ui_log.info("") def list_profiles(self): if not self.profiles: @@ -1178,9 +1178,7 @@ class SoSReport(object): lines = _format_list("%-15s " % profile, plugins, indent=True) for line in lines: self.ui_log.info(" %s" % line) - self.ui_log.info("") self._report_profiles_and_plugins() - self.ui_log.info("") def batch(self): if self.opts.batch: |