Re: [-SPAM-] [PATCH v2 3/6] doc: check for absence of multiple terms in each entry of desc list
On Tuesday, 5 August 2025 22:30:23 CEST Ramsay Jones wrote:
Show 6 quoted lines
> On 05/08/2025 20:10, Jean-Noël Avila via GitGitGadget wrote:
> > From: =?UTF-8?q?Jean-No=C3=ABl=20Avila?= <jn.avila@free.fr>
> >
> > Due to portability issues, the script generate-configlist.sh was fixed to
> > not use carriage returns in the output. However, the result is that it no
> > longer correctly handles multiple terms in a single entry of the
Show 29 quoted lines
> > list.
> >
> > We now check that these entries do not exist in the documentation.
> >
> > Signed-off-by: Jean-Noël Avila <jn.avila@free.fr>
> > ---
> >
> > Documentation/Makefile | 10 +++++++++
> > Documentation/git-check-attr.adoc | 3 ++-
> > Documentation/git-check-ignore.adoc | 9 +++++---
> > Documentation/git-http-fetch.adoc | 4 +++-
> > Documentation/lint-documentation-style.perl | 24 +++++++++++++++++++++
> > Documentation/technical/api-path-walk.adoc | 5 ++++-
> > shared.mak | 1 +
> > 7 files changed, 50 insertions(+), 6 deletions(-)
> > create mode 100755 Documentation/lint-documentation-style.perl
> >
> > diff --git a/Documentation/Makefile b/Documentation/Makefile
> > index 76a9e1d02b26..ac8a21e3015c 100644
> > --- a/Documentation/Makefile
> > +++ b/Documentation/Makefile
> > @@ -508,6 +508,15 @@ $(LINT_DOCS_DELIMITED_SECTIONS):
> > .build/lint-docs/delimited-sections/%.ok: %.ado>
> > .PHONY: lint-docs-delimited-sections
> > lint-docs-delimited-sections: $(LINT_DOCS_DELIMITED_SECTIONS)
> >
> > +## Lint: Documentation style
> > +LINT_DOCS_DOC_STYLE = $(patsubst
> > %.adoc,.build/lint-docs/doc-style/%.ok,$(MAN_TXT)) +$
Show 29 quoted lines
> > lint-documentation-style.perl
> > +$(LINT_DOCS_DOC_STYLE): .build/lint-docs/doc-style/%.ok: %.adoc
> > + $(call mkdir_p_parent_template)
> > + $(QUIET_LINT_DOCSTYLE)$(PERL_PATH) lint-documentation-style.perl $<
>$@
> > +.PHONY: lint-docs-doc-style
> > +lint-docs-doc-style: $(LINT_DOCS_DOC_STYLE)
> > +
> >
> > lint-docs-manpages:
> > $(QUIET_GEN)./lint-manpages.sh
> >
> > @@ -537,6 +546,7 @@ lint-docs: lint-docs-gitlink
> >
> > lint-docs: lint-docs-man-end-blurb
> > lint-docs: lint-docs-man-section-order
> > lint-docs: lint-docs-delimited-sections
> >
> > +lint-docs: lint-docs-doc-style
> >
> > lint-docs: lint-docs-manpages
> > lint-docs: lint-docs-meson
> >
> > diff --git a/Documentation/git-check-attr.adoc
> > b/Documentation/git-check-attr.adoc
> > index 503b6446574d..15a37a38e3f7 100644
> > --- a/Documentation/git-check-attr.adoc
> > +++ b/Documentation/git-check-attr.adoc
> > @@ -19,7 +19,8 @@ For every pathname, this command will list if each
Show 14 quoted lines
> > 'unspecified',>
> > OPTIONS
> > -------
> >
> > --a, --all::
> > +-a::
> >
> > +--all::
> > List all attributes that are associated with the specified
> > paths. If this option is used, then 'unspecified' attributes
> > will not be included in the output.
> >
> > diff --git a/Documentation/git-check-ignore.adoc
> > b/Documentation/git-check-ignore.adoc index 3e3b4e344629..a6c6c1b6e5be
Show 54 quoted lines
> > --- a/Documentation/git-check-ignore.adoc
> > +++ b/Documentation/git-check-ignore.adoc
> > @@ -25,11 +25,13 @@ subject to exclude rules; but see `--no-index'.
> >
> > OPTIONS
> > -------
> >
> > --q, --quiet::
> > +-q::
> >
> > +--quiet::
> > Don't output anything, just set exit status. This is only
> > valid with a single pathname.
> >
> > --v, --verbose::
> > +-v::
> >
> > +--verbose::
> > Instead of printing the paths that are excluded, for each path
> > that matches an exclude pattern, print the exclude pattern
> > together with the path. (Matching an exclude pattern usually
> >
> > @@ -49,7 +51,8 @@ linkgit:gitignore[5].
> >
> > below). If `--stdin` is also given, input paths are separated
> > with a NUL character instead of a linefeed character.
> >
> > --n, --non-matching::
> > +-n::
> >
> > +--non-matching::
> > Show given paths which don't match any pattern. This only
> > makes sense when `--verbose` is enabled, otherwise it would
> > not be possible to distinguish between paths which match a
> >
> > diff --git a/Documentation/git-http-fetch.adoc
> > b/Documentation/git-http-fetch.adoc
> > index 4ec7c68d3b9e..dcb05890aefd 100644
> > --- a/Documentation/git-http-fetch.adoc
> > +++ b/Documentation/git-http-fetch.adoc
> >
> > @@ -25,8 +25,10 @@ commit-id::
> > Either the hash or the filename under [URL]/refs/ to
> > pull.
> >
> > --a, -c, -t::
> > +-a::-c::
>
> > +-t::
> s/-a::-c::/-a::
> -c::/ ?
>
> ATB,
> Ramsay Jones
Yep, the newline is inserted too late.
Thank you for the review. Will reroll in a couple days.
Jean-Noël Avila