Re: [PATCH v3 0/6] Update libs build process

["Gonzalez Monroy, Sergio" <sergio.gonzalez.monroy-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>]

Are there any more comments on this patch set?

Thanks,
Sergio


> -----Original Message-----
> From: dev [mailto:dev-bounces-VfR2kkLFssw@public.gmane.org] On Behalf Of Sergio Gonzalez
> Monroy
> Sent: Thursday, October 9, 2014 2:05 PM
> To: dev-VfR2kkLFssw@public.gmane.org
> Subject: [dpdk-dev] [PATCH v3 0/6] Update libs build process
> 
> As per the proposal, this patch set does:
>  - Remove CONFIG_RTE_BUILD_COMBINE_LIBS as a configuration option.
>  - For static library, build a single/combined library.
>  - For shared libraries, build both individual/separated and single/combined
>    libraries.
>  - Link apps only against single/combined libs.
>  - Include external shared libs dependencies when building shared libraries.
> 
> v3:
>  - Split some of the patches for easier review
>  - Improve patches descriptions
> 
> Sergio Gonzalez Monroy (6):
>   Link combined shared library using CC
>   Link apps only against single/combined library
>   Remove CONFIG_RTE_BUILD_COMBINE_LIBS and related
>   Update library build process
>   Avoid duplicated code
>   Link apps/DSOs against EXECENV_LDLIBS with --as-needed
> 
>  config/common_bsdapp   |   3 +-
>  config/common_linuxapp |   3 +-
>  mk/rte.app.mk          | 164 ++-----------------------------------------------
>  mk/rte.lib.mk          |  81 ++++++------------------
>  mk/rte.sdkbuild.mk     |   2 +-
>  mk/rte.sharelib.mk     |  54 ++++++++--------
>  mk/rte.vars.mk         |   4 --
>  7 files changed, 54 insertions(+), 257 deletions(-)
> 
> --
> 1.9.3