All of lore.kernel.org
 help / color / mirror / Atom feed
From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 5/6] package/collectd: add capabilities option
Date: Wed, 15 Apr 2020 08:17:09 +0200	[thread overview]
Message-ID: <CAPi7W83P0bQ9fogjOThP0LThBs8GgqJ_tPZZ0zMJybE87P=BgQ@mail.gmail.com> (raw)
In-Reply-To: <20200415074725.437cc6b6@windsurf.home>

Hi Thomas,

Le mer. 15 avr. 2020 ? 07:47, Thomas Petazzoni
<thomas.petazzoni@bootlin.com> a ?crit :
>
> On Tue, 14 Apr 2020 18:37:59 +0200
> Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote:
>
> > capabilities plugin is available since version 5.11.0 and
> > https://github.com/collectd/collectd/commit/b934f1572cdb54cf485b600546a72dbf9929ba2c
> >
> > It is currently enabled by default depending on jansson and
> > libmicrohttpd availability
> >
> > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> > ---
> >  package/collectd/Config.in   | 8 ++++++++
> >  package/collectd/collectd.mk | 2 ++
> >  2 files changed, 10 insertions(+)
> >
> > diff --git a/package/collectd/Config.in b/package/collectd/Config.in
> > index 7186d67c36..15c871025c 100644
> > --- a/package/collectd/Config.in
> > +++ b/package/collectd/Config.in
> > @@ -128,6 +128,14 @@ config BR2_PACKAGE_COLLECTD_BIND
> >       help
> >         Collects BIND DNS statistics.
> >
> > +config BR2_PACKAGE_COLLECTD_CAPABILITIES
> > +     bool "capabilities"
> > +     select BR2_PACKAGE_JANSSON
> > +     select BR2_PACKAGE_LIBMICROHTTPD
> > +     help
> > +       Collects selected static platform data using dmidecode and
> > +       expose it through micro embedded webserver.
>
> So it will use dmidecode, but it does not depend/enable dmidecode ?
Indeed, you're right I missed the runtime dependency on dmidecode,
I'll send a v3.
>
> Thanks!
>
> Thomas
> --
> Thomas Petazzoni, CTO, Bootlin
> Embedded Linux and Kernel engineering
> https://bootlin.com
Best Regards,

Fabrice

  reply	other threads:[~2020-04-15  6:17 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-14 16:37 [Buildroot] [PATCH 1/6] package/collectd: drop autoreconf Fabrice Fontaine
2020-04-14 16:37 ` [Buildroot] [PATCH 2/6] package/collectd: remove lvm Fabrice Fontaine
2020-04-14 16:37 ` [Buildroot] [PATCH 3/6] package/collectd: add DPDK_TELEMETRY option Fabrice Fontaine
2020-04-14 16:37 ` [Buildroot] [PATCH 4/6] package/collectd: add InfluxDB support Fabrice Fontaine
2020-04-14 16:37 ` [Buildroot] [PATCH 5/6] package/collectd: add capabilities option Fabrice Fontaine
2020-04-15  5:47   ` Thomas Petazzoni
2020-04-15  6:17     ` Fabrice Fontaine [this message]
2020-04-14 16:38 ` [Buildroot] [PATCH 6/6] package/collectd: add buddyinfo option Fabrice Fontaine
2020-04-14 16:51   ` Asaf Kahlon
2020-04-14 17:02     ` Fabrice Fontaine

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='CAPi7W83P0bQ9fogjOThP0LThBs8GgqJ_tPZZ0zMJybE87P=BgQ@mail.gmail.com' \
    --to=fontaine.fabrice@gmail.com \
    --cc=buildroot@busybox.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.