[dpdk-dev] [PATCH v2 3/4] Update library build process

2014-10-09 Thread Sergio Gonzalez Monroy
On Wed, Oct 08, 2014 at 03:36:55PM -0700, Matthew Hall wrote: > On Tue, Oct 07, 2014 at 10:55:11AM +0100, Sergio Gonzalez Monroy wrote: > > I don't see how this particular patch would force people to change their > > code, > > though in all likelihood they will have to as a result of ABI changes i

[dpdk-dev] [PATCH v2 3/4] Update library build process

2014-10-08 Thread Thomas Monjalon
Hi Sergio, 2014-10-06 11:52, Sergio Gonzalez Monroy: > Remove COMBINE_LIBS option and by default build: > - CONFIG_RTE_BUILD_SHARED_LIB=y : both individual and combined libraries > - CONFIG_RTE_BUILD_SHARED_LIB=n : single combined library > > Signed-off-by: Sergio Gonzalez Monroy I'd appreciate

[dpdk-dev] [PATCH v2 3/4] Update library build process

2014-10-08 Thread Matthew Hall
On Tue, Oct 07, 2014 at 10:55:11AM +0100, Sergio Gonzalez Monroy wrote: > I don't see how this particular patch would force people to change their code, > though in all likelihood they will have to as a result of ABI changes in the > following release. > > The only difference now would be when they

[dpdk-dev] [PATCH v2 3/4] Update library build process

2014-10-07 Thread Sergio Gonzalez Monroy
On Mon, Oct 06, 2014 at 01:46:07PM -0700, Matthew Hall wrote: > On Mon, Oct 06, 2014 at 11:52:34AM +0100, Sergio Gonzalez Monroy wrote: > > Remove COMBINE_LIBS option and by default build: > > - CONFIG_RTE_BUILD_SHARED_LIB=y : both individual and combined libraries > > - CONFIG_RTE_BUILD_SHARED_LIB

[dpdk-dev] [PATCH v2 3/4] Update library build process

2014-10-06 Thread Matthew Hall
On Mon, Oct 06, 2014 at 11:52:34AM +0100, Sergio Gonzalez Monroy wrote: > Remove COMBINE_LIBS option and by default build: > - CONFIG_RTE_BUILD_SHARED_LIB=y : both individual and combined libraries > - CONFIG_RTE_BUILD_SHARED_LIB=n : single combined library As previously discussed.,It would be bet

[dpdk-dev] [PATCH v2 3/4] Update library build process

2014-10-06 Thread Sergio Gonzalez Monroy
Remove COMBINE_LIBS option and by default build: - CONFIG_RTE_BUILD_SHARED_LIB=y : both individual and combined libraries - CONFIG_RTE_BUILD_SHARED_LIB=n : single combined library Signed-off-by: Sergio Gonzalez Monroy --- config/common_bsdapp | 3 +-- config/common_linuxapp | 3 +-- mk/rte.l