[PATCH] Fix the --show-commands option.

Christian Mauderer christian.mauderer at embedded-brains.de
Fri Jul 8 06:34:12 UTC 2016


From: Christian Mauderer <Christian.Mauderer at embedded-brains.de>

If the --show-commands is set, the cmd-string is joined for printing.
This breaks a later Popen call that expects an array instead of an fully
assembled string. This patch fixes the problem.
---
 rtems.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/rtems.py b/rtems.py
index 3c0c976..b2e3285 100644
--- a/rtems.py
+++ b/rtems.py
@@ -571,8 +571,8 @@ def output_command_line():
         if isinstance(cmd, str):
             Logs.info('%s' % cmd)
         else:
-            cmd = ' '.join(cmd)
-            Logs.info('(%d) %s' % (len(cmd), cmd)) # here is the change
+            cmdstr = ' '.join(cmd)
+            Logs.info('(%d) %s' % (len(cmdstr), cmdstr)) # here is the change
         Logs.debug('runner_env: kw=%s' % kw)
         try:
             if self.logger:
-- 
1.8.4.5




More information about the devel mailing list