[PATCH] setting raw in get_items

Chris Johns chrisj at rtems.org
Wed Apr 11 00:50:43 UTC 2018


On 11/04/2018 07:04, vijaykumar9597 at gmail.com wrote:
> From: thelunatic <vijaykumar9597 at gmail.com>
> 
> ---
>  rtemstoolkit/configuration.py | 4 ++--
>  
>  2 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/rtemstoolkit/configuration.py b/rtemstoolkit/configuration.py
> index 10d97e5..53ad79b 100644
> --- a/rtemstoolkit/configuration.py
> +++ b/rtemstoolkit/configuration.py
> @@ -98,7 +98,7 @@ class configuration:
>      def get_items(self, section, err = True, flatten = True):
>          try:
>              items = []
> -            for name, key in self.config.items(section):
> +            for name, key in self.config.items(section,raw=True):

As stated in another email, this breaks Python 2.7 users where interpolation is
being used. It is a requirement for this module to support interpolation and to
support it to the level Python 3 has. This is why this module has functionality
specific to Python 2.7.

I suspect there is a problem else where in the code. I have not looked to see
what the problem is you are having.

>                  if flatten:
>                      items += [(name, key.replace(os.linesep, ' '))]
>                  else:
> @@ -117,7 +117,7 @@ class configuration:
>  
>      def get_item_names(self, section, err = True):
>          try:
> -            return [item[0] for item in self.config.items(section)]
> +            return [item[0] for item in self.config.items(section,raw=True)]

Sorry, this change is not going to be accepted in the rtems-tools repo.

FYI all the Python code we have has spaces after the ',' and between '='. You
will find it easier to get patches accept by following the coding standard that
is already present.

Thanks
Chris



More information about the devel mailing list