* [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).