On Tue, Jan 30, 2024 at 07:22:08AM -0700, Jonathan Corbet wrote:
Jani Nikula <jani.nikula@xxxxxxxxxxxxxxx> writes:
On Tue, 21 Nov 2023, Breno Leitao <leitao@xxxxxxxxxx> wrote:
This is a simple script that parses the Netlink YAML spec files
(Documentation/netlink/specs/), and generates RST files to be rendered
in the Network -> Netlink Specification documentation page.
First of all, my boilerplate complaint: All extra processing for Sphinx
should really be done using Sphinx extensions instead of adding Makefile
hacks. I don't think it's sustainable to keep adding this stuff. We
chose Sphinx because it is extensible, and to avoid the Rube Goldberg
machine that the previous documentation build system was.
So I feel like we've (me included) have kind of sent Breno around in
circles on this one. This *was* implemented as an extension once:
https://lore.kernel.org/netdev/20231103135622.250314-1-leitao@xxxxxxxxxx/
At that time it seemed too complex, and I thought that an external
script would lead to a simpler implementation overall. Perhaps I was
wrong.
I think you are correct. I personally _think_ that the external script
is better, mainly because it is self contained, thus, easier to
maintain.