Mauro Carvalho Chehab [off-list ref] writes:

Em Thu, 10 Jul 2025 15:25:20 +0100
Donald Hunter [off-list ref] escreveu:

quoted

Donald Hunter [off-list ref] writes:

quoted

quoted

             # Parse message with RSTParser
-            for i, line in enumerate(msg.split('\n')):
-                result.append(line, document.current_source, i)
+            lineoffset = 0;
+            for line in msg.split('\n'):
+                match = self.re_lineno.match(line)
+                if match:
+                    lineoffset = int(match.group(1))
+                    continue
+
+                result.append(line, document.current_source, lineoffset)  

I expect this would need to be source=document.current_source, offset=lineoffset  

Ignore that. I see it's not kwargs. It's just the issue below.

quoted

quoted

             rst_parser = RSTParser()
             rst_parser.parse('\n'.join(result), document)  

But anyway this discards any line information by just concatenating the
lines together again.  

Looks to me like there's no Parser() API that works with ViewList() so
it would be necessary to directly use the docutils RSTStateMachine() for
this approach to work.

It sounds so.

The enclosed patch seems to address it:

	$ make cleandocs; make SPHINXDIRS="netlink/specs" htmldocs
	...
	Using alabaster theme
	source directory: netlink/specs
	Using Python kernel-doc
	/new_devel/v4l/docs/Documentation/netlink/specs/rt-neigh.yaml:13: ERROR: Unknown directive type "bogus".

	.. bogus:: [docutils]

Please notice that I added a hunk there to generate the error, just
to make easier to test - I'll drop it at the final version, and add
the proper reported-by/closes/... tags once you test it.

Regards,
Mauro

Awesome!

Tested-by: Donald Hunter <donald.hunter@gmail.com>

Patch comments below.

nodes is not used

 from docutils.parsers.rst import Parser as RSTParser

This import is no longer needed

I'm wondering if it makes much sense for this to inherit from Parser any
more?

 
         result = ViewList()
 
+        tab_width = 8
+
+        self.state_classes = states.state_classes
+        self.initial_state = 'Body'
+
+        self.statemachine = states.RSTStateMachine(
+              state_classes=self.state_classes,
+              initial_state=self.initial_state,
+              debug=document.reporter.debug_flag)

I don't think 'self.' is needed for any of these. They can be local to
the method. You could just inline states.state_classes and 'Body' into
the parameter list.

+
         try:
             # Parse message with RSTParser

Comment is out of date.

             lineoffset = 0;

Rogue semicolon

I think you could catch StateMachineError here.

             document.reporter.error("YAML parsing error: %s" % pformat(e))

Can you change this to an f"" string.