Hi Keith, On 02/14/2015 07:13 PM, Keith Wiles wrote: > Signed-off-by: Keith Wiles <keith.wiles at intel.com> > --- > mk/rte.extvars.mk | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/mk/rte.extvars.mk b/mk/rte.extvars.mk > index 3e5a990..83a5721 100644 > --- a/mk/rte.extvars.mk > +++ b/mk/rte.extvars.mk > @@ -66,6 +66,10 @@ endif > RTE_OUTPUT ?= $(RTE_SRCDIR)/build > export RTE_OUTPUT > > +# define Q to '@' or not. $(Q) is used to prefix all shell commands to > +# be executed silently. > +Q=@ > + > # if we are building an external application, include SDK > # configuration and include project configuration if any > include $(RTE_SDK_BIN)/.config >
In the examples/ directory, rte.extvars.mk is never included directly. They use rte.vars.mk, which already properly defines the $(Q) variable (its value depends on V= argument). So I think we don't need this patch. Regards, Olivier
