Message ID | 1474587767-20033-1-git-send-email-thomas.monjalon@6wind.com (mailing list archive) |
---|---|
State | Accepted, archived |
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id 7D1A75A72; Fri, 23 Sep 2016 01:43:07 +0200 (CEST) Received: from mail-wm0-f49.google.com (mail-wm0-f49.google.com [74.125.82.49]) by dpdk.org (Postfix) with ESMTP id 16D9C2BDC for <dev@dpdk.org>; Fri, 23 Sep 2016 01:43:06 +0200 (CEST) Received: by mail-wm0-f49.google.com with SMTP id l132so635333wmf.0 for <dev@dpdk.org>; Thu, 22 Sep 2016 16:43:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=8qvhPdlIBACLI7OiGEQLMMff3iDrhlx3Z5P+tPzeKGM=; b=xS5tRUZv8BOckfnIwTfCG2Hwi14A9/7tzkEtr4VUN/zSxGbGC/liwPzkgrK6VTF1z/ l31gbiEkYVH/PE/A0szXf94/Z3big/wKA2qfagNAkuaiUANq0NmpgOkEVkkUFc4egAof tpltu9r5aOji/aVS12gPj7EwK/XmddHIDJb+6O+J2gP5/z+4PGuMGF2N5w67LnP/YvWQ zwKFpHwe5BUw+o3M+nZXmaj1szI7lbW5CQzCcUNzFH2blvPVx3IQEBZxxr/efCaKMxL+ ELZqJvqOjyLReQKzyFCWDYIRuQajuOPpO12ypefrXFB6Z+BigV0IK0aTkpcinkiTtpGK GVjg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=8qvhPdlIBACLI7OiGEQLMMff3iDrhlx3Z5P+tPzeKGM=; b=Qz3jdz+eycGCDv735JbkueSdzG1XT1uaV3pNQ3mpJmzsfkUyqMXOHvCteL58waGjDx GDMQLG5xYsdkDDZb6xCQYVc09Ki9LvKb118nZUh0N4cPl0TL+oWGWEzya3gzcXTKTRGj t0GHdNEAxu9yF/+x8NEHY2L0rUWHj5s3dTcqwWxtlnvrlSpk5+qtCAuY4r3UzWSycKwi Cdsfad51ofsAV1PAsYEegLruaDIj/HYHV/0O9vwP4q2IFuzRQU33+MEzy1hTZ4NlJt2E FtJrWiBVeyrFbG+JCVOt+3uzwL1fO+6p3stPcoJOCPXAojka3riGixo4k41EKc/i+x3k TMbA== X-Gm-Message-State: AA6/9RnsZgs6u6yBuKPykdTmjbjddarA4dHTIflR0tbLnzk7/8J30NuC4Pka1CI10meQETo4 X-Received: by 10.194.140.77 with SMTP id re13mr4833351wjb.79.1474587785805; Thu, 22 Sep 2016 16:43:05 -0700 (PDT) Received: from XPS13.localdomain (guy78-1-82-235-116-147.fbx.proxad.net. [82.235.116.147]) by smtp.gmail.com with ESMTPSA id lj2sm4301305wjc.38.2016.09.22.16.43.04 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Thu, 22 Sep 2016 16:43:04 -0700 (PDT) From: Thomas Monjalon <thomas.monjalon@6wind.com> To: dev@dpdk.org Cc: Ferruh Yigit <ferruh.yigit@intel.com> Date: Fri, 23 Sep 2016 01:42:47 +0200 Message-Id: <1474587767-20033-1-git-send-email-thomas.monjalon@6wind.com> X-Mailer: git-send-email 2.7.0 In-Reply-To: <6873211.QDWnoOkD9j@xps13> References: <6873211.QDWnoOkD9j@xps13> Subject: [dpdk-dev] [PATCH] mk: fix verbosity zero X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK <dev.dpdk.org> List-Unsubscribe: <http://dpdk.org/ml/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://dpdk.org/ml/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <http://dpdk.org/ml/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Commit Message
Thomas Monjalon
Sept. 22, 2016, 11:42 p.m. UTC
Verbosity is considered enabled when $V is not empty.
So V=0 and V=1 are equivalent.
It is fixed by unsetting V when it is 0.
A side effect is to fix kernel module compilation verbosity
which is set to 0 when V is empty.
Reported-by: Ferruh Yigit <ferruh.yigit@intel.com>
Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
---
mk/rte.sdkroot.mk | 3 +++
mk/rte.vars.mk | 3 +++
2 files changed, 6 insertions(+)
Comments
On 9/23/2016 12:42 AM, Thomas Monjalon wrote: > Verbosity is considered enabled when $V is not empty. > So V=0 and V=1 are equivalent. > It is fixed by unsetting V when it is 0. > > A side effect is to fix kernel module compilation verbosity > which is set to 0 when V is empty. > > Reported-by: Ferruh Yigit <ferruh.yigit@intel.com> > Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com> > --- > mk/rte.sdkroot.mk | 3 +++ > mk/rte.vars.mk | 3 +++ > 2 files changed, 6 insertions(+) > > diff --git a/mk/rte.sdkroot.mk b/mk/rte.sdkroot.mk > index 55a9d8a..04ad523 100644 > --- a/mk/rte.sdkroot.mk > +++ b/mk/rte.sdkroot.mk > @@ -34,6 +34,9 @@ MAKEFLAGS += --no-print-directory > # define Q to '@' or not. $(Q) is used to prefix all shell commands to > # be executed silently. > Q=@ > +ifeq '$V' '0' > +override V= > +endif > ifdef V Why not simply: -ifdef V +ifeq ($(V),1) > ifeq ("$(origin V)", "command line") > Q= > diff --git a/mk/rte.vars.mk b/mk/rte.vars.mk > index 28982a5..c240a0e 100644 > --- a/mk/rte.vars.mk > +++ b/mk/rte.vars.mk > @@ -46,6 +46,9 @@ endif > # define Q to '@' or not. $(Q) is used to prefix all shell commands to > # be executed silently. > Q=@ > +ifeq '$V' '0' > +override V= > +endif > ifdef V > ifeq ("$(origin V)", "command line") > Q= >
2016-09-23 10:06, Ferruh Yigit: > On 9/23/2016 12:42 AM, Thomas Monjalon wrote: > > Verbosity is considered enabled when $V is not empty. > > So V=0 and V=1 are equivalent. > > It is fixed by unsetting V when it is 0. > > > > A side effect is to fix kernel module compilation verbosity > > which is set to 0 when V is empty. > > > > Reported-by: Ferruh Yigit <ferruh.yigit@intel.com> > > Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com> [...] > > --- a/mk/rte.sdkroot.mk > > +++ b/mk/rte.sdkroot.mk > > @@ -34,6 +34,9 @@ MAKEFLAGS += --no-print-directory > > # define Q to '@' or not. $(Q) is used to prefix all shell commands to > > # be executed silently. > > Q=@ > > +ifeq '$V' '0' > > +override V= > > +endif > > ifdef V > > Why not simply: > -ifdef V > +ifeq ($(V),1) Because: - V could have a higher value - $(if $V) construct is used in several places in the makefiles See git grep '$(if $(*V'
On 9/23/2016 12:42 AM, Thomas Monjalon wrote: > Verbosity is considered enabled when $V is not empty. > So V=0 and V=1 are equivalent. > It is fixed by unsetting V when it is 0. > > A side effect is to fix kernel module compilation verbosity > which is set to 0 when V is empty. > > Reported-by: Ferruh Yigit <ferruh.yigit@intel.com> > Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
2016-09-23 10:33, Ferruh Yigit: > On 9/23/2016 12:42 AM, Thomas Monjalon wrote: > > Verbosity is considered enabled when $V is not empty. > > So V=0 and V=1 are equivalent. > > It is fixed by unsetting V when it is 0. > > > > A side effect is to fix kernel module compilation verbosity > > which is set to 0 when V is empty. > > > > Reported-by: Ferruh Yigit <ferruh.yigit@intel.com> > > Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com> > > Acked-by: Ferruh Yigit <ferruh.yigit@intel.com> Applied with additional comment: "It is a well spread shortcut in makefiles, see git grep '$(if $(*V'"
diff --git a/mk/rte.sdkroot.mk b/mk/rte.sdkroot.mk index 55a9d8a..04ad523 100644 --- a/mk/rte.sdkroot.mk +++ b/mk/rte.sdkroot.mk @@ -34,6 +34,9 @@ MAKEFLAGS += --no-print-directory # define Q to '@' or not. $(Q) is used to prefix all shell commands to # be executed silently. Q=@ +ifeq '$V' '0' +override V= +endif ifdef V ifeq ("$(origin V)", "command line") Q= diff --git a/mk/rte.vars.mk b/mk/rte.vars.mk index 28982a5..c240a0e 100644 --- a/mk/rte.vars.mk +++ b/mk/rte.vars.mk @@ -46,6 +46,9 @@ endif # define Q to '@' or not. $(Q) is used to prefix all shell commands to # be executed silently. Q=@ +ifeq '$V' '0' +override V= +endif ifdef V ifeq ("$(origin V)", "command line") Q=