Re: [PATCH v4 07/14] tools: ynl_gen_rst.py: move index.rst generator to the script

Mauro Carvalho Chehab
2025-06-14T15:35:34Z

Em Sat, 14 Jun 2025 15:15:25 +0100
Donald Hunter <donald.hunter@gmail.com> escreveu:

> On Sat, 14 Jun 2025 at 09:56, Mauro Carvalho Chehab
> <mchehab+huawei@kernel.org> wrote:
> >
> > The index.rst generator doesn't really belong to the parsing
> > function. Move it to the command line tool, as it won't be
> > used elsewhere.
> >
> > While here, make it more generic, allowing it to handle either
> > .yaml or .rst as input files.
>
> I think this patch can be dropped from the series, if instead you
> remove the index generation code before refactoring into a library.

Works for me. I'll wait for you and Jon's comments with regards to the
location before sending a new version.

Regards,
Mauro

>
> > Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
> > ---
> > scripts/lib/netlink_yml_parser.py | 101 ++++++++---------------------
> > tools/net/ynl/pyynl/ynl_gen_rst.py | 28 +++++++-

Thanks,
Mauro