Re: [common] codegen: Fix compatibility with Python 2.6

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> This is needed for RHEL 6.8.
> 
> Signed-off-by: Francois Gouget <fgouget@xxxxxxxxxxxxxxx>
> ---
> 
> This is not the only patch needed to get the latest Spice server to
> compile on RHEL 6.8. But this dependency on Python 2.7 seems pretty
> needless so it may be worth applying.
> 

Yes, I would add in the comment that "In Python 2.6 re.sub has no flags parameter."

>  python_modules/marshal.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/python_modules/marshal.py b/python_modules/marshal.py
> index 0483d3a..5445109 100644
> --- a/python_modules/marshal.py
> +++ b/python_modules/marshal.py
> @@ -11,7 +11,7 @@ def write_includes(writer):
>          src = writer.header.options["dest_file"]
>      else:
>          src = "generated_headers.h"
> -    src = re.sub(r'[^a-z0-9]+', '_', src, flags=re.IGNORECASE)
> +    src = re.sub(r'(?i)[^a-z0-9]+', '_', src, 0)

src = re.sub(r'(?i)[^a-z0-9]+', '_', src)

works too and it's more similar (the count is the default).

>      src = src.upper()
>      if src.endswith("_H"):
>          src = "_H_"+src[:-2]

Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]