Re: [PATCH 03/13] docs: move scripts/check-variable-fonts.sh to tools/doc

[Mauro Carvalho Chehab <mchehab+huawei@kernel.org>]

Em Wed, 13 Aug 2025 15:32:02 -0600
Jonathan Corbet <corbet@lwn.net> escreveu:

> Move this script to live with the rest of the documentation tools.

This one will also be obsoleted with the new sphinx-build-wrapper tool.

Please don't touch on it yet.

> 
> Signed-off-by: Jonathan Corbet <corbet@lwn.net>
> ---
>  Documentation/Makefile                         | 2 +-
>  MAINTAINERS                                    | 1 -
>  {scripts => tools/doc}/check-variable-fonts.sh | 2 +-
>  3 files changed, 2 insertions(+), 3 deletions(-)
>  rename {scripts => tools/doc}/check-variable-fonts.sh (98%)
> 
> diff --git a/Documentation/Makefile b/Documentation/Makefile
> index 820f07e0afe6..70095465dce1 100644
> --- a/Documentation/Makefile
> +++ b/Documentation/Makefile
> @@ -146,7 +146,7 @@ pdfdocs: DENY_VF = XDG_CONFIG_HOME=$(FONTS_CONF_DENY_VF)
>  pdfdocs: latexdocs
>  	@$(srctree)/scripts/sphinx-pre-install --version-check
>  	$(foreach var,$(SPHINXDIRS), \
> -	   $(MAKE) PDFLATEX="$(PDFLATEX)" LATEXOPTS="$(LATEXOPTS)" $(DENY_VF) -C $(BUILDDIR)/$(var)/latex || sh $(srctree)/scripts/check-variable-fonts.sh || exit; \
> +	   $(MAKE) PDFLATEX="$(PDFLATEX)" LATEXOPTS="$(LATEXOPTS)" $(DENY_VF) -C $(BUILDDIR)/$(var)/latex || sh $(srctree)/tools/doc/check-variable-fonts.sh || exit; \
>  	   mkdir -p $(BUILDDIR)/$(var)/pdf; \
>  	   mv $(subst .tex,.pdf,$(wildcard $(BUILDDIR)/$(var)/latex/*.tex)) $(BUILDDIR)/$(var)/pdf/; \
>  	)
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a3a396fc1c3f..ca4c7992369d 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -7302,7 +7302,6 @@ P:	Documentation/doc-guide/maintainer-profile.rst
>  T:	git git://git.lwn.net/linux.git docs-next
>  F:	Documentation/
>  F:	tools/doc/
> -F:	scripts/check-variable-fonts.sh
>  F:	scripts/documentation-file-ref-check
>  F:	scripts/get_abi.py
>  F:	scripts/kernel-doc*
> diff --git a/scripts/check-variable-fonts.sh b/tools/doc/check-variable-fonts.sh
> similarity index 98%
> rename from scripts/check-variable-fonts.sh
> rename to tools/doc/check-variable-fonts.sh
> index ce63f0acea5f..9660df53d716 100755
> --- a/scripts/check-variable-fonts.sh
> +++ b/tools/doc/check-variable-fonts.sh
> @@ -106,7 +106,7 @@ if [ "x$notocjkvffonts" != "x" ] ; then
>  	echo 'Or, CJK pages can be skipped by uninstalling texlive-xecjk.'
>  	echo
>  	echo 'For more info on denylisting, other options, and variable font, see header'
> -	echo 'comments of scripts/check-variable-fonts.sh.'
> +	echo 'comments of tools/doc/check-variable-fonts.sh.'
>  	echo '============================================================================='
>  fi
>  



Thanks,
Mauro