Skip to content
This repository was archived by the owner on Apr 9, 2025. It is now read-only.

Commit d19ae73

Browse files
committed
target explicit intersphinx module name depending on sphinx version
Signed-off-by: Christian López Barrón <[email protected]>
1 parent 102703e commit d19ae73

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

hoverxref/extension.py

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
from sphinx.roles import XRefRole
99
from sphinx.util.fileutil import copy_asset
1010
from sphinx.util import logging
11+
from packaging.version import parse, Version
1112

1213
from . import __version__
1314
from .domains import (
@@ -172,9 +173,14 @@ def setup_intersphinx(app, config):
172173
# does not have hoverxref intersphinx enabled
173174
return
174175

176+
# Sphinx-7.4.0 turned the intersphinx module into a package, so target the module according to the version.
177+
if parse(sphinx.__version__) >= Version("7.4"):
178+
intersphinx_module_listener_name = 'sphinx.ext.intersphinx._resolve'
179+
else:
180+
intersphinx_module_listener_name = 'sphinx.ext.intersphinx'
175181
for listener in app.events.listeners.get('missing-reference'):
176182
module_name = inspect.getmodule(listener.handler).__name__
177-
if module_name.startswith('sphinx.ext.intersphinx'):
183+
if module_name == intersphinx_module_listener_name:
178184
app.disconnect(listener.id)
179185

180186

0 commit comments

Comments
 (0)