diff options
author | Janusz Dziedzic <janusz.dziedzic@tieto.com> | 2016-04-29 05:07:32 (GMT) |
---|---|---|
committer | Jouni Malinen <j@w1.fi> | 2016-05-14 14:19:59 (GMT) |
commit | 6a003eb2a6d5a33eba3e141679abb55052542097 (patch) | |
tree | b5f249856ddf0fd2b99e2a2d00af5da679740a0e | |
parent | 3b11ad34ebac7616255577729d731e6a363e8e75 (diff) | |
download | hostap-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.py | 13 |
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() |