Changeset d793097 in rtems_waf

Timestamp:

Jul 8, 2016, 6:34:12 AM (5 years ago)

Author:

Christian Mauderer <Christian.Mauderer@…>

Branches:

1a118bbcd52138dbdc3236e64bc23fd430a064b1

Children:

8fcd500

Parents:

254b827

git-author:

Christian Mauderer <Christian.Mauderer@…> (07/08/16 06:34:12)

git-committer:

Chris Johns <chrisj@…> (07/09/16 04:18:26)

Message:

Fix the --show-commands option.

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.

File:

• rtems.py (modified) (1 diff)

rtems.py

@@ -572 +572 @@
             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: