Re: [docs] [PATCH 3/5] tools/build-docs-container: add CentOS Stream 9 support

Quentin Schulz
2025-12-05T15:30:02Z

Hi Antonin,

On 12/5/25 3:51 PM, Antonin Godard via lists.yoctoproject.org wrote:

[...]

> diff --git a/documentation/tools/build-docs-container b/documentation/tools/build-docs-container
> index 7d25b15b9..d753e2153 100755
> --- a/documentation/tools/build-docs-container
> +++ b/documentation/tools/build-docs-container
> @@ -41,6 +41,7 @@ $0 OCI_IMAGE [make arguments...]
> - fedora:42
> - leap:15.5
> - leap:15.6
> + - tgagor/centos:stream9

quay.io/centos/centos:stream9 is probably better?

We would need to remove the docker.io prefix from the ARG_FROM and make
it configurable but that should be easy to do.

[...]

> diff --git a/documentation/tools/containerfiles/entrypoint b/documentation/tools/containerfiles/entrypoint
> new file mode 100644
> index 000000000..97ee93c29
> --- /dev/null
> +++ b/documentation/tools/containerfiles/entrypoint
> @@ -0,0 +1,6 @@
> +#!/usr/bin/env sh
> +
> +tldir=/opt/docs-build-tex-tools/tl/bin/x86_64-linux
> +[ -d $tldir ] && export PATH="$PATH:$tldir"
> +
> +make -C documentation/ "$@"

Doesn't

ENV PATH="$PATH:$TEXTOOLDIR/tl/bin/x86_64-linux"

in Containerfile.stream work just fine instead of having to go through
an ENTRYPOINT?

This also makes the instructions x86_64-only. I have no clue if one can
run the current instructions on Aarch64 and make it work, but this for
sure won't help.

> diff --git a/documentation/tools/host_packages_scripts/centosstream_docs.sh b/documentation/tools/host_packages_scripts/centosstream_docs.sh
> new file mode 100644
> index 000000000..9202afc5b
> --- /dev/null
> +++ b/documentation/tools/host_packages_scripts/centosstream_docs.sh
> @@ -0,0 +1 @@
> +sudo dnf install -y git glibc-locale-source librsvg2-tools make python3-pip which
> diff --git a/documentation/tools/host_packages_scripts/centosstream_essential.sh b/documentation/tools/host_packages_scripts/centosstream_essential.sh
> new file mode 100644
> index 000000000..e38fa93c2
> --- /dev/null
> +++ b/documentation/tools/host_packages_scripts/centosstream_essential.sh
> @@ -0,0 +1,5 @@
> +sudo dnf install -y epel-release
> +sudo yum install -y dnf-plugins-core
> +sudo dnf config-manager --set-enabled crb
> +sudo dnf makecache
> +sudo dnf install -y bzip2 ccache chrpath cpio cpp diffstat diffutils gawk gcc gcc-c++ git glibc-devel glibc-langpack-en gzip libacl make patch perl perl-Data-Dumper perl-Text-ParseWords perl-Thread-Queue python3 python3-GitPython python3-jinja2 python3-pexpect python3-pip rpcgen socat tar texinfo unzip wget which xz zstd
> diff --git a/documentation/tools/host_packages_scripts/tlmgr_docs_pdf.sh b/documentation/tools/host_packages_scripts/tlmgr_docs_pdf.sh
> new file mode 100644
> index 000000000..f3b81da3e
> --- /dev/null
> +++ b/documentation/tools/host_packages_scripts/tlmgr_docs_pdf.sh
> @@ -0,0 +1,7 @@
> +textooldir=${TEXTOOLDIR:-"./docs-build-tex-tools"}
> +mkdir -p "$textooldir"/tl
> +wget https://mirror.ctan.org/systems/texlive/tlnet/install-tl-unx.tar.gz -O "$textooldir"/install-tl-unx.tar.gz
> +zcat < "$textooldir"/install-tl-unx.tar.gz | tar xf - -C "$textooldir"

Can't this simply be

tar xzf "$textooldir"/install-tl-unx.tar.gz -C "$textooldir"

?

Cheers,
Quentin