mbox series

[0/6] allow more DPDK libraries to be disabled on build

Message ID 20220113173918.2700651-1-bruce.richardson@intel.com (mailing list archive)
Headers show
Series allow more DPDK libraries to be disabled on build | expand

Message

Bruce Richardson Jan. 13, 2022, 5:39 p.m. UTC
A common request on-list has been to allow more of the DPDK build to be disabled by those who are
doing their own builds and only use a subset of the libraries. To this end, this patchset makes some
infrastructure changes [first two patches] to make it easier to have libraries disabled, and then
adds a six libraries to the "optional" list.

Bruce Richardson (6):
  lib: allow recursive disabling of libs in build
  app/test: link unit test binary against all available libs
  build: add node library to optional list
  build: add flow classification library to optional list
  build: add "packet framework" libs to optional list
  build: add cfgfile library to optional list

 app/test/meson.build | 74 ++++++++++++--------------------------------
 lib/meson.build      | 30 ++++++++++++------
 2 files changed, 40 insertions(+), 64 deletions(-)

--
2.32.0

Comments

Stephen Hemminger Jan. 13, 2022, 6:26 p.m. UTC | #1
On Thu, 13 Jan 2022 17:39:12 +0000
Bruce Richardson <bruce.richardson@intel.com> wrote:

> A common request on-list has been to allow more of the DPDK build to be disabled by those who are
> doing their own builds and only use a subset of the libraries. To this end, this patchset makes some
> infrastructure changes [first two patches] to make it easier to have libraries disabled, and then
> adds a six libraries to the "optional" list.
> 
> Bruce Richardson (6):
>   lib: allow recursive disabling of libs in build
>   app/test: link unit test binary against all available libs
>   build: add node library to optional list
>   build: add flow classification library to optional list
>   build: add "packet framework" libs to optional list
>   build: add cfgfile library to optional list
> 
>  app/test/meson.build | 74 ++++++++++++--------------------------------
>  lib/meson.build      | 30 ++++++++++++------
>  2 files changed, 40 insertions(+), 64 deletions(-)
> 
> --
> 2.32.0
> 

Acked-by: Stephen Hemminger <stephen@networkplumber.org>
Morten Brørup Jan. 14, 2022, 8:07 a.m. UTC | #2
> From: Bruce Richardson [mailto:bruce.richardson@intel.com]
> Sent: Thursday, 13 January 2022 18.39
> 
> A common request on-list has been to allow more of the DPDK build to be
> disabled by those who are
> doing their own builds and only use a subset of the libraries. To this
> end, this patchset makes some
> infrastructure changes [first two patches] to make it easier to have
> libraries disabled, and then
> adds a six libraries to the "optional" list.
> 
> Bruce Richardson (6):
>   lib: allow recursive disabling of libs in build
>   app/test: link unit test binary against all available libs
>   build: add node library to optional list
>   build: add flow classification library to optional list
>   build: add "packet framework" libs to optional list
>   build: add cfgfile library to optional list
> 
>  app/test/meson.build | 74 ++++++++++++--------------------------------
>  lib/meson.build      | 30 ++++++++++++------
>  2 files changed, 40 insertions(+), 64 deletions(-)
> 
> --
> 2.32.0
> 

Thank you very much, Bruce!

Acked-by: Morten Brørup <mb@smartsharesystems.com>
David Marchand Jan. 19, 2022, 4:52 p.m. UTC | #3
On Thu, Jan 13, 2022 at 6:40 PM Bruce Richardson
<bruce.richardson@intel.com> wrote:
>
> A common request on-list has been to allow more of the DPDK build to be disabled by those who are
> doing their own builds and only use a subset of the libraries. To this end, this patchset makes some
> infrastructure changes [first two patches] to make it easier to have libraries disabled, and then
> adds a six libraries to the "optional" list.
>
> Bruce Richardson (6):
>   lib: allow recursive disabling of libs in build
>   app/test: link unit test binary against all available libs
>   build: add node library to optional list
>   build: add flow classification library to optional list
>   build: add "packet framework" libs to optional list
>   build: add cfgfile library to optional list
>
>  app/test/meson.build | 74 ++++++++++++--------------------------------
>  lib/meson.build      | 30 ++++++++++++------
>  2 files changed, 40 insertions(+), 64 deletions(-)

Except an issue in patch 2, this series looks good to me.
Thanks Bruce.