From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 908ABA0562; Fri, 3 Apr 2020 11:36:10 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id D634B1C128; Fri, 3 Apr 2020 11:36:09 +0200 (CEST) Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id 2F8C41C0D2 for ; Fri, 3 Apr 2020 11:36:06 +0200 (CEST) IronPort-SDR: EsHxYhVEzN2Q9ULTMJHdhTVHrNYrZEQheJq2Z5lsQiuGnpy6l6qWOMRyz3sug4SjxcpzhclysI 70cj0R/A0YWw== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 03 Apr 2020 02:36:04 -0700 IronPort-SDR: oeYM0VB+9Obgfs3ycm74yxofVxyTRP1xR37RI7z1+tXwdWBPBgLMOsraGLAdiul8RpLO8jUcXR fvxLZLO3rpfg== X-IronPort-AV: E=Sophos;i="5.72,339,1580803200"; d="scan'208";a="423469762" Received: from bricha3-mobl.ger.corp.intel.com ([10.249.36.192]) by orsmga005-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-SHA; 03 Apr 2020 02:36:02 -0700 Date: Fri, 3 Apr 2020 10:35:58 +0100 From: Bruce Richardson To: David Marchand Cc: Pavan Nikhilesh Bhagavatula , Jerin Jacob , Thomas Monjalon , Ferruh Yigit , dev , Neil Horman Message-ID: <20200403093558.GA547@bricha3-MOBL.ger.corp.intel.com> References: <20200402183253.866-1-jerinjacobk@gmail.com> <20200403091034.GA543@bricha3-MOBL.ger.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Subject: Re: [dpdk-dev] [EXT] Re: [PATCH v2 1/2] build: meson make experimental tag as global X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Fri, Apr 03, 2020 at 11:14:03AM +0200, David Marchand wrote: > On Fri, Apr 3, 2020 at 11:10 AM Bruce Richardson > wrote: > > > diff --git a/drivers/meson.build b/drivers/meson.build > > > index 987e616ae7..4d8f842ab8 100644 > > > --- a/drivers/meson.build > > > +++ b/drivers/meson.build > > > @@ -20,7 +20,7 @@ dpdk_driver_classes = ['common', > > > disabled_drivers = run_command(list_dir_globs, get_option('disable_drivers'), > > > ).stdout().split() > > > > > > -default_cflags = machine_args > > > +default_cflags = machine_args + ['-DALLOW_EXPERIMENTAL_API'] > > > if cc.has_argument('-Wno-format-truncation') > > > default_cflags += '-Wno-format-truncation' > > > endif > > > > I see nothing wrong with that. > > > > Out of interest, is there a reason we don't just set it at the top level as > > a project c_arg, e.g. after setting "-include rte_config.h" in > > config/meson.build? > > As long as it is not inherited by examples and external apps, I am > fine with any simple solution :-). > It will be inherited by everything inside the current build, but building any examples separately or any external apps separately won't be affected. /Bruce