<div dir="ltr">I was using Python 3, I just checked that it works fine with Python2.7 without the changes mentioned in the patch.<div class="gmail_extra"><br><div class="gmail_quote">On 11 April 2018 at 06:20, Chris Johns <span dir="ltr"><<a href="mailto:chrisj@rtems.org" target="_blank">chrisj@rtems.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 11/04/2018 07:04, <a href="mailto:vijaykumar9597@gmail.com">vijaykumar9597@gmail.com</a> wrote:<br>
> From: thelunatic <<a href="mailto:vijaykumar9597@gmail.com">vijaykumar9597@gmail.com</a>><br>
><br>
> ---<br>
>  rtemstoolkit/configuration.py | 4 ++--<br>
><br>
>  2 files changed, 3 insertions(+), 2 deletions(-)<br>
><br>
> diff --git a/rtemstoolkit/configuration.<wbr>py b/rtemstoolkit/configuration.<wbr>py<br>
> index 10d97e5..53ad79b 100644<br>
> --- a/rtemstoolkit/configuration.<wbr>py<br>
> +++ b/rtemstoolkit/configuration.<wbr>py<br>
> @@ -98,7 +98,7 @@ class configuration:<br>
>      def get_items(self, section, err = True, flatten = True):<br>
>          try:<br>
>              items = []<br>
> -            for name, key in self.config.items(section):<br>
> +            for name, key in self.config.items(section,raw=<wbr>True):<br>
<br>
</span>As stated in another email, this breaks Python 2.7 users where interpolation is<br>
being used. It is a requirement for this module to support interpolation and to<br>
support it to the level Python 3 has. This is why this module has functionality<br>
specific to Python 2.7.<br>
<br>
I suspect there is a problem else where in the code. I have not looked to see<br>
what the problem is you are having.<br>
<span class=""><br>
>                  if flatten:<br>
>                      items += [(name, key.replace(os.linesep, ' '))]<br>
>                  else:<br>
> @@ -117,7 +117,7 @@ class configuration:<br>
><br>
>      def get_item_names(self, section, err = True):<br>
>          try:<br>
> -            return [item[0] for item in self.config.items(section)]<br>
> +            return [item[0] for item in self.config.items(section,raw=<wbr>True)]<br>
<br>
</span>Sorry, this change is not going to be accepted in the rtems-tools repo.<br>
<br>
FYI all the Python code we have has spaces after the ',' and between '='. You<br>
will find it easier to get patches accept by following the coding standard that<br>
is already present.<br>
<br></blockquote><div>I will keep this in mind .</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Thanks<br>
<span class="HOEnZb"><font color="#888888">Chris<br>
</font></span></blockquote></div><br></div></div>