Messages in this thread Patch in this message | | | Date | Thu, 26 Dec 2013 21:10:07 +0100 | From | Jiri Olsa <> | Subject | Re: [PATCH 0/9] tools: Factor traceevent/perf Makefile |
| |
On Thu, Dec 26, 2013 at 10:38:42AM -0300, Arnaldo Carvalho de Melo wrote: > Em Thu, Dec 19, 2013 at 02:41:58PM +0100, Jiri Olsa escreveu: > > hi, > > sending out the change to introduce single: > > INSTALL trace_plugins > > build output line. Some more code factoring was needed. > > > > Also updated the tests/make install tests to check for > > installed plugins. > > Ok, better now, applied, but can you take a look at: > > [acme@ssdandy linux]$ make O=/tmp/build/perf -C tools/perf/ install-bin > make: Entering directory `/home/acme/git/linux/tools/perf' > BUILD: Doing 'make -j8' parallel build > GEN perf-archive > SUBDIR /home/acme/git/linux/tools/lib/traceevent/ > make[3]: Nothing to be done for `plugins'. > make[2]: Nothing to be done for `plugins'. > INSTALL GTK UI > INSTALL binaries > INSTALL libexec > INSTALL perf-archive > INSTALL perl-scripts > INSTALL python-scripts > INSTALL perf_completion-script > INSTALL tests > make: Leaving directory `/home/acme/git/linux/tools/perf' > [acme@ssdandy linux]$ > > Removing those two "Nothing to be done" lines, please?
never happy, are we? ;-) looks like change below would do.. I'll check/test more and send out.
jirka
--- diff --git a/tools/scripts/Makefile.include b/tools/scripts/Makefile.include index 8abbef1..555de0b 100644 --- a/tools/scripts/Makefile.include +++ b/tools/scripts/Makefile.include @@ -55,7 +55,7 @@ descend = \ +mkdir -p $(OUTPUT)$(1) && \ $(MAKE) $(COMMAND_O) subdir=$(if $(subdir),$(subdir)/$(1),$(1)) $(PRINT_DIR) -C $(1) $(2) -QUIET_SUBDIR0 = +$(MAKE) $(COMMAND_O) -C # space to separate -C and subdir +QUIET_SUBDIR0 = +$(MAKE) $(COMMAND_O) -s -C # space to separate -C and subdir QUIET_SUBDIR1 = ifneq ($(findstring $(MAKEFLAGS),s),s)
| |