Re: [gentoo-dev] [PATCH scons-utils] Support setting common SCons arguments using myesconsargs.

["Michał Górny" <mgorny@gentoo.org>]

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

On Tue, 25 Oct 2011 01:57:11 -0400
Mike Frysinger <vapier@gentoo.org> wrote:

> On Mon, Oct 24, 2011 at 03:46, Michał Górny wrote:
> > On Mon, 24 Oct 2011 03:42:24 +0000 Nathan Phillip Brink wrote:
> >> On Sun, Oct 23, 2011 at 08:20:37PM +0200, Micha?? G??rny wrote:
> >> > ---
> >> >  scons-utils.eclass |   33 +++++++++++++++++++++++++--------
> >> >  1 files changed, 25 insertions(+), 8 deletions(-)
> >> ...
> >> > +# @ECLASS-VARIABLE: myesconsargs
> >> > +# @DEFAULT_UNSET
> >> > +# @DESCRIPTION:
> >> > +# List of package-specific options to pass to all SCons calls.
> >> > Supposed to be +# set in src_configure().
> >>
> >> Shouldn't this variable be named MYESCONSARGS since it is being
> >> introduced into the global scope?
> >
> > I can't deny that but I'd like to keep it consistent with
> > cmake-utils and autotools-utils. Alternate approach would be to
> > make it local to src_configure() and add
> > scons-utils_src_configure() which would export it under internal
> > name but I don't think that's really worth the effort.
> 
> sounds like you want @VARIABLE and not @ECLASS-VARIABLE

Thanks, fixed.

-- 
Best regards,
Michał Górny

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