osmith has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/37897?usp=email )

Change subject: testenv: fix duplicate 2>&1
......................................................................

testenv: fix duplicate 2>&1

The pipe variable already has 2>&1, therefore we don't need to add it in
cmd.

Change-Id: Ifeae4c53b57f90242041e61ca5ab7073d155d7a8
---
M _testenv/testenv/daemons.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  laforge: Looks good to me, approved
  Jenkins Builder: Verified




diff --git a/_testenv/testenv/daemons.py b/_testenv/testenv/daemons.py
index 9e274d9..c211cd7 100644
--- a/_testenv/testenv/daemons.py
+++ b/_testenv/testenv/daemons.py
@@ -49,7 +49,7 @@
             pipe = f"2>&1 | tee {shlex.quote(log)}"
         else:
             pipe = f">{shlex.quote(log)} 2>&1"
-        cmd = ["sh", "-c", f"{program} 2>&1 {pipe}"]
+        cmd = ["sh", "-c", f"{program} {pipe}"]

         env = {}
         if testenv.args.io_uring:

--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/37897?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: merged
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: Ifeae4c53b57f90242041e61ca5ab7073d155d7a8
Gerrit-Change-Number: 37897
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>

Reply via email to