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 Development <gentoo-dev@g.o>
From: Arfrever Frehtes Taifersar Arahesis <Arfrever@g.o>
Subject: Re: Minor changes in python.eclass and distutils.eclass
Date: Tue, 6 Jul 2010 00:07:01 +0200
2010-07-05 23:49:58 Samuli Suominen napisał(a):
> On 07/06/2010 12:38 AM, Arfrever Frehtes Taifersar Arahesis wrote:
> > 2010-07-05 23:28:01 Samuli Suominen napisał(a):
> >> On 07/05/2010 11:23 PM, Arfrever Frehtes Taifersar Arahesis wrote:
> >>> 2010-07-05 21:18:57 Mark Loeser napisał(a):
> >>>> Arfrever Frehtes Taifersar Arahesis <Arfrever@g.o> said:
> >>>>> 2010-07-05 20:00:11 Mark Loeser napisał(a):
> >>>>>> Everyone else has already made valid points.  I'm just picking this one
> >>>>>> to reply to now.  Please remove the colors you have added.  If you need
> >>>>>> a new function, say "eqawarn", we should have that added in the next
> >>>>>> EAPI with a description of when and where to use it.
> >>>>>
> >>>>> In case of the colored message added in this patch, if einfo/elog/ewarn/eqawarn/eerror was used,
> >>>>> then its output wouldn't be logged by Portage.
> >>>>
> >>>> I don't understand what you are trying to say.
> >>>
> >>> Portage doesn't log output of einfo/elog/ewarn/eqawarn/eerror called in global scope.
> >>
> >> Can you point to the location of python.eclass / distutils.eclass where
> >> you need to send output from global scope, please?
> > 
> > It's at the beginning of this patch. NEED_PYTHON is parsed in global scope.
> > 
> 
> OK, so let me get this right... are trying to justify using all of the
> echo+custom colorization in the eclass by this one occurance, in basis
> of it doesn't get logged otherwise?

I was justifying using colors in this case (NEED_PYTHON). I'm completely against not using colors
in status messages (e.g. "Building of dev-python/setuptools-0.6.13 with CPython 3.1..."), which
are more useful, when they are easily noticeable by ebuild maintainers. I might agree to remove
colors in other deprecation warnings, but this patch wasn't adding any other deprecation warnings.
Not using colors in deprecation warnings doesn't have any benefits and only decreases noticeability
of deprecation warnings.

-- 
Arfrever Frehtes Taifersar Arahesis
Attachment:
signature.asc (This is a digitally signed message part.)
References:
Minor changes in python.eclass and distutils.eclass
-- Arfrever Frehtes Taifersar Arahesis
Re: Minor changes in python.eclass and distutils.eclass
-- Arfrever Frehtes Taifersar Arahesis
Re: Minor changes in python.eclass and distutils.eclass
-- Samuli Suominen
Navigation:
Lists: gentoo-dev: < Prev By Thread Next > < Prev By Date Next >
Previous by thread:
Re: Minor changes in python.eclass and distutils.eclass
Next by thread:
Re: Minor changes in python.eclass and distutils.eclass
Previous by date:
Re: Minor changes in python.eclass and distutils.eclass
Next by date:
Re: Minor changes in python.eclass and distutils.eclass


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.