[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: The package/inherit trap
From: |
Maxim Cournoyer |
Subject: |
Re: The package/inherit trap |
Date: |
Mon, 07 Aug 2023 10:41:16 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Hello,
Mark H Weaver <mhw@netris.org> writes:
> Simon Tournier <zimon.toutoune@gmail.com> writes:
>
>> Hi Tobias,
>>
>> On Tue, 07 Mar 2023 at 22:11, Tobias Geerinckx-Rice <me@tobias.gr> wrote:
>>
>>> However, merely documenting something is not enough when we have the
>>> chance to fix misleading naming, as we do here. It would be nice to
>>> have, but orthogonal.
>>
>> I would not say it is orthogonal because renaming would not have been
>> enough, at least for me, in order to get the difference with ’inherit’.
>>
>> For sure, it is a real trap. :-) For instance,
>>
>> $ git log --grep='package/inherit' --oneline | grep use
>> 5f83dd03a2 gnu: kodi/wayland: Do not use package/inherit.
>> 6ecf88a6a1 gnu: poppler-next: Don't use 'package/inherit'.
>> 5a5b729d66 gnu: abseil-cpp: Don't use 'package/inherit'.
>> dbcf2b61b1 gnu: Fix erroneous uses of 'package/inherit'.
>> 2f97a666a5 gnu: python-urllib3: Don't use 'package/inherit' on
>> replacement package.
>> 4163b6d855 gnu: avahi: Don't use package/inherit.
>>
>> and in the light of this discussion, 5f83dd03a2 seems incorrect, no?
>
> I agree that commit 5f83dd03a2 is incorrect. 'kodi/wayland' is quite
> clearly a case where 'package/inherit' should be used.
Would you be able to write a bit of doc explaining when both should be
used? It's a common pitfalls among contributors, and I suspect few of
us have an understanding good enough to write it down in a manner clear
enough to be understood by new contributors.
--
Thanks,
Maxim