From: Luca Boccassi <bluca@debian.org>
To: Bruce Richardson <bruce.richardson@intel.com>, dev@dpdk.org
Cc: thomas@monjalon.net, david.marchand@redhat.com, ktraynor@redhat.com
Subject: Re: [dpdk-dev] [PATCH v2 0/7] improve DPDK static builds with meson
Date: Fri, 01 May 2020 15:02:56 +0100 [thread overview]
Message-ID: <e0cc76aea4e5f4b3fa70c6dcecf2a605c728cd84.camel@debian.org> (raw)
In-Reply-To: <20200501135401.28124-1-bruce.richardson@intel.com>
On Fri, 2020-05-01 at 14:53 +0100, Bruce Richardson wrote:
> This set fixes a number of minor issues with static builds when using
> meson, both for linking apps/examples as part of a meson build itself or
> when using pkg-config subsequently.
>
> Following this patchset, all DPDK static builds should be linking with
> --whole-archive to ensure all lib and driver constructors are included,
> and the use of pkg-config for doing static builds is simplified. The
> downside is that for correctness we need two .pc files for DPDK rather
> than just one.
>
> v2: improved log messages for a number of patches, and clearly marked
> internal-only pkg-config file as such
>
> Bruce Richardson (7):
> build: always link-whole DPDK static libraries
> build: remove unnecessary variable
> devtools/test-meson-builds.sh: add pkg-config static builds
> build: move pkg-config creation to separate file
> build/pkg-config: output driver libs first for static build
> build/pkg-config: improve static linking flags
> build/pkg-config: prevent overlinking
>
> app/meson.build | 2 +-
> app/test/meson.build | 2 +-
> buildtools/pkg-config/meson.build | 43 +++++++++++++++++++
> .../pkg-config/set-static-linker-flags.py | 38 ++++++++++++++++
> devtools/test-meson-builds.sh | 2 +-
> doc/guides/prog_guide/build-sdk-meson.rst | 2 +-
> drivers/meson.build | 2 +-
> examples/bbdev_app/Makefile | 2 +-
> examples/bond/Makefile | 2 +-
> examples/cmdline/Makefile | 2 +-
> examples/distributor/Makefile | 2 +-
> examples/eventdev_pipeline/Makefile | 2 +-
> examples/fips_validation/Makefile | 2 +-
> examples/flow_classify/Makefile | 2 +-
> examples/flow_filtering/Makefile | 2 +-
> examples/helloworld/Makefile | 2 +-
> examples/ioat/Makefile | 2 +-
> examples/ip_fragmentation/Makefile | 2 +-
> examples/ip_pipeline/Makefile | 2 +-
> examples/ip_reassembly/Makefile | 2 +-
> examples/ipsec-secgw/Makefile | 2 +-
> examples/ipv4_multicast/Makefile | 2 +-
> examples/kni/Makefile | 2 +-
> examples/l2fwd-cat/Makefile | 2 +-
> examples/l2fwd-crypto/Makefile | 2 +-
> examples/l2fwd-event/Makefile | 2 +-
> examples/l2fwd-jobstats/Makefile | 2 +-
> examples/l2fwd-keepalive/Makefile | 2 +-
> examples/l2fwd/Makefile | 2 +-
> examples/l3fwd-acl/Makefile | 2 +-
> examples/l3fwd-power/Makefile | 2 +-
> examples/l3fwd/Makefile | 2 +-
> examples/link_status_interrupt/Makefile | 2 +-
> examples/meson.build | 6 +--
> examples/ntb/Makefile | 2 +-
> examples/packet_ordering/Makefile | 2 +-
> examples/ptpclient/Makefile | 2 +-
> examples/qos_meter/Makefile | 2 +-
> examples/qos_sched/Makefile | 2 +-
> examples/rxtx_callbacks/Makefile | 2 +-
> examples/service_cores/Makefile | 2 +-
> examples/skeleton/Makefile | 2 +-
> examples/tep_termination/Makefile | 2 +-
> examples/timer/Makefile | 2 +-
> examples/vdpa/Makefile | 2 +-
> examples/vhost/Makefile | 2 +-
> examples/vhost_blk/Makefile | 2 +-
> examples/vhost_crypto/Makefile | 2 +-
> examples/vmdq/Makefile | 2 +-
> examples/vmdq_dcb/Makefile | 2 +-
> lib/librte_telemetry/meson.build | 1 -
> lib/meson.build | 2 +-
> meson.build | 26 +----------
> 53 files changed, 134 insertions(+), 76 deletions(-)
> create mode 100644 buildtools/pkg-config/meson.build
> create mode 100644 buildtools/pkg-config/set-static-linker-flags.py
Series-acked-by: Luca Boccassi <bluca@debian.org>
--
Kind regards,
Luca Boccassi
next prev parent reply other threads:[~2020-05-01 14:03 UTC|newest]
Thread overview: 63+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-04-29 10:08 [dpdk-dev] [PATCH " Bruce Richardson
2020-04-29 10:08 ` [dpdk-dev] [PATCH 1/7] build: always link-whole DPDK static libraries Bruce Richardson
2020-04-29 10:18 ` Thomas Monjalon
2020-04-29 10:42 ` Bruce Richardson
2020-04-29 10:49 ` Thomas Monjalon
2020-04-29 14:04 ` Andrzej Ostruszka [C]
2020-04-29 10:08 ` [dpdk-dev] [PATCH 2/7] build: remove unnecessary variable Bruce Richardson
2020-04-29 10:19 ` Thomas Monjalon
2020-04-29 10:29 ` Bruce Richardson
2020-04-29 10:08 ` [dpdk-dev] [PATCH 3/7] devtools/test-meson-builds.sh: add pkg-config static builds Bruce Richardson
2020-04-29 10:08 ` [dpdk-dev] [PATCH 4/7] build: move pkg-config creation to separate file Bruce Richardson
2020-04-29 10:08 ` [dpdk-dev] [PATCH 5/7] build/pkg-config: output driver libs first for static build Bruce Richardson
2020-04-29 10:30 ` Thomas Monjalon
2020-04-29 10:54 ` Bruce Richardson
2020-04-29 10:40 ` Luca Boccassi
2020-04-29 11:03 ` Bruce Richardson
2020-04-29 11:12 ` Luca Boccassi
2020-04-29 10:08 ` [dpdk-dev] [PATCH 6/7] build/pkg-config: improve static linking flags Bruce Richardson
2020-04-29 10:32 ` Thomas Monjalon
2020-04-29 10:56 ` Bruce Richardson
2020-04-29 10:57 ` Thomas Monjalon
2020-04-29 10:37 ` Luca Boccassi
2020-04-29 10:58 ` Bruce Richardson
2020-04-29 11:10 ` Luca Boccassi
2020-04-29 10:08 ` [dpdk-dev] [PATCH 7/7] build/pkg-config: prevent overlinking Bruce Richardson
2020-04-29 10:34 ` Thomas Monjalon
2020-04-29 11:00 ` Bruce Richardson
2020-05-01 13:53 ` [dpdk-dev] [PATCH v2 0/7] improve DPDK static builds with meson Bruce Richardson
2020-05-01 13:53 ` [dpdk-dev] [PATCH v2 1/7] build: always link-whole DPDK static libraries Bruce Richardson
2020-05-01 13:53 ` [dpdk-dev] [PATCH v2 2/7] build: remove unnecessary variable Bruce Richardson
2020-05-01 13:53 ` [dpdk-dev] [PATCH v2 3/7] devtools/test-meson-builds.sh: add pkg-config static builds Bruce Richardson
2020-05-01 13:53 ` [dpdk-dev] [PATCH v2 4/7] build: move pkg-config creation to separate file Bruce Richardson
2020-05-01 13:53 ` [dpdk-dev] [PATCH v2 5/7] build/pkg-config: output driver libs first for static build Bruce Richardson
2020-05-01 13:54 ` [dpdk-dev] [PATCH v2 6/7] build/pkg-config: improve static linking flags Bruce Richardson
2020-05-01 13:54 ` [dpdk-dev] [PATCH v2 7/7] build/pkg-config: prevent overlinking Bruce Richardson
2020-05-01 14:02 ` Luca Boccassi [this message]
2020-06-30 13:25 ` [dpdk-dev] [PATCH v2 0/7] improve DPDK static builds with meson Pai G, Sunil
2020-06-30 16:37 ` Stokes, Ian
2020-07-01 17:29 ` Thomas Monjalon
2020-07-02 10:46 ` Stokes, Ian
2020-06-30 14:14 ` [dpdk-dev] [PATCH v3 " Bruce Richardson
2020-06-30 14:14 ` [dpdk-dev] [PATCH v3 1/7] build: always link-whole DPDK static libraries Bruce Richardson
2020-07-01 14:19 ` Thomas Monjalon
2020-06-30 14:14 ` [dpdk-dev] [PATCH v3 2/7] build: remove unnecessary variable Bruce Richardson
2020-06-30 14:14 ` [dpdk-dev] [PATCH v3 3/7] devtools/test-meson-builds.sh: add pkg-config static builds Bruce Richardson
2020-06-30 14:14 ` [dpdk-dev] [PATCH v3 4/7] build: move pkg-config creation to separate file Bruce Richardson
2020-06-30 14:14 ` [dpdk-dev] [PATCH v3 5/7] build/pkg-config: output driver libs first for static build Bruce Richardson
2020-07-01 7:50 ` Thomas Monjalon
2020-07-01 8:43 ` Bruce Richardson
2020-07-01 14:42 ` Thomas Monjalon
2020-07-01 15:16 ` Bruce Richardson
2020-07-01 15:36 ` Thomas Monjalon
2020-07-01 15:45 ` Bruce Richardson
2020-07-01 16:04 ` Thomas Monjalon
2020-06-30 14:14 ` [dpdk-dev] [PATCH v3 6/7] build/pkg-config: improve static linking flags Bruce Richardson
2020-07-01 14:30 ` Thomas Monjalon
2020-07-01 14:33 ` Thomas Monjalon
2020-07-01 14:36 ` Bruce Richardson
2020-07-01 14:45 ` Thomas Monjalon
2020-06-30 14:14 ` [dpdk-dev] [PATCH v3 7/7] build/pkg-config: prevent overlinking Bruce Richardson
2020-07-01 7:56 ` Thomas Monjalon
2020-07-01 8:58 ` Bruce Richardson
2020-07-01 17:33 ` [dpdk-dev] [PATCH v3 0/7] improve DPDK static builds with meson Thomas Monjalon
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=e0cc76aea4e5f4b3fa70c6dcecf2a605c728cd84.camel@debian.org \
--to=bluca@debian.org \
--cc=bruce.richardson@intel.com \
--cc=david.marchand@redhat.com \
--cc=dev@dpdk.org \
--cc=ktraynor@redhat.com \
--cc=thomas@monjalon.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).