Gentoo Logo
Gentoo Spaceship




Note: Due to technical difficulties, the Archives are currently not up to date. GMANE provides an alternative service for most mailing lists.
c.f. bug 424647
List Archive: gentoo-dev
Navigation:
Lists: gentoo-dev: < Prev By Thread Next > < Prev By Date Next >
Headers:
To: gentoo-dev@g.o
From: Michał Górny <gentoo@...>
Subject: Re: New eclass: scons.eclass
Date: Mon, 23 Aug 2010 19:16:38 +0200
On Mon, 23 Aug 2010 12:39:50 -0400
Mike Frysinger <vapier@g.o> wrote:

> On Monday, August 23, 2010 12:24:54 Michał Górny wrote:
> > The third revision, almost all suggestions applied. The ChangeLog:
> 
> if there are suggestions you've actively ignored, those should be
> noted

I've replied to the specific suggestions but if you want, here's a
short summary:
- stripping ! from varname in use_scons() suggestion (I don't see a
  real point there as I replied you),
- src_prepare() calling base_src_prepare() (it is better to inherit
  base within the specific ebuild),
- src_install() calling base_src_install() (the base.eclass
  implementation calls make).

> i'm not sure caching the value and using it in between runs is a good
> idea. unless you also cache the thing you're caching and compare it
> to make sure your cache is no longer invalid.
> 
> i.e. export _SCONSOPTS_MAKEOPTS=${MAKEOPTS}, and then on every escons 
> invocation, make sure ${MAKEOPTS} hasnt changed in which case you
> need to regenerate it.  or just avoid the cache altogether and leave
> SCONSOPTS as a hook specific to scons.

Can do, though I don't see a reason why anyone would mangle MAKEOPTS in
a middle of an ebuild using SCons.

PS Why don't your mails contain 'Reply-To' header like others do?

-- 
Best regards,
Michał Górny

<http://mgorny.alt.pl>
<xmpp:mgorny@...>
Attachment:
signature.asc (PGP signature)
Replies:
Re: New eclass: scons.eclass
-- Mike Frysinger
References:
New eclass: scons.eclass
-- Michał Górny
Re: New eclass: scons.eclass
-- Michał Górny
Re: New eclass: scons.eclass
-- Michał Górny
Re: New eclass: scons.eclass
-- Mike Frysinger
Navigation:
Lists: gentoo-dev: < Prev By Thread Next > < Prev By Date Next >
Previous by thread:
Re: New eclass: scons.eclass
Next by thread:
Re: New eclass: scons.eclass
Previous by date:
Re: New eclass: scons.eclass
Next by date:
Re: Re: The future of sys-apps/openrc in Gentoo


Updated Jun 29, 2012

Summary: Archive of the gentoo-dev mailing list.

Donate to support our development efforts.

Copyright 2001-2013 Gentoo Foundation, Inc. Questions, Comments? Contact us.