Re: [docs] [yocto-docs PATCH 4/5] system-requirements.rst: add dependencies for pdf builds

Quentin Schulz
2024-11-29T14:29:05Z

Hi Antonin,

On 11/29/24 3:06 PM, Antonin Godard via lists.yoctoproject.org wrote:
> Hi Quentin,
>
> On Mon Nov 25, 2024 at 7:11 PM CET, Quentin Schulz wrote:
>> Hi Antonin,
>>
>> On 11/20/24 4:10 PM, Antonin Godard via lists.yoctoproject.org wrote:
>>> With the recent changes from commit
>>> bd6265ca323fac547a197bb516dc4a9ef3897508 ("doc: Makefile: add support
>>> for xelatex"), the list of dependencies needs to be updated.
>>>
>>> The initial list of dependencies was made for Ubuntu/Debian, and the
>>> packages were translated for the other distros using pkgs.org.
>>>
>>> Add them separately from the rest of the dependencies as they pull a
>>> _lot_ of additional dependencies (for latex).
>>>
>>> Notes:
>>> - Fedora/OpenSUSE do not have the "lang-all" texlive collection, so
>>> install all available languages.
>>> - AlmaLinux does not seem to provide texlive-collection-fontsextra,
>>> texlive-collection-lang*, and texlive-collection-latexextra, so add a
>>> warning about that.
>>>
>>> Signed-off-by: Antonin Godard <antonin.godard@bootlin.com>
>>> ---
>>> documentation/poky.yaml.in | 79 +++++++++++++++++++++++-
>>> documentation/ref-manual/system-requirements.rst | 27 ++++++++
>>> 2 files changed, 105 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/documentation/poky.yaml.in b/documentation/poky.yaml.in
>>> index d0c8973db27340838f78f560c01d44fb67771b25..d95b73c74ad7979609c6e7f4e1be5343968da743 100644
>>> --- a/documentation/poky.yaml.in
>>> +++ b/documentation/poky.yaml.in
>>> @@ -67,8 +67,17 @@ UBUNTU_DEBIAN_HOST_PACKAGES_DOC: >-
>>> python3-saneyaml
>>> python3-sphinx-rtd-theme
>>> sphinx
>>> - texlive-latex-extra
>>>
>>> +UBUNTU_DEBIAN_HOST_PACKAGES_DOC_PDF: >-
>>> + fonts-freefont-otf
>>> + latexmk
>>> + tex-gyre
>>> + texlive-fonts-extra
>>> + texlive-fonts-recommended
>>> + texlive-lang-all
>>> + texlive-latex-extra
>>> + texlive-latex-recommended
>>> + texlive-xetex
>>>
>>> FEDORA_HOST_PACKAGES_ESSENTIAL: >-
>>> bzip2
>>> @@ -131,6 +140,35 @@ FEDORA_HOST_PACKAGES_DOC: >-
>>> which
>>> texlive-fncychap
>>>
>>> +FEDORA_HOST_PACKAGES_DOC_PDF: >-
>>> + latexmk
>>> + texlive-collection-fontsextra
>>> + texlive-collection-fontsrecommended
>>> + texlive-collection-langarabic
>>> + texlive-collection-langchinese
>>> + texlive-collection-langcjk
>>> + texlive-collection-langcyrillic
>>> + texlive-collection-langczechslovak
>>> + texlive-collection-langenglish
>>> + texlive-collection-langeuropean
>>> + texlive-collection-langfrench
>>> + texlive-collection-langgerman
>>> + texlive-collection-langgreek
>>> + texlive-collection-langitalian
>>> + texlive-collection-langjapanese
>>> + texlive-collection-langkorean
>>> + texlive-collection-langother
>>> + texlive-collection-langpolish
>>> + texlive-collection-langportuguese
>>> + texlive-collection-langspanish
>>
>> I'm wondering if we really need all languages or if we were too lazy to
>> figure out which ones we actually need.
>
> Actually, I'd go with all of them... the initial reason for including those is
> because contributors in the release notes may have names with international
> characters. I'd include them all "to be safe" and avoid wondering about LaTeX
> errors later.
>

Ah true, that was the reason, I had forgotten already with my name with
boring letters :)

>> Also, you should be able to avoid listing them all by using
>> 'texlive-collection-lang*', dnf should be able to handle that just fine.
>
> Thanks for the tip. I should be able to test that with my container scripts soon :)
>
>>> + texlive-collection-latex
>>> + texlive-collection-latexextra
>>> + texlive-collection-latexrecommended
>>
>> latex-extra already requires latex-recommended which already requires
>> latex so only the former is necessary.
>
> +1
>

Actually I think this is potentially a bad recommendation. It depends
exactly what we need. But if we need stuff from latex-recommended or
latex, we should explicitly list either or both, because it would
prevent issues when the distro decides latexextra doesn't require
latexrecommended which doesn't require latex anymore. This has happened
to me (in a different context) when upgrading from Debian Bullseye to
Bookworm.

Cheers,
Quentin