[PATCH rtems-tools] check.py: Fix incorrect use of os.linesep

Chris Johns chrisj at rtems.org
Tue Sep 14 22:49:49 UTC 2021


OK and thanks

Chris

On 15/9/21 1:29 am, Ryan Long wrote:
> Replaced a couple of calls to os.linesep() with os.linesep because
> os.linesep() does not exist.
> ---
>  tester/rt/check.py | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/tester/rt/check.py b/tester/rt/check.py
> index 95fa47e..c01c25d 100755
> --- a/tester/rt/check.py
> +++ b/tester/rt/check.py
> @@ -966,13 +966,13 @@ class builder:
>      def _create_config(self, job, commands):
>          filename = 'config-%s-%s-%s.ini' % (job.arch, job.bsp, job.build)
>          cfg_file = open(path.join(self.rtems, filename),'w+')
> -        cfg_file.write('[%s/%s]' + os.linsep() % (job.arch, job.bsp))
> +        cfg_file.write('[%s/%s]' % (job.arch, job.bsp) + os.linesep)
>          new_cfg_cmds = []
>          for option in commands['configure'].split():
>              if 'waf' in option or '--' in option or 'configure' in option:
>                  new_cfg_cmds += [option]
>              else:
> -                cfg_file.write(option + os.linesep())
> +                cfg_file.write(option + os.linesep)
>          commands['configure'] = ' '.join(new_cfg_cmds)
>          cfg_file.close()
>  
> 


More information about the devel mailing list