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

["Antonin Godard" <antonin.godard@bootlin.com>]

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.

> 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


Thank you,
Antonin

-- 
Antonin Godard, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com