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

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

Hi Quentin,

On Fri Dec 6, 2024 at 3:00 PM CET, Quentin Schulz wrote:
> Hi Antonin,
>
> On 12/4/24 1:23 PM, Antonin Godard 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                       | 47 +++++++++++++++++++++++-
>>   documentation/ref-manual/system-requirements.rst | 27 ++++++++++++++
>>   2 files changed, 73 insertions(+), 1 deletion(-)
>> 
>> diff --git a/documentation/poky.yaml.in b/documentation/poky.yaml.in
>> index 94849acf9942c7ebddb2ac7f8aaf5eb9d87e0c5b..25385068679a51e0f2e9e516b02b62e19e126291 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
>> @@ -129,6 +138,19 @@ FEDORA_HOST_PACKAGES_DOC: >-
>>     texlive-latex-extra
>>     which
>>   
>> +FEDORA_HOST_PACKAGES_DOC_PDF: >-
>> +  latexmk
>> +  texlive-collection-fontsextra
>> +  texlive-collection-fontsrecommended
>> +  texlive-collection-lang*
>
> Please use single quotes to surround texlive-collection-lang* otherwise 
> it may be interpreted by the shell, c.f. for me:
>
> ```sh
> $ sudo dnf install texlive-collection-lang*
> zsh: no matches found: libgpiod*
> ```
>
> while `sudo dnf install 'texlive-collection-lang*'` works just fine.
>
> Ditto for the other occurrences later in this diff.

So I had issues with this in the container scripts, which is why I left it at
that. I initially single quoted this but it didn't work. Are you sure you can
mix non-quotes and quoted strings here? I'll give a try, might be specific to
the dockerfile.

Thanks,
Antonin

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