Re: [PATCH 1/1] buildtools: clear out regex syntax warnings

[Ariel Otilibili-Anieli]

On Friday, December 13, 2024 15:07 CET, "Robin Jarry" <rjarry at redhat.com> wrote:

> Ariel Otilibili, Dec 08, 2024 at 03:25:
> > * invalid escape sequences now generate SyntaxWarning
> > * therefore changed syntax to raw string noration.
> >
> > Link: https://docs.python.org/3/library/re.html#module-re
> > Signed-off-by: Ariel Otilibili <otilibil at eurecom.fr>
> 
> Hi Ariel, thanks for the fix!
> 
> Acked-by: Robin Jarry <rjarry at redhat.com>

Awesome, Robin!
> 
> If you have time, I there are other bits that would need the same fix:
> 
> buildtools/get-numa-count.py:14:        numa_nodes.sort(key=lambda l: int(re.findall('\d+', l)[0]))
> devtools/check-meson.py:54:        if re.match('^ *\t', code):
> doc/api/generate_doxygen.py:8:pattern = re.compile('^Preprocessing (.*)...$')
> 

Good, then; I'll push them in a subsequent series.
> Cheers.
>