public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
From: Ryan Hill <dirtyepic@gentoo.org>
To: gentoo-dev@lists.gentoo.org
Subject: [gentoo-dev]  Re: [RFC] Dropping (or enabling only on request) bootstrap from SCM eclasses
Date: Thu, 24 Sep 2009 18:26:03 -0600	[thread overview]
Message-ID: <20090924182603.73b8b43a@gentoo.org> (raw)
In-Reply-To: 200909240339.27356.reavertm@poczta.fm

[-- Attachment #1: Type: text/plain, Size: 1176 bytes --]

On Thu, 24 Sep 2009 03:39:26 +0200
Maciej Mrozowski <reavertm@poczta.fm> wrote:

> And this is the problem (some people may be even unaware of it).
> In pre EAPI-2 it was sufficient to do the following in live ebuilds:
> 
> inherit ${some_eclass} ${scm_eclass}
> 
> ${scm_eclass} inherited as last one, would just shadow src_unpack, providing 
> what we want.

Oogh, no.  Relying on ordering of the inherit line is never a good idea.  If
you need to call scm_src_unpack, call scm_src_unpack.

> Because SCM bootstrap is either not used at all, or used very rarely, there's 
> suggestion to:
> - either drop it
> - or (preferably) to make SCM eclasses export src_prepare only on specific 
> request
> - or to make it easier - to not export it at all - thus making it required for 
> developer to intentionally invoke ${ECLASS}_src_prepare if bootstrapping is 
> required.

Why not do number 3 but call it ${ECLASS}_bootstrap?  If people need it they
can call it from src_prepare.


-- 
fonts,                             Character is what you are in the dark.
gcc-porting,
wxwidgets @ gentoo     EFFD 380E 047A 4B51 D2BD C64F 8AA8 8346 F9A4 0662

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

      parent reply	other threads:[~2009-09-25  0:26 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-24  1:39 [gentoo-dev] [RFC] Dropping (or enabling only on request) bootstrap from SCM eclasses Maciej Mrozowski
2009-09-24  6:32 ` Ulrich Mueller
2009-09-24 13:46   ` Maciej Mrozowski
2009-09-29  0:53     ` Donnie Berkholz
2009-09-25  0:26 ` Ryan Hill [this message]

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=20090924182603.73b8b43a@gentoo.org \
    --to=dirtyepic@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