Re: [gentoo-dev] Minor changes in python.eclass and distutils.eclass

[Arfrever Frehtes Taifersar Arahesis <Arfrever@gentoo.org>]

[-- Attachment #1: Type: Text/Plain, Size: 1573 bytes --]

2010-07-05 18:26:40 Samuli Suominen napisał(a):
> On 07/05/2010 07:17 PM, Arfrever Frehtes Taifersar Arahesis wrote:
> > 2010-07-05 18:13:26 Samuli Suominen napisał(a):
> >> On 07/05/2010 06:23 PM, Arfrever Frehtes Taifersar Arahesis wrote:
> >>> These minor changes in python.eclass and distutils.eclass have been already
> >>> reviewed on alias of Gentoo Python Project. It's recommended to be familiar
> >>> with internals of current code before trying to understand these minor changes.
> >>> Suggestions about indentation and quoting will be rejected.
> >>>
> >>
> >> You have been already told to get rid of all the color customizations in
> >> the python eclasses here:
> >>
> >> http://bugs.gentoo.org/show_bug.cgi?id=309057#c2
> >> http://bugs.gentoo.org/show_bug.cgi?id=309057#c3
> >>
> >> [ .. ]
> >>
> >> http://bugs.gentoo.org/show_bug.cgi?id=309057#c5
> >>
> >> The bug was wrongly closed as fixed, as it's not really fixed before
> >> it's all punted
> > 
> > Colors can be used with echo.
> > 
> 
> Stop using echo for output and switch to standard output functions, like
> einfo/eerror/elog/... like told in
> http://bugs.gentoo.org/show_bug.cgi?id=309057#c5

You should read relevant part of comment #7:
"The colors can of course be continued to be used in outputs that are purely build
time outputting and not for communicating things for users like what cmake builds do."

python.eclass uses colors for build time outputting, which doesn't communicate anything for users.

-- 
Arfrever Frehtes Taifersar Arahesis

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]