>> Neat idea, but you need to check "sys.stdout.isatty()" and conditionally >> add the color. Then you're going to want a way to override that if you're >> piping through less :) Seems like we could add a --color option to the >> SCons command line. > > It actually crossed my mind earlier today to use colorization to distinguish > the common prefix part of the source path from the source-specific part, as > a way of eliminating the ambiguity of how to form the target. > > The isatty check would take care of the less problem, though... Actually less -r allows you to take the control characters, so the --color would override the isatty() == false and output the control codes anyway.
As a thought, TRANSFORM should probably take the action abbreviation (e.g. "CC") as a parameter if you're going to do the colorization. This is all overkill, but is a nice feature imho. Nate _______________________________________________ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev