DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] mk: mute build of test resources
@ 2016-06-16 22:01 Thomas Monjalon
  2016-06-17  9:55 ` Bruce Richardson
  0 siblings, 1 reply; 3+ messages in thread
From: Thomas Monjalon @ 2016-06-16 22:01 UTC (permalink / raw)
  To: Jan Viktorin; +Cc: dev

The objcopy and tar commands were printed even in quiet mode.
They are now replaced by a simple line and still visible in verbose mode.

Fixes: ab64f5df8004 ("app/test: support resources externally linked")
Fixes: 66819e6c11d8 ("app/test: support resources archived by tar")

Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
---
 app/test/Makefile | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/app/test/Makefile b/app/test/Makefile
index 053f3a2..5e3ebdc 100644
--- a/app/test/Makefile
+++ b/app/test/Makefile
@@ -42,7 +42,8 @@ ifeq ($(CONFIG_RTE_APP_TEST),y)
 define linked_resource
 SRCS-y += $(1).res.o
 $(1).res.o: $(2)
-	$(OBJCOPY) -I binary -B $(RTE_OBJCOPY_ARCH) -O $(RTE_OBJCOPY_TARGET) \
+	@  echo '  MKRES $$@'
+	$Q $(OBJCOPY) -I binary -B $(RTE_OBJCOPY_ARCH) -O $(RTE_OBJCOPY_TARGET) \
 		--rename-section                                         \
 			.data=.rodata,alloc,load,data,contents,readonly  \
 		--redefine-sym _binary__dev_stdin_start=beg_$(1)         \
@@ -54,7 +55,8 @@ endef
 ifeq ($(CONFIG_RTE_APP_TEST_RESOURCE_TAR),y)
 define linked_tar_resource
 $(1).tar: $(2)
-	tar -C $$(dir $$<) -cf $$@ $$(notdir $$<)
+	@  echo '  TAR $$@'
+	$Q tar -C $$(dir $$<) -cf $$@ $$(notdir $$<)
 $(call linked_resource,$(1),$(1).tar)
 endef
 else # ! CONFIG_RTE_APP_TEST_RESOURCE_TAR
-- 
2.7.0

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] mk: mute build of test resources
  2016-06-16 22:01 [dpdk-dev] [PATCH] mk: mute build of test resources Thomas Monjalon
@ 2016-06-17  9:55 ` Bruce Richardson
  2016-06-17 10:10   ` Thomas Monjalon
  0 siblings, 1 reply; 3+ messages in thread
From: Bruce Richardson @ 2016-06-17  9:55 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: Jan Viktorin, dev

On Fri, Jun 17, 2016 at 12:01:09AM +0200, Thomas Monjalon wrote:
> The objcopy and tar commands were printed even in quiet mode.
> They are now replaced by a simple line and still visible in verbose mode.
> 
> Fixes: ab64f5df8004 ("app/test: support resources externally linked")
> Fixes: 66819e6c11d8 ("app/test: support resources archived by tar")
> 
> Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
> ---
Acked-by: Bruce Richardson <bruce.richardson@intel.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] mk: mute build of test resources
  2016-06-17  9:55 ` Bruce Richardson
@ 2016-06-17 10:10   ` Thomas Monjalon
  0 siblings, 0 replies; 3+ messages in thread
From: Thomas Monjalon @ 2016-06-17 10:10 UTC (permalink / raw)
  To: Bruce Richardson; +Cc: Jan Viktorin, dev

> > The objcopy and tar commands were printed even in quiet mode.
> > They are now replaced by a simple line and still visible in verbose mode.
> > 
> > Fixes: ab64f5df8004 ("app/test: support resources externally linked")
> > Fixes: 66819e6c11d8 ("app/test: support resources archived by tar")
> > 
> > Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
> > ---
> Acked-by: Bruce Richardson <bruce.richardson@intel.com>

Applied

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-06-17 10:10 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-16 22:01 [dpdk-dev] [PATCH] mk: mute build of test resources Thomas Monjalon
2016-06-17  9:55 ` Bruce Richardson
2016-06-17 10:10   ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).