[PATCH] Fix --show-commands.

Gedare Bloom gedare at rtems.org
Fri Jun 8 12:33:52 UTC 2018


On Fri, Jun 8, 2018 at 2:21 AM, Christian Mauderer
<christian.mauderer at embedded-brains.de> wrote:
> 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.
>

rtems_waf should be migrated to the public set. it is now an integral
part of the infrastructure for waf builds of apps/libraries.

> 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.
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel


More information about the devel mailing list