aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJanusz Dziedzic <janusz.dziedzic@tieto.com>2016-04-29 05:07:32 (GMT)
committerJouni Malinen <j@w1.fi>2016-05-14 14:19:59 (GMT)
commit6a003eb2a6d5a33eba3e141679abb55052542097 (patch)
treeb5f249856ddf0fd2b99e2a2d00af5da679740a0e
parent3b11ad34ebac7616255577729d731e6a363e8e75 (diff)
downloadhostap-6a003eb2a6d5a33eba3e141679abb55052542097.zip
hostap-6a003eb2a6d5a33eba3e141679abb55052542097.tar.gz
hostap-6a003eb2a6d5a33eba3e141679abb55052542097.tar.bz2
tests: remotehost.py use join
Use join instead of for. Signed-off-by: Janusz Dziedzic <janusz.dziedzic@tieto.com>
-rw-r--r--tests/hwsim/remotehost.py13
1 files changed, 4 insertions, 9 deletions
diff --git a/tests/hwsim/remotehost.py b/tests/hwsim/remotehost.py
index 213c2fc..6895fc2 100644
--- a/tests/hwsim/remotehost.py
+++ b/tests/hwsim/remotehost.py
@@ -11,6 +11,8 @@ import threading
logger = logging.getLogger()
def execute_thread(command, reply):
+ cmd = ' '.join(command)
+ logger.debug("thread run: " + cmd)
try:
status = 0;
buf = subprocess.check_output(command, stderr=subprocess.STDOUT)
@@ -18,9 +20,6 @@ def execute_thread(command, reply):
status = e.returncode
buf = e.output
- cmd = ""
- for c in command:
- cmd = cmd + " " + c
logger.debug("thread cmd: " + cmd)
logger.debug("thread exit status: " + str(status))
logger.debug("thread exit buf: " + str(buf))
@@ -55,9 +54,7 @@ class Host():
return self.local_execute(command)
cmd = ["ssh", self.user + "@" + self.host, ' '.join(command)]
- _cmd = self.name + " execute: "
- for c in cmd:
- _cmd = _cmd + " " + c
+ _cmd = self.name + " execute: " + ' '.join(cmd)
logger.debug(_cmd)
try:
status = 0
@@ -76,9 +73,7 @@ class Host():
cmd = command
else:
cmd = ["ssh", self.user + "@" + self.host, ' '.join(command)]
- _cmd = self.name + " execute_run: "
- for c in cmd:
- _cmd = _cmd + " " + c
+ _cmd = self.name + " execute_run: " + ' '.join(cmd)
logger.debug(_cmd)
t = threading.Thread(target = execute_thread, args=(cmd, res))
t.start()