Hehe. http://www.imdb.com/title/tt0584442/quotes?qt0399229
Ali Saidi wrote: > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/366/ > > > How about leaving the tool name grey? > > Ali > > > - Ali > > > On January 6th, 2011, 11:15 a.m., Steve Reinhardt wrote: > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, > and Nathan Binkert. > By Steve Reinhardt. > > /Updated 2011-01-06 11:15:30/ > > > Description > > scons: show sources and targets when building. > > I like the brevity of Ali's recent change, but the ambiguity of > sometimes showing the source and sometimes the target is a little > confusing. This patch makes scons typically list all sources and > all targets for each action, with the common path prefix factored > out for brevity. It's a little more verbose now but also more > informative. > > > Testing > > quick regressions pass > > > Diffs > > * SConstruct (9f9e10967912) > * src/SConscript (9f9e10967912) > * src/arch/SConscript (9f9e10967912) > * src/arch/isa_parser.py (9f9e10967912) > > View Diff <http://reviews.m5sim.org/r/366/diff/> > > > Screenshots > > sample colorized output <http://reviews.m5sim.org/r/366/s/1/> > > ------------------------------------------------------------------------ > > _______________________________________________ > m5-dev mailing list > m5-dev@m5sim.org > http://m5sim.org/mailman/listinfo/m5-dev > _______________________________________________ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev
