2019-04-25 16:55:07 +03:00
|
|
|
# SPDX-License-Identifier: GPL-2.0
|
|
|
|
# Copyright 2019 Jonathan Corbet <corbet@lwn.net>
|
|
|
|
#
|
|
|
|
# Apply kernel-specific tweaks after the initial document processing
|
|
|
|
# has been done.
|
|
|
|
#
|
|
|
|
from docutils import nodes
|
|
|
|
from sphinx import addnodes
|
docs: automarkup.py: ignore exceptions when seeking for xrefs
When using the automarkup extension with:
make pdfdocs
without passing an specific book, the code will raise an exception:
File "/devel/v4l/docs/Documentation/sphinx/automarkup.py", line 86, in auto_markup
node.parent.replace(node, markup_funcs(name, app, node))
File "/devel/v4l/docs/Documentation/sphinx/automarkup.py", line 59, in markup_funcs
'function', target, pxref, lit_text)
File "/devel/v4l/docs/sphinx_2.0/lib/python3.7/site-packages/sphinx/domains/c.py", line 308, in resolve_xref
contnode, target)
File "/devel/v4l/docs/sphinx_2.0/lib/python3.7/site-packages/sphinx/util/nodes.py", line 450, in make_refnode
'#' + targetid)
File "/devel/v4l/docs/sphinx_2.0/lib/python3.7/site-packages/sphinx/builders/latex/__init__.py", line 159, in get_relative_uri
return self.get_target_uri(to, typ)
File "/devel/v4l/docs/sphinx_2.0/lib/python3.7/site-packages/sphinx/builders/latex/__init__.py", line 152, in get_target_uri
raise NoUri
sphinx.environment.NoUri
This happens because not all references will belong to a single
PDF/LaTeX document.
Better to just ignore those than breaking Sphinx build.
Fixes: d74b0d31ddde ("Docs: An initial automarkup extension for sphinx")
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
[jc: Narrowed the "except" and tweaked the comment]
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2019-07-06 19:28:42 +03:00
|
|
|
from sphinx.environment import NoUri
|
2019-04-25 16:55:07 +03:00
|
|
|
import re
|
|
|
|
|
|
|
|
#
|
|
|
|
# Regex nastiness. Of course.
|
|
|
|
# Try to identify "function()" that's not already marked up some
|
|
|
|
# other way. Sphinx doesn't like a lot of stuff right after a
|
|
|
|
# :c:func: block (i.e. ":c:func:`mmap()`s" flakes out), so the last
|
|
|
|
# bit tries to restrict matches to things that won't create trouble.
|
|
|
|
#
|
|
|
|
RE_function = re.compile(r'([\w_][\w\d_]+\(\))')
|
|
|
|
|
|
|
|
#
|
|
|
|
# Many places in the docs refer to common system calls. It is
|
|
|
|
# pointless to try to cross-reference them and, as has been known
|
|
|
|
# to happen, somebody defining a function by these names can lead
|
|
|
|
# to the creation of incorrect and confusing cross references. So
|
|
|
|
# just don't even try with these names.
|
|
|
|
#
|
|
|
|
Skipfuncs = [ 'open', 'close', 'read', 'write', 'fcntl', 'mmap'
|
|
|
|
'select', 'poll', 'fork', 'execve', 'clone', 'ioctl']
|
|
|
|
|
|
|
|
#
|
|
|
|
# Find all occurrences of function() and try to replace them with
|
|
|
|
# appropriate cross references.
|
|
|
|
#
|
|
|
|
def markup_funcs(docname, app, node):
|
|
|
|
cdom = app.env.domains['c']
|
|
|
|
t = node.astext()
|
|
|
|
done = 0
|
|
|
|
repl = [ ]
|
|
|
|
for m in RE_function.finditer(t):
|
|
|
|
#
|
|
|
|
# Include any text prior to function() as a normal text node.
|
|
|
|
#
|
|
|
|
if m.start() > done:
|
|
|
|
repl.append(nodes.Text(t[done:m.start()]))
|
|
|
|
#
|
|
|
|
# Go through the dance of getting an xref out of the C domain
|
|
|
|
#
|
|
|
|
target = m.group(1)[:-2]
|
|
|
|
target_text = nodes.Text(target + '()')
|
|
|
|
xref = None
|
|
|
|
if target not in Skipfuncs:
|
|
|
|
lit_text = nodes.literal(classes=['xref', 'c', 'c-func'])
|
|
|
|
lit_text += target_text
|
|
|
|
pxref = addnodes.pending_xref('', refdomain = 'c',
|
|
|
|
reftype = 'function',
|
|
|
|
reftarget = target, modname = None,
|
|
|
|
classname = None)
|
docs: automarkup.py: ignore exceptions when seeking for xrefs
When using the automarkup extension with:
make pdfdocs
without passing an specific book, the code will raise an exception:
File "/devel/v4l/docs/Documentation/sphinx/automarkup.py", line 86, in auto_markup
node.parent.replace(node, markup_funcs(name, app, node))
File "/devel/v4l/docs/Documentation/sphinx/automarkup.py", line 59, in markup_funcs
'function', target, pxref, lit_text)
File "/devel/v4l/docs/sphinx_2.0/lib/python3.7/site-packages/sphinx/domains/c.py", line 308, in resolve_xref
contnode, target)
File "/devel/v4l/docs/sphinx_2.0/lib/python3.7/site-packages/sphinx/util/nodes.py", line 450, in make_refnode
'#' + targetid)
File "/devel/v4l/docs/sphinx_2.0/lib/python3.7/site-packages/sphinx/builders/latex/__init__.py", line 159, in get_relative_uri
return self.get_target_uri(to, typ)
File "/devel/v4l/docs/sphinx_2.0/lib/python3.7/site-packages/sphinx/builders/latex/__init__.py", line 152, in get_target_uri
raise NoUri
sphinx.environment.NoUri
This happens because not all references will belong to a single
PDF/LaTeX document.
Better to just ignore those than breaking Sphinx build.
Fixes: d74b0d31ddde ("Docs: An initial automarkup extension for sphinx")
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
[jc: Narrowed the "except" and tweaked the comment]
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
2019-07-06 19:28:42 +03:00
|
|
|
#
|
|
|
|
# XXX The Latex builder will throw NoUri exceptions here,
|
|
|
|
# work around that by ignoring them.
|
|
|
|
#
|
|
|
|
try:
|
|
|
|
xref = cdom.resolve_xref(app.env, docname, app.builder,
|
|
|
|
'function', target, pxref, lit_text)
|
|
|
|
except NoUri:
|
|
|
|
xref = None
|
2019-04-25 16:55:07 +03:00
|
|
|
#
|
|
|
|
# Toss the xref into the list if we got it; otherwise just put
|
|
|
|
# the function text.
|
|
|
|
#
|
|
|
|
if xref:
|
|
|
|
repl.append(xref)
|
|
|
|
else:
|
|
|
|
repl.append(target_text)
|
|
|
|
done = m.end()
|
|
|
|
if done < len(t):
|
|
|
|
repl.append(nodes.Text(t[done:]))
|
|
|
|
return repl
|
|
|
|
|
|
|
|
def auto_markup(app, doctree, name):
|
|
|
|
#
|
|
|
|
# This loop could eventually be improved on. Someday maybe we
|
|
|
|
# want a proper tree traversal with a lot of awareness of which
|
|
|
|
# kinds of nodes to prune. But this works well for now.
|
|
|
|
#
|
|
|
|
# The nodes.literal test catches ``literal text``, its purpose is to
|
|
|
|
# avoid adding cross-references to functions that have been explicitly
|
|
|
|
# marked with cc:func:.
|
|
|
|
#
|
|
|
|
for para in doctree.traverse(nodes.paragraph):
|
|
|
|
for node in para.traverse(nodes.Text):
|
|
|
|
if not isinstance(node.parent, nodes.literal):
|
|
|
|
node.parent.replace(node, markup_funcs(name, app, node))
|
|
|
|
|
|
|
|
def setup(app):
|
|
|
|
app.connect('doctree-resolved', auto_markup)
|
|
|
|
return {
|
|
|
|
'parallel_read_safe': True,
|
|
|
|
'parallel_write_safe': True,
|
|
|
|
}
|