Re: [ovs-dev] [PATCH v2 1/5] docs: Wrap more table entries in text blocks

On Mon, Aug 07, 2023 at 03:00:41PM +0200, Simon Horman wrote: > On Fri, Aug 04, 2023 at 06:24:41PM +0100, Colin Watson wrote: > > This fixes a number of "table wider than line length minus indentation" > > warnings from tbl. The table cells are too narrow for centered

[ovs-dev] [PATCH v2 5/5] docs: Run tbl preprocessor in manpage-check rule

If we omit this, groff 1.23.0 warns: tbl preprocessor failed, or it or soelim was not run; table(s) likely not rendered (TE macro called with TW register undefined) Reported-by: Lucas Nussbaum Reported-at: https://bugs.debian.org/1042358 Signed-off-by: Colin Watson --- Makefile.am | 2

[ovs-dev] [PATCH v2 4/5] docs: Fix rendering of VLAN Comparison Chart

, and match the style used by build-aux/extract-ofp-fields. Reported-by: Lucas Nussbaum Reported-at: https://bugs.debian.org/1042358 Signed-off-by: Colin Watson --- lib/meta-flow.xml | 25 + 1 file changed, 13 insertions(+), 12 deletions(-) diff --git a/lib/meta-flow.xml b

[ovs-dev] [PATCH v2 2/5] docs: Shorten overly-wide table heading

eported-at: https://bugs.debian.org/1042358 Signed-off-by: Colin Watson --- build-aux/extract-ofp-fields | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/build-aux/extract-ofp-fields b/build-aux/extract-ofp-fields index 2f566d2b9..808c6527d 100755 --- a/build-aux/extract-ofp-fields

[ovs-dev] [PATCH v2 3/5] docs: Tweak width of name column in field property tables

: Colin Watson --- build-aux/extract-ofp-fields | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/build-aux/extract-ofp-fields b/build-aux/extract-ofp-fields index 808c6527d..449885981 100755 --- a/build-aux/extract-ofp-fields +++ b/build-aux/extract-ofp-fields @@ -322,7

[ovs-dev] [PATCH v2 1/5] docs: Wrap more table entries in text blocks

This fixes a number of "table wider than line length minus indentation" warnings from tbl. The table cells are too narrow for centered text to look good, so left-align the contents of the text blocks. Reported-by: Lucas Nussbaum Reported-at: https://bugs.debian.org/1042358 Signed-off

[ovs-dev] [PATCH v2 0/5] docs: Fix manpage-check warnings with groff 1.23.0

https://bugs.debian.org/1042358 reported a manpage-check failure with groff 1.23.0 in Debian testing/unstable. Fixing the immediate mistake here exposed a few other issues in how the tables in ovs-fields(7) are rendered. Colin Watson (5): docs: Wrap more table entries in text blocks docs

[ovs-dev] [PATCH 0/3] docs: Fix manpage-check warnings with groff 1.23.0

https://bugs.debian.org/1042358 reported a manpage-check failure with groff 1.23.0 in Debian testing/unstable. Fixing the immediate mistake here exposed a few other issues in how the tables in ovs-fields(7) are rendered. Colin Watson (3): docs: Wrap more table entries in text blocks docs

[ovs-dev] [PATCH 3/3] docs: Run tbl preprocessor in manpage-check rule

If we omit this, groff 1.23.0 warns: tbl preprocessor failed, or it or soelim was not run; table(s) likely not rendered (TE macro called with TW register undefined) Reported-by: Lucas Nussbaum Reported-at: https://bugs.debian.org/1042358 Signed-off-by: Colin Watson --- Makefile.am | 2

[ovs-dev] [PATCH 1/3] docs: Wrap more table entries in text blocks

This fixes a number of "table wider than line length minus indentation" warnings from tbl. Reported-by: Lucas Nussbaum Reported-at: https://bugs.debian.org/1042358 Signed-off-by: Colin Watson --- build-aux/extract-ofp-fields | 14 +- 1 file changed, 9 insertions(+), 5

[ovs-dev] [PATCH 2/3] docs: Fix rendering of VLAN Comparison Chart

, and match the style used by build-aux/extract-ofp-fields. Reported-by: Lucas Nussbaum Reported-at: https://bugs.debian.org/1042358 Signed-off-by: Colin Watson --- lib/meta-flow.xml | 25 + 1 file changed, 13 insertions(+), 12 deletions(-) diff --git a/lib/meta-flow.xml b