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

Mauro Carvalho Chehab <[email protected]>
Newsgroups dev.linux.lists.lkmm,dev.linux.lists.linux-kernel-mentees,org.kernel.vger.linux-doc,org.kernel.vger.linux-kernel,org.kernel.vger.netdev
Message-ID <[email protected]>
Em Sat, 14 Jun 2025 15:15:25 +0100
Donald Hunter <[email protected]> escreveu:

> On Sat, 14 Jun 2025 at 09:56, Mauro Carvalho Chehab
> <[email protected]> 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 <[email protected]>
> > ---
> >  scripts/lib/netlink_yml_parser.py  | 101 ++++++++---------------------
> >  tools/net/ynl/pyynl/ynl_gen_rst.py |  28 +++++++-  



Thanks,
Mauro
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.