Re: deduplicating noarch subpackages

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2/12/20 12:45 AM, Nicolas Mailhot wrote:
> Le 2020-02-12 01:21, Josh Stone a écrit :
> 
>> The problem is that those cross-target libraries built by two different
>> host arches will have different metadata hashes in the filenames,
>> because the hash includes the full "rustc -Vv" version output, 
>> including
>> the host triple.
> 
> koji is doing the right thing. For audits and QA checks a noarch package 
> must be produced exactly the same ways regardless of the arch of the 
> builder.
> 
> Either recording the triplet is meaningless, and you should inhibit 
> it(the same way reproducible builds inhibit date recording), or it 
> serves a purpose, and your builds are archfull by construction.

It's not so black and white. In theory, the only thing that should
matter is the target triple, but the metadata also hashes the metadata
of all its build dependencies. That in turn may include procedural
macros (essentially compiler plugins) which are host binaries. The end
result for the target should be the same, but it's a real concern for
upstream that ignoring the host would break caching that relies on
filename differences.
_______________________________________________
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




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Users]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]

  Powered by Linux