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

David Marchand david.marchand at redhat.com
Fri Dec 13 15:15:38 CET 2024


On Fri, Dec 13, 2024 at 3:11 PM Ariel Otilibili-Anieli
<otilibil at eurecom.fr> wrote:
>
> 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.

Those fixes are little one-liners worth backporting.

Please add a Fixes: tag identifying the commit introducing the issue,
then Cc: stable at dpdk.org.
If you need more details on what is expected, see
https://doc.dpdk.org/guides/contributing/patches.html#commit-messages-body
and https://doc.dpdk.org/guides/contributing/patches.html#patch-for-stable-releases


-- 
David Marchand



More information about the dev mailing list