[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: 01/03: gnu: lld-as-ld-wrapper: Add version 15.
From: |
Maxim Cournoyer |
Subject: |
Re: 01/03: gnu: lld-as-ld-wrapper: Add version 15. |
Date: |
Sat, 19 Nov 2022 20:30:17 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Hi,
Marius Bakke <marius@gnu.org> writes:
> Christopher Baines <mail@cbaines.net> skriver:
[...]
>> I spotted this change when looking at [1] as the Guix Data Service is
>> failing at extracting a consistent derivation for lld-as-ld-wrapper@0.
>>
>> 1: https://qa.guix.gnu.org/issue/59367
>>
>> Maybe that could be viewed as a bug in the Guix Data Service, as it
>> assumes/wants packages to be unique by name and version (at least the
>> public/visible packages).
>>
>> But this is something raised by the linter also:
>>
>> → ./pre-inst-env guix lint lld-as-ld-wrapper
>> guix lint: warning: ambiguous package specification `lld-as-ld-wrapper'
>> guix lint: warning: choosing lld-as-ld-wrapper@0 from
>> gnu/packages/llvm.scm:1594:2
>>
>> Is there something that can be done about this, perhaps by using the lld
>> package version in the name or version for this package, or by hiding
>> this package?
>
> I think using the LLD version is sensible. I made this change in
> cd9161372aa3ec9b8902385d5937d04e1868495c and also inherited the
> home-page and license fields.
>
> Maxim: please let me know if you this this was misguided!
No, it looks reasonable to me :-).
--
Thanks,
Maxim