On Wed, Sep 25, 2024 at 1:30 PM Miroslav Suchý <msu...@redhat.com> wrote:
>
> In Mock upstream we are right not discussing
>
> https://github.com/rpm-software-management/mock/pull/1462
>
> Here is the summary:
>
>  > Config files that uses DNF now contains `tsflags=nodocs` that tells RPM to 
> not
>  > install documentation files.
>  > This results to smaller buildroot. For fedora-rawhide, with only minimal 
> set of
>  > packages, this is reduction from 260MB to 246MB.
>
> We are not sure if this would/may affect some packages. If this can affect 
> you, or you have any comment about it, I will
> welcome a feedback.

We semi-frequenstly get bug reports for Rust packages because of
tsflags=nodocs, for example, when people build inside containers
instead of mock. It's not an uncommon pattern for Rust packages to
#include the contents of README.md as a documentation annotation at
build-time. Since we mark README.md as %doc (correctly, I think),
builds that use these libraries fail when %doc files are excluded from
installation.

As such, I would be against a change to set tsflags=nodocs in mock by
default. It would create a not insignificant amount of work for us,
and it's not clear to me how to solve that in a way that both 1) works
and 2) is compliant with packaging guidelines.

Fabio
-- 
_______________________________________________
devel mailing list -- devel@lists.fedoraproject.org
To unsubscribe send an email to devel-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue

Reply via email to