Re: [PATCH v3 15/15] docs: conf.py: Check Sphinx and docutils version
From: Mauro Carvalho Chehab
Date: Sun Jun 22 2025 - 17:51:04 EST
Em Sun, 22 Jun 2025 14:58:04 -0600
Jonathan Corbet <corbet@xxxxxxx> escreveu:
> Akira Yokosawa <akiyks@xxxxxxxxx> writes:
>
> > On Sun, 22 Jun 2025 08:02:44 +0200, Mauro Carvalho Chehab wrote:
> >> As reported by Akira, there were incompatibility issues with
> >> Sphinx and docutils with docutils 0.19. There's already
> >> a fix for it, but, as there are incompatibility issues with
> >> different versions, better to add a check to verify if the
> >> combination is supported/tested.
> >>
> >
> > I've been skeptical of adding such checks in conf.py.
>
> I have to kind of agree with this concern. We have managed without this
> complexity so far. It looks like we could always be behind on
> maintaining it going forward. Do we *really* need this one?
IMO having a check is interesting, as the dependency between
Sphinx and docutils is high. Yet, with the testing script, this may
not be needed, provided that we run it to check if changes at Sphinx
extensions won't cause regressions. Still, the dependency check
at test_doc_build.py is not complete.
Anyway, if you prefer, don't pick this one. We can revisit it later
when needed.
Thanks,
Mauro