Change in osmo-gsm-tester[master]: log: process.py: fix weird space in stdout/stderr log output

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

neels gerrit-no-reply at lists.osmocom.org
Thu Dec 10 23:42:08 UTC 2020


neels has submitted this change. ( https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21514 )

Change subject: log: process.py: fix weird space in stdout/stderr log output
......................................................................

log: process.py: fix weird space in stdout/stderr log output

Log output before:

  stdout:
   | line1
  | line2
  | line3

After:

  stdout:
  | line1
  | line2
  | line3

Change-Id: I4202e7dfbb550acaf55a6c6487fc07819ab18b94
---
M src/osmo_gsm_tester/core/process.py
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  pespin: Looks good to me, approved



diff --git a/src/osmo_gsm_tester/core/process.py b/src/osmo_gsm_tester/core/process.py
index bff8dba..9cde7de 100644
--- a/src/osmo_gsm_tester/core/process.py
+++ b/src/osmo_gsm_tester/core/process.py
@@ -291,13 +291,13 @@
         m = self.get_stdout_tail(prefix='| ')
         if not m:
             return
-        self.log('stdout:\n', m, '\n')
+        self.log('stdout:', '\n' + m, '\n')
 
     def log_stderr_tail(self):
         m = self.get_stderr_tail(prefix='| ')
         if not m:
             return
-        self.log('stderr:\n', m, '\n')
+        self.log('stderr:', '\n' + m, '\n')
 
     def close_output_logs(self):
         for k, v in self.outputs.items():

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21514
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I4202e7dfbb550acaf55a6c6487fc07819ab18b94
Gerrit-Change-Number: 21514
Gerrit-PatchSet: 4
Gerrit-Owner: neels <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20201210/e8b55c2f/attachment.htm>


More information about the gerrit-log mailing list