From d650f4138eaa68c856d0879e39a78f66d205b017 Mon Sep 17 00:00:00 2001 From: Steve Reinhardt Date: Fri, 7 Jan 2011 21:50:13 -0800 Subject: scons: show sources and targets when building, and colorize output. 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. Somehow Ali talked me into adding colors too, which is a whole 'nother story. --- src/arch/SConscript | 2 +- src/arch/isa_parser.py | 2 -- 2 files changed, 1 insertion(+), 3 deletions(-) (limited to 'src/arch') diff --git a/src/arch/SConscript b/src/arch/SConscript index 620de4d1c..8c537182d 100644 --- a/src/arch/SConscript +++ b/src/arch/SConscript @@ -118,7 +118,7 @@ def isa_desc_action_func(target, source, env): cpu_models = [CpuModel.dict[cpu] for cpu in models] parser = isa_parser.ISAParser(target[0].dir.abspath, cpu_models) parser.parse_isa_desc(source[0].abspath) -isa_desc_action = MakeAction(isa_desc_action_func, " [ISA DESC] $STRIP_SOURCE") +isa_desc_action = MakeAction(isa_desc_action_func, Transform("ISA DESC", 1)) # Also include the CheckerCPU as one of the models if it is being # enabled via command line. diff --git a/src/arch/isa_parser.py b/src/arch/isa_parser.py index 2a03f1968..00bfd33d2 100755 --- a/src/arch/isa_parser.py +++ b/src/arch/isa_parser.py @@ -1980,13 +1980,11 @@ StaticInstPtr old_contents = f.read() f.close() if contents != old_contents: - print 'Updating', file os.remove(file) # in case it's write-protected update = True else: print 'File', file, 'is unchanged' else: - print ' [GENERATE]', file update = True if update: f = open(file, 'w') -- cgit v1.2.3