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

Reply via email to