[PATCH] Fix --show-commands.

Christian Mauderer christian.mauderer at embedded-brains.de
Fri Jun 8 06:21:55 UTC 2018


Am 06.06.2018 um 09:48 schrieb Christian Mauderer:
> In the current version of libbsd, if the --show-commands option is used,
> the cwd is passed as a Nod3. Popen does not work with that. Therefore
> create a string from cwd if it isn't already.
> ---
>  rtems.py | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/rtems.py b/rtems.py
> index 6907709..a88c679 100644
> --- a/rtems.py
> +++ b/rtems.py
> @@ -579,6 +579,8 @@ def output_command_line():
>          else:
>              cmdstr = ' '.join(cmd)
>              Logs.info('(%d) %s' % (len(cmdstr), cmdstr)) # here is the change
> +        if not isinstance(kw['cwd'], str):
> +            kw['cwd'] = str(kw['cwd'])
>          Logs.debug('runner_env: kw=%s' % kw)
>          try:
>              if self.logger:
> 

Hello Chris,

do you see any problems with that patch?

Please note that this is for your personal rtems_waf.git repo. So I
can't check it in even if it would be acceptable. But if it is OK for
you, I'll create and commit a patch for libbsd to use the new version as
soon as this patch is commited.

Best regards

Christian Mauderer
-- 
--------------------------------------------
embedded brains GmbH
Herr Christian Mauderer
Dornierstr. 4
D-82178 Puchheim
Germany
email: christian.mauderer at embedded-brains.de
Phone: +49-89-18 94 741 - 18
Fax:   +49-89-18 94 741 - 08
PGP: Public key available on request.

Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.


More information about the devel mailing list