On 04/01/2022 08.46, [email protected] wrote:
From: Marc-André Lureau <[email protected]>

SphinxDirective was added with sphinx 1.8 (2018-09-13).

Reported-by: Thomas Huth <[email protected]>
Signed-off-by: Marc-André Lureau <[email protected]>
---
  docs/sphinx/fakedbusdoc.py | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/docs/sphinx/fakedbusdoc.py b/docs/sphinx/fakedbusdoc.py
index a680b257547f..d2c507904654 100644
--- a/docs/sphinx/fakedbusdoc.py
+++ b/docs/sphinx/fakedbusdoc.py
@@ -7,12 +7,12 @@
  # Author: Marc-André Lureau <[email protected]>
  """dbus-doc is a Sphinx extension that provides documentation from D-Bus 
XML."""
+from docutils.parsers.rst import Directive
  from sphinx.application import Sphinx
-from sphinx.util.docutils import SphinxDirective
  from typing import Any, Dict
-class FakeDBusDocDirective(SphinxDirective):
+class FakeDBusDocDirective(Directive):
      has_content = True
      required_arguments = 1

Thanks, that fixes my issue, indeed!

Tested-by: Thomas Huth <[email protected]>

I wonder whether we could turn such sphinx warnings during the configure phase into a hard failure so that such bugs would fail in the CI instead of getting merge without notice...?

 Thomas


Reply via email to