From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Petazzoni Date: Mon, 17 Aug 2020 12:18:34 +0200 Subject: [Buildroot] [PATCH v3, 3/6] package/collectd: add DPDK_TELEMETRY option In-Reply-To: <20200415063008.2237470-3-fontaine.fabrice@gmail.com> References: <20200415063008.2237470-1-fontaine.fabrice@gmail.com> <20200415063008.2237470-3-fontaine.fabrice@gmail.com> Message-ID: <20200817121834.3054906a@windsurf.home> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net On Wed, 15 Apr 2020 08:30:05 +0200 Fabrice Fontaine wrote: > DPDK support is available since version 5.11.0 and > https://github.com/collectd/collectd/commit/00cbd4d2c016a676db476ae66bbc58d6bf11a6a0 > > It depends on jansson and is currently enabled by default raising the > following build failure on musl: > > src/dpdk_telemetry.c:43:10: fatal error: sys/unistd.h: No such file or directory > #include > ^~~~~~~~~~~~~~ > > Fixes: > - autobuild.buildroot.org/results/aafb8c72f147fefc7a988c45e4dc17de48b07a95 > > Signed-off-by: Fabrice Fontaine > --- > ...c-dpdk_telemetry.c-fix-build-on-musl.patch | 37 +++++++++++++++++++ > package/collectd/Config.in | 7 ++++ > package/collectd/collectd.mk | 5 +++ > 3 files changed, 49 insertions(+) > create mode 100644 package/collectd/0001-src-dpdk_telemetry.c-fix-build-on-musl.patch Even though I dislike how the options/dependencies are handled in collectd.mk, this particular patch is not really the culprit, and it fixes a build issue, so I've applied. Thomas -- Thomas Petazzoni, CTO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com