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 5574FA04AF; Wed, 19 Aug 2020 15:03:49 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 3F3F2AAD5; Wed, 19 Aug 2020 15:03:48 +0200 (CEST) Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id 6BF83AAB7 for ; Wed, 19 Aug 2020 15:03:46 +0200 (CEST) IronPort-SDR: HuWPIk+qBQaWeylLYu6KH/dz3T8pDv6ZhGyVN5pHTqiCUvZ8/yb21y7I9wJxaA93DCFK9KwP4n RP317wJwBHSg== X-IronPort-AV: E=McAfee;i="6000,8403,9717"; a="152515473" X-IronPort-AV: E=Sophos;i="5.76,331,1592895600"; d="scan'208";a="152515473" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by fmsmga104.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Aug 2020 06:03:34 -0700 IronPort-SDR: 4PKlKdr90A+xpLoeeEv3Ck++zEq9bORtSYvTcrvSnxNT+kRuT+wBLBRE2925GVGbZ1aSmbc9k9 zdoZOHjCHqjg== X-IronPort-AV: E=Sophos;i="5.76,331,1592895600"; d="scan'208";a="472226578" Received: from bricha3-mobl.ger.corp.intel.com ([10.252.0.199]) by orsmga005-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-SHA; 19 Aug 2020 06:03:33 -0700 Date: Wed, 19 Aug 2020 14:03:27 +0100 From: Bruce Richardson To: Ciara Power Cc: dev@dpdk.org, thomas@monjalon.net Message-ID: <20200819130327.GA813@bricha3-MOBL.ger.corp.intel.com> References: <20200807123009.21266-1-ciara.power@intel.com> <20200807123009.21266-13-ciara.power@intel.com> <20200818114053.GB500@bricha3-MOBL.ger.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200818114053.GB500@bricha3-MOBL.ger.corp.intel.com> Subject: Re: [dpdk-dev] [PATCH 20.11 12/19] doc: remove references to make in Linux gsg guides 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 Tue, Aug 18, 2020 at 12:40:53PM +0100, Bruce Richardson wrote: > On Fri, Aug 07, 2020 at 01:30:02PM +0100, Ciara Power wrote: > > Make is no longer supported for compiling DPDK, references are now > > removed in the documentation. > > > > Signed-off-by: Ciara Power > > --- > > doc/guides/linux_gsg/build_dpdk.rst | 58 ------------------- > > doc/guides/linux_gsg/build_sample_apps.rst | 16 +---- > > .../linux_gsg/cross_build_dpdk_for_arm64.rst | 42 -------------- > > doc/guides/linux_gsg/enable_func.rst | 3 - > > doc/guides/linux_gsg/intro.rst | 2 +- > > doc/guides/linux_gsg/linux_drivers.rst | 2 - > > doc/guides/linux_gsg/sys_reqs.rst | 4 -- > > 7 files changed, 4 insertions(+), 123 deletions(-) > > > > > diff --git a/doc/guides/linux_gsg/sys_reqs.rst b/doc/guides/linux_gsg/sys_reqs.rst > > index a124656bcb..a009f3146b 100644 > > --- a/doc/guides/linux_gsg/sys_reqs.rst > > +++ b/doc/guides/linux_gsg/sys_reqs.rst > > @@ -85,10 +85,6 @@ A number of DPDK components, such as libraries and poll-mode drivers (PMDs) have > > For DPDK builds using meson, the presence or absence of these dependencies will be > > automatically detected enabling or disabling the relevant components appropriately. > > > > I think you can drop the words "using meson" here as there is no > alternative any more. > > > -For builds using make, these components are disabled in the default configuration and > > -need to be enabled manually by changing the relevant setting to "y" in the build configuration file > > -i.e. the ``.config`` file in the build folder. > > - > > In each case, the relevant library development package (``-devel`` or ``-dev``) is needed to build the DPDK components. > > > > For libraries the additional dependencies include: > > In sys_reqs.rst there is also a reference to make in the "Required Tools > and Libraries". Change: > > "General development tools including make, and a supported C compiler such > as gcc (version 4.9+) or clang (version 3.4+)." > > To something like: > > "General development tools including a supported C compiler such > as gcc (version 4.9+) or clang (version 3.4+)." Also, one more thing I spot in that section is that we need to update our python minimum version to 3.5+ and drop references to 2.7. Therefore, I'd suggest just removing the whole sub-bullets and put a single bullet saying "Python, v3.5 or later"