On Thu, Nov 28, 2024 at 12:38 PM Zbigniew Jędrzejewski-Szmek <zbyszek@xxxxxxxxx> wrote: > > We'll have the vendoring patches applied in hundreds of packages, No, it's definitely not "hundreds". A sourcegraph query shows 14 packages affected in the rawhide branch: https://sourcegraph.com/search?q=context:global+r:src.fedoraproject.org+file:.*%5C.spec+bundled_rust_deps&patternType=standard&sm=0 There's a few more with a dedicated "eln" branch, but there the bcond doing the wrong thing "if rhel" is a noop. > since it's needed for RHEL. rust2rpm has native support for vendoring. > So I think the only maintainable option is to use that. > (I don't know how it works exactly… From a quick look, it does > the vendoring unconditionally when enabled. Maybe it needs to be > ehanced to make the vendoring optional à la the patches that Yaakov > was applying.) I have, multiple times, offered to ELN people to implement support for ELN-style conditional vendoring in rust2rpm itself. There was little to no interest in it. Though with the snippets that have been copied around the place being buggy, that might finally be an argument in favor of implementing this in rust2rpm itself. Fabio -- _______________________________________________ devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx 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@xxxxxxxxxxxxxxxxxxxxxxx Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
