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 BB1C8A034C; Tue, 18 Aug 2020 16:31:50 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id B0FD42B94; Tue, 18 Aug 2020 16:31:49 +0200 (CEST) Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id 174E61F28 for ; Tue, 18 Aug 2020 16:31:47 +0200 (CEST) IronPort-SDR: xz+sSKEMdXJVwq06ObKeNnnPT2SdJ/zxKHwe4Kz951x33jepulQOEAlGy5BkF7qiM/4ED7jsBw Ki2tuZyNG7PA== X-IronPort-AV: E=McAfee;i="6000,8403,9716"; a="216439241" X-IronPort-AV: E=Sophos;i="5.76,327,1592895600"; d="scan'208";a="216439241" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 18 Aug 2020 07:31:46 -0700 IronPort-SDR: p8ErZz7618MEBqUaOifHYR4UBIdtlw/J1RW0pHBt4X2eFdJrTtZvEPw+qDJVJO5ivuc3euR/fh deWen3eilLgQ== X-IronPort-AV: E=Sophos;i="5.76,327,1592895600"; d="scan'208";a="471829275" Received: from bricha3-mobl.ger.corp.intel.com ([10.252.19.21]) by orsmga005-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-SHA; 18 Aug 2020 07:31:45 -0700 Date: Tue, 18 Aug 2020 15:31:42 +0100 From: Bruce Richardson To: Ciara Power Cc: dev@dpdk.org, thomas@monjalon.net Message-ID: <20200818143142.GF500@bricha3-MOBL.ger.corp.intel.com> References: <20200807123009.21266-1-ciara.power@intel.com> <20200807123009.21266-13-ciara.power@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200807123009.21266-13-ciara.power@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 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/build_sample_apps.rst b/doc/guides/linux_gsg/build_sample_apps.rst > index 2f606535c3..afa94acbbe 100644 > --- a/doc/guides/linux_gsg/build_sample_apps.rst > +++ b/doc/guides/linux_gsg/build_sample_apps.rst > @@ -18,14 +18,8 @@ Compiling a Sample Application > Once an DPDK target environment directory has been created (such as ``x86_64-native-linux-gcc``), > it contains all libraries and header files required to build an application. > This reference to x86_64-native-linuxapp-gcc is also a make-build reference. It should be removed. In fact, since building example apps separately is covered in section 3.2.3, I think we can rename this whole section from "compiling and running sample applications" to just "running sample applications". Rather than renumbering the sections and heading levels, section 6.1 can be kept just referring back to 3.2.3, since that info may well be useful to the reader who starts on this section. > -When compiling an application in the Linux* environment on the DPDK, the following variables must be exported: > - > -* ``RTE_SDK`` - Points to the DPDK installation directory. > - > -* ``RTE_TARGET`` - Points to the DPDK target environment directory. > - > The following is an example of creating the ``helloworld`` application, which runs in the DPDK Linux environment. > -This example may be found in the ``${RTE_SDK}/examples`` directory. > +This example may be found in the ``${DPDK installation directory}/examples`` directory. > > The directory contains the ``main.c`` file. This file, when combined with the libraries in the DPDK target environment, > calls the various functions to initialize the DPDK environment, > @@ -35,8 +29,6 @@ By default, the binary is generated in the build directory. > .. code-block:: console > > cd examples/helloworld/ > - export RTE_SDK=$HOME/DPDK > - export RTE_TARGET=x86_64-native-linux-gcc > > make > CC main.o > @@ -55,10 +47,8 @@ By default, the binary is generated in the build directory. > > .. code-block:: console > > - export RTE_SDK=/home/user/DPDK > - cp -r $(RTE_SDK)/examples/helloworld my_rte_app > + cp -r $(DPDK installation directory)/examples/helloworld my_rte_app > cd my_rte_app/ > - export RTE_TARGET=x86_64-native-linux-gcc > > make > CC main.o Drop all this text and refer to previous chapter. > @@ -232,7 +222,7 @@ If the DPDK cannot allocate enough memory on each socket, the EAL initialization > Additional Sample Applications > ------------------------------ > > -Additional sample applications are included in the ${RTE_SDK}/examples directory. > +Additional sample applications are included in the ${DPDK installation directory}/examples directory. I think "${DPDK installation directory}/examples" can be written simply as "DPDK examples" to make it a plain sentence without any explicit path references. > These sample applications may be built and run in a manner similar to that described in earlier sections in this manual. > In addition, see the *DPDK Sample Applications User Guide* for a description of the application, > specific instructions on compilation and execution and some explanation of the code. The subsection on "Additonal Test Applications" following this block seems very out of date, and should be removed IMHO.