summaryrefslogtreecommitdiffstats
path: root/tools/check-version-history.py
diff options
context:
space:
mode:
Diffstat (limited to '')
-rw-r--r--tools/check-version-history.py135
1 files changed, 135 insertions, 0 deletions
diff --git a/tools/check-version-history.py b/tools/check-version-history.py
new file mode 100644
index 0000000..c4d4b3e
--- /dev/null
+++ b/tools/check-version-history.py
@@ -0,0 +1,135 @@
+#!/usr/bin/env python3
+# SPDX-License-Identifier: LGPL-2.1-or-later
+
+import os
+import sys
+import lxml.etree as tree
+
+_parser = tree.XMLParser(resolve_entities=False)
+tree.set_default_parser(_parser)
+
+
+def find_undocumented_functions(pages, ignorelist):
+ undocumented = []
+ for page in pages:
+ filename = os.path.basename(page)
+ pagetree = tree.parse(page)
+
+ assert pagetree.getroot().tag == "refentry"
+
+ hist_section = pagetree.find("refsect1[title='History']")
+ for func in pagetree.findall("//funcprototype/funcdef/function"):
+ path = f"/refsynopsisdiv/funcsynopsis/funcprototype/funcdef/function[.='{func.text}']"
+ assert pagetree.findall(path) == [func]
+
+ if (
+ hist_section is None
+ or hist_section.find(f"para/function[.='{func.text}()']") is None
+ ):
+ if func.text not in ignorelist:
+ undocumented.append((filename, func.text))
+ return undocumented
+
+
+def construct_path(element):
+ tag = element.tag
+
+ if tag == "refentry":
+ return ""
+
+ predicate = ""
+ if tag == "varlistentry":
+ text = "".join(element.find("term").itertext())
+ predicate = f'[term="{text}"]'
+ elif tag.startswith("refsect"):
+ text = "".join(element.find("title").itertext())
+ predicate = f'[title="{text}"]'
+ elif tag == "variablelist":
+ varlists = element.getparent().findall(tag)
+ if len(varlists) > 1:
+ predicate = f"[{varlists.index(element)+1}]"
+
+ return construct_path(element.getparent()) + "/" + tag + predicate
+
+
+def find_undocumented_commands(pages, ignorelist):
+ undocumented = []
+ for page in pages:
+ filename = os.path.basename(page)
+
+ pagetree = tree.parse(page)
+ if pagetree.getroot().tag != "refentry":
+ continue
+
+ for varlistentry in pagetree.findall("*//variablelist/varlistentry"):
+ path = construct_path(varlistentry)
+
+ assert pagetree.findall(path) == [varlistentry]
+
+ listitem = varlistentry.find("listitem")
+ parent = listitem if listitem is not None else varlistentry
+
+ rev = parent.getchildren()[-1]
+ if rev.get("href") != "version-info.xml":
+ if (filename, path) not in ignorelist:
+ undocumented.append((filename, path))
+ return undocumented
+
+
+def process_pages(pages):
+ command_pages = []
+ function_pages = []
+
+ for page in pages:
+ filename = os.path.basename(page)
+ if filename.startswith("org.freedesktop."): # dbus
+ continue
+
+ if (
+ filename.startswith("sd_")
+ or filename.startswith("sd-")
+ or filename.startswith("udev_")
+ ):
+ function_pages.append(page)
+ continue
+
+ command_pages.append(page)
+
+ undocumented_commands = find_undocumented_commands(
+ command_pages, command_ignorelist
+ )
+ undocumented_functions = find_undocumented_functions(
+ function_pages, function_ignorelist
+ )
+
+ return undocumented_commands, undocumented_functions
+
+
+if __name__ == "__main__":
+ with open(os.path.join(os.path.dirname(__file__), "command_ignorelist")) as f:
+ command_ignorelist = []
+ for l in f.read().splitlines():
+ if l.startswith("#"):
+ continue
+ fname, path = l.split(" ", 1)
+ path = path.replace("\\n", "\n")
+ command_ignorelist.append((fname, path))
+ with open(os.path.join(os.path.dirname(__file__), "function_ignorelist")) as f:
+ function_ignorelist = f.read().splitlines()
+
+ undocumented_commands, undocumented_functions = process_pages(sys.argv[1:])
+
+ if undocumented_commands or undocumented_functions:
+ for filename, func in undocumented_functions:
+ print(
+ f"Function {func}() in {filename} isn't documented in the History section."
+ )
+ for filename, path in undocumented_commands:
+ print(filename, path, "is undocumented")
+ if undocumented_commands:
+ print(
+ "Hint: if you reorganized this part of the documentation, "
+ "please update tools/commands_ignorelist."
+ )
+
+ sys.exit(1)