From: Zac Medico <zmedico@gentoo.org>
To: gentoo-dev@lists.gentoo.org
Subject: Re: [gentoo-dev] RFC: [Future EAPI] Exporting phase funcs from direct inherits only
Date: Tue, 14 Aug 2012 14:09:17 -0700 [thread overview]
Message-ID: <502ABE7D.9050204@gentoo.org> (raw)
In-Reply-To: <20120814225413.7aff9e02@pomiocik.lan>
On 08/14/2012 01:54 PM, Michał Górny wrote:
> On Tue, 14 Aug 2012 21:45:56 +0100
> Ciaran McCreesh <ciaran.mccreesh@googlemail.com> wrote:
>
>> On Tue, 14 Aug 2012 11:44:49 +0200
>> Michał Górny <mgorny@gentoo.org> wrote:
>>> As some of you may have noticed, lately introduced 'double include
>>> preventions' have caused changes in effective phase functions in a
>>> few ebuilds. Also, often it is undesirable that change in inherits
>>> of an eclass may cause an undesired change of exported functions.
>>
>> The problem here is that eclasses aren't clearly split between
>> "utility" and "does stuff", so people are inheriting "does stuff"
>> eclasses to get utilities. The fix is to stop having stupidly huge
>> complicated eclasses; changing inherit behaviour is just wallpapering
>> over the gaping hole.
Ciaran's assessment sounds pretty accurate to me.
> Soo, how do you propose to handle bug 422533 without changing inherit
> behavior?
Close it as WONTFIX. The ifndef thing that we're doing now seems like a
reasonable approach.
--
Thanks,
Zac
next prev parent reply other threads:[~2012-08-14 21:17 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-14 9:44 [gentoo-dev] RFC: [Future EAPI] Exporting phase funcs from direct inherits only Michał Górny
2012-08-14 19:46 ` Zac Medico
2012-08-14 20:39 ` Michał Górny
2012-08-18 3:20 ` Mike Frysinger
2012-08-14 20:45 ` Ciaran McCreesh
2012-08-14 20:54 ` Michał Górny
2012-08-14 20:56 ` Ciaran McCreesh
2012-08-14 21:01 ` hasufell
2012-08-14 21:17 ` Ciaran McCreesh
2012-08-14 21:09 ` Michał Górny
2012-08-14 21:11 ` Ciaran McCreesh
2012-08-14 21:09 ` Zac Medico [this message]
2012-08-14 21:51 ` Michał Górny
2012-08-14 21:56 ` Ciaran McCreesh
2012-08-14 21:59 ` Zac Medico
2012-08-18 3:19 ` Mike Frysinger
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=502ABE7D.9050204@gentoo.org \
--to=zmedico@gentoo.org \
--cc=gentoo-dev@lists.gentoo.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox