[Qemu-devel] [PATCH] qapi: Fix cgen() for Python older than 2.7

[Qemu-devel] [PATCH] qapi: Fix cgen() for Python older than 2.7

[Markus Armbruster]

A feature new in Python 2.7 crept into commit 77e703b: re.subn()'s
fifth argument.  Avoid that, use re.compile().

Reported-by: Laurent Desnogues <laurent.desnogues@gmail.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
---
 scripts/qapi.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/scripts/qapi.py b/scripts/qapi.py
index 817d824..88fa073 100644
--- a/scripts/qapi.py
+++ b/scripts/qapi.py
@@ -944,7 +944,9 @@ def cgen(code, **kwds):
     raw = code % kwds
     if indent_level:
         indent = genindent(indent_level)
-        raw = re.subn("^.", indent + r'\g<0>', raw, 0, re.MULTILINE)
+        # re.subn() lacks flags support before Python 2.7, use re.compile()
+        raw = re.subn(re.compile("^.", re.MULTILINE),
+                      indent + r'\g<0>', raw)
         raw = raw[0]
     return re.sub(re.escape(eatspace) + ' *', '', raw)
 
-- 
2.4.3

Re: [Qemu-devel] [PATCH] qapi: Fix cgen() for Python older than 2.7

[Eric Blake]

[-- Attachment #1: Type: text/plain, Size: 1175 bytes --]

On 09/07/2015 09:45 AM, Markus Armbruster wrote:
> A feature new in Python 2.7 crept into commit 77e703b: re.subn()'s
> fifth argument.  Avoid that, use re.compile().
> 
> Reported-by: Laurent Desnogues <laurent.desnogues@gmail.com>
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> ---
>  scripts/qapi.py | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)

Reviewed-by: Eric Blake <eblake@redhat.com>

> 
> diff --git a/scripts/qapi.py b/scripts/qapi.py
> index 817d824..88fa073 100644
> --- a/scripts/qapi.py
> +++ b/scripts/qapi.py
> @@ -944,7 +944,9 @@ def cgen(code, **kwds):
>      raw = code % kwds
>      if indent_level:
>          indent = genindent(indent_level)
> -        raw = re.subn("^.", indent + r'\g<0>', raw, 0, re.MULTILINE)
> +        # re.subn() lacks flags support before Python 2.7, use re.compile()
> +        raw = re.subn(re.compile("^.", re.MULTILINE),
> +                      indent + r'\g<0>', raw)
>          raw = raw[0]
>      return re.sub(re.escape(eatspace) + ' *', '', raw)
>  
> 

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 604 bytes --]

Re: [Qemu-devel] [PATCH] qapi: Fix cgen() for Python older than 2.7

[Laurent Desnogues]

On Mon, Sep 7, 2015 at 5:45 PM, Markus Armbruster <armbru@redhat.com> wrote:
> A feature new in Python 2.7 crept into commit 77e703b: re.subn()'s
> fifth argument.  Avoid that, use re.compile().
>
> Reported-by: Laurent Desnogues <laurent.desnogues@gmail.com>
> Signed-off-by: Markus Armbruster <armbru@redhat.com>

Tested-by: Laurent Desnogues <laurent.desnogues@gmail.com>

Works fine here. Thanks,

Laurent

> ---
>  scripts/qapi.py | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/scripts/qapi.py b/scripts/qapi.py
> index 817d824..88fa073 100644
> --- a/scripts/qapi.py
> +++ b/scripts/qapi.py
> @@ -944,7 +944,9 @@ def cgen(code, **kwds):
>      raw = code % kwds
>      if indent_level:
>          indent = genindent(indent_level)
> -        raw = re.subn("^.", indent + r'\g<0>', raw, 0, re.MULTILINE)
> +        # re.subn() lacks flags support before Python 2.7, use re.compile()
> +        raw = re.subn(re.compile("^.", re.MULTILINE),
> +                      indent + r'\g<0>', raw)
>          raw = raw[0]
>      return re.sub(re.escape(eatspace) + ' *', '', raw)

Re: [Qemu-devel] [PATCH] qapi: Fix cgen() for Python older than 2.7

[Markus Armbruster]

Peter, are you willing to commit this directly as a build fix?  Or would
you prefer a pull request?

Re: [Qemu-devel] [PATCH] qapi: Fix cgen() for Python older than 2.7

[Peter Maydell]

On 7 September 2015 at 16:45, Markus Armbruster <armbru@redhat.com> wrote:
> A feature new in Python 2.7 crept into commit 77e703b: re.subn()'s
> fifth argument.  Avoid that, use re.compile().
>
> Reported-by: Laurent Desnogues <laurent.desnogues@gmail.com>
> Signed-off-by: Markus Armbruster <armbru@redhat.com>

Should we cc: stable on this?

thanks
-- PMM

Re: [Qemu-devel] [PATCH] qapi: Fix cgen() for Python older than 2.7

[Markus Armbruster]

Peter Maydell <peter.maydell@linaro.org> writes:

> On 7 September 2015 at 16:45, Markus Armbruster <armbru@redhat.com> wrote:
>> A feature new in Python 2.7 crept into commit 77e703b: re.subn()'s
>> fifth argument.  Avoid that, use re.compile().
>>
>> Reported-by: Laurent Desnogues <laurent.desnogues@gmail.com>
>> Signed-off-by: Markus Armbruster <armbru@redhat.com>
>
> Should we cc: stable on this?

Not needed: commit 77e703b is not in any release, and not meant for
stable.

Re: [Qemu-devel] [PATCH] qapi: Fix cgen() for Python older than 2.7

[Peter Maydell]

On 8 September 2015 at 07:38, Markus Armbruster <armbru@redhat.com> wrote:
> Peter, are you willing to commit this directly as a build fix?  Or would
> you prefer a pull request?

Applied to master, thanks (sorry I missed this email earlier).

-- PMM