[dpdk-dev] [PATCH v3 1/3] mk: add sensible default target with defconfig
Hunt, David
david.hunt at intel.com
Fri Aug 4 10:22:20 CEST 2017
-----Original Message-----
From: Thomas Monjalon [mailto:thomas at monjalon.net]
Sent: Thursday, 3 August, 2017 11:40 PM
To: Hunt, David <david.hunt at intel.com>
Cc: dev at dpdk.org; shreyansh.jain at nxp.com
Subject: Re: [dpdk-dev] [PATCH v3 1/3] mk: add sensible default target with defconfig
07/06/2017 16:37, David Hunt:
> Users can now use 'make defconfig' to generate a configuration using
> the most appropriate defaults for the current machine.
>
> <arch-machine-execenv-toolchain>
> arch taken from uname -m
> machine defaults to native
> execenv is taken from uname, Linux=linuxapp, otherwise bsdapp
> toolchain is taken from $CC -v to see which compiler to use
>
> Signed-off-by: David Hunt <david.hunt at intel.com>
> Acked-by: Shreyansh Jain <shreyansh.jain at nxp.com>
Looks to be a good idea if it is really automatic.
> + ${CC} -v 2>&1 | \
> + grep " version " | cut -d ' ' -f 1)
Unfortunately, it depends on $CC which is not commonly exported.
What about defaulting to gcc?
> - @echo "Configuration done"
> + @echo "Configuration done using "$(shell basename \
> + $(RTE_CONFIG_TEMPLATE) | sed "s/defconfig_//g")
RTE_CONFIG_TEMPLATE is not defined in this patch (and I do not see the benefit in next patch).
Thomas,
Does this mean that this patch is not going into this release? It has been acked for almost a month now, with no further comment. The one hour between your comment and the release of RC4 did not give me a reasonable amount of time to address your concerns. I also feel that the lack of comments in the last month should mean that the patch should be applied as is. If changes are required, I am happy to address in the next release.
Regards,
Dave.
More information about the dev
mailing list