On Tue, Apr 09, 2024 at 09:19:02AM +0200, Adrian Moreno wrote: > The current __check_doc_is_listed() verifies that the new .rst file is > listed in Documentation/automake.mk with the full path (i.e: > "{directory}/{filename}"). > > While this holds true for generic documentation files, which are added > to DOC_SOURCE with the full path, it's not true for rST manpages which > are added only by filename to RST_MANPAGES target (see > Documentation/automake.mk). > > This makes the current implementation of the check to incorrectly raise > a warning as follows even though the patch does add the file to > RST_MANPAGES: > > """ > WARNING: New doc ovs-flowviz.8.rst not listed in > Documentation/automake.mk > """ > > Fix it by making the {dir}/ part of the docre regexp optional. > > Signed-off-by: Adrian Moreno <amore...@redhat.com>
Acked-by: Simon Horman <ho...@ovn.org> _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev