Re: [dpdk-dev] [PATCH v3 3/3] doc: update build-sdk-quick txt file

["Hunt, David" <david.hunt@intel.com>]

Hi Thomas


On 13/2/2018 11:41 PM, Thomas Monjalon wrote:
> 13/02/2018 13:18, Ferruh Yigit:
>> On 6/12/2017 1:50 PM, Mcnamara, John wrote:
>>>> -----Original Message-----
>>>> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of David Hunt
>>>> Sent: Wednesday, June 7, 2017 3:38 PM
>>>> To: dev@dpdk.org
>>>> Cc: thomas@monjalon.net; shreyansh.jain@nxp.com; Hunt, David
>>>> <david.hunt@intel.com>
>>>> Subject: [dpdk-dev] [PATCH v3 3/3] doc: update build-sdk-quick txt file
>>>>
>>>>    * Add in information about 'make defconfig'
>>>>    * Add in information about RTE_TEMPLATE environment variable
>>>>    * Explain the default directory for 'O=' a bit better
>>>>    * Clean up text alignment
>>>>
>>>> Signed-off-by: David Hunt <david.hunt@intel.com>
>>> Acked-by: John McNamara <john.mcnamara@intel.com>
>> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
> I made some comments in patch 2 about RTE_TEMPLATE and got no reply
> (in August).

Since you correctly point out that "make install T=" is an old syntax, and
this variable only encourages this, let's drop this patch altogether.

There are two patches in patchwork relevant to this cahange, one is
the change, and the other is the docs update. There was a third part,
which was for the "make defconfig" functionality, which has already
been merged, including a portion of 25113, so both of the following
can be closed off.

http://dpdk.org/dev/patchwork/patch/25112/
http://dpdk.org/dev/patchwork/patch/25113/



> This patch is dependent on RTE_TEMPLATE.
>
> One more nit: the indent is broken for T variable explanation.