Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Julien Olivain via buildroot <buildroot@buildroot.org>
To: Andrea Ricchi <andrea.ricchi@amarulasolutions.com>
Cc: buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH v3] package/libcppconnman: add new package
Date: Tue, 21 Jul 2026 21:59:42 +0200	[thread overview]
Message-ID: <ddab76e125902757f19b380a6cf1b3ac@free.fr> (raw)
In-Reply-To: <CABNF3=Z2VPCDu6Od07=qNV7V=_xFSp-Jc1m294MP+AnKsNZpwg@mail.gmail.com>

Hi Andrea,

On 21/07/2026 09:06, Andrea Ricchi wrote:
> Hi Julien,
> 
> Thanks again for the review, and sorry about the pipeline issue.
> 
> I tried running ./utils/test-pkg -a -p libcppconnman locally and got
> bootlin-sparc-uclibc [24/32]: OK. In my .config, I have the following
> set:
> 
> BR2_PACKAGE_LIBCPPCONNMAN=y
> BR2_PACKAGE_LIBCPPCONNMAN_EXAMPLE=y
> 
> Do you have any idea why it works on my machine but fails in the
> GitLab pipeline? Could it be due to some specific build environment
> configuration?

The host environment should not have any impact for this kind of issue.

Are you using the correct command and are you looking at
the right config file?

For enabling examples in a test-pkg run, you should use
commands like:

cat >my.config <<EOF
BR2_PACKAGE_LIBCPPCONNMAN=y
BR2_PACKAGE_LIBCPPCONNMAN_EXAMPLE=y
EOF
utils/test-pkg -a -p libcppconnman -c my.config

The generated config of a failing builds should be in:
"~/br-test-pkg/bootlin-microblazeel-uclibc/.config"
rather than in "./.config".

> Thanks!
> 
> Andrea Ricchi
> Embedded Software Engineer
> M. +39 333 775 4973
> andrea.ricchi@amarulasolutions.com
> ―――――――――――――――
> Amarula Solutions SRL
>  Via Felice Cavallotti 25D, 41012 Carpi, MO, IT
> info@amarulasolutions.com
> www.amarulasolutions.com [2]
> 
> On Mon, Jul 20, 2026 at 9:22 PM Julien Olivain <ju.o@free.fr> wrote:
> 
>> Hi Andrea,
>> 
>> Thanks for the updated patch!
>> 
>> On 20/07/2026 14:22, Andrea Ricchi wrote:
>>> libcppconnman is a C++ library that provides an easy-to-use API
>>> to communicate with ConnMan over D-Bus using GDBus.
>>> 
>>> https://github.com/amarula/libcppconnman
>>> 
>>> Also add myself to DEVELOPERS for this package.
>>> 
>>> Signed-off-by: Andrea Ricchi <andrea.ricchi@amarulasolutions.com>
>>> ---
>>> Changes v2 -> v3:
>>> - Config.in: add BR2_TOOLCHAIN_GCC_AT_LEAST_10 dependency (C++20
>> /
>>> <ranges>),
>>> fixing build failures on GCC < 10 toolchains reported by the
>>> autobuilders
>>> - Config.in, libcppconnman.mk [1]: select and depend on readline
>> for the
>>> example,
>>> fixing the missing readline library configure error when
>> examples
>>> are enabled
>>> 
>>> Changes v1 -> v2:
>>> - add missing libcppconnman entry in package/Config.in
>>> - merge the DEVELOPERS addition into this patch
>>> 
>>> DEVELOPERS                               |  1 +
>>> package/Config.in                        |  1 +
>>> package/libcppconnman/Config.in          | 28
>> ++++++++++++++++++++++++
>>> package/libcppconnman/libcppconnman.hash |  3 +++
>>> package/libcppconnman/libcppconnman.mk [1]   | 25
>> +++++++++++++++++++++
>>> 5 files changed, 58 insertions(+)
>>> create mode 100644 package/libcppconnman/Config.in
>>> create mode 100644 package/libcppconnman/libcppconnman.hash
>>> create mode 100644 package/libcppconnman/libcppconnman.mk [1]
>>> 
>> [...]
>>> diff --git a/package/libcppconnman/Config.in
>>> b/package/libcppconnman/Config.in
>>> new file mode 100644
>>> index 0000000000..d99d711e25
>>> --- /dev/null
>>> +++ b/package/libcppconnman/Config.in
>>> @@ -0,0 +1,28 @@
>>> +config BR2_PACKAGE_LIBCPPCONNMAN
>>> +     bool "libcppconnman"
>>> +     depends on BR2_INSTALL_LIBSTDCPP
>>> +     depends on BR2_TOOLCHAIN_GCC_AT_LEAST_10 # C++20
>>> +     depends on BR2_USE_WCHAR # libglib2
>>> +     depends on BR2_TOOLCHAIN_HAS_THREADS # libglib2
>>> +     depends on BR2_USE_MMU # libglib2
>>> +     select BR2_PACKAGE_LIBGLIB2
>>> +     help
>>> +       libcppconnman provide easy to use C++ API to communicate
>>> +       with ConnMan via the DBus using GDBus.
>>> +
>>> +       https://amarula.github.io/libcppconnman/
>>> +
>>> +comment "libcppconnman needs a toolchain w/ C++20, wchar,
>> threads"
>>> +     depends on BR2_USE_MMU
>>> +     depends on !BR2_INSTALL_LIBSTDCPP || !BR2_USE_WCHAR || \
>>> +             !BR2_TOOLCHAIN_HAS_THREADS ||
>> !BR2_TOOLCHAIN_GCC_AT_LEAST_10
>>> +
>>> +if BR2_PACKAGE_LIBCPPCONNMAN
>>> +
>>> +config BR2_PACKAGE_LIBCPPCONNMAN_EXAMPLE
>>> +     bool "libcppconnman example"
>> 
>> It seems there is still 2 build failures with the examples. See:
>> https://gitlab.com/jolivain/buildroot/-/pipelines/2691451879
>> 
>> It is likely due to a missing dependency on
>> BR2_TOOLCHAIN_HAS_SYNC_1.
>> 
>> Could you have a look please?
>> 
>>> +     select BR2_PACKAGE_READLINE
>>> +     help
>>> +       Compile CLI example.
>>> +
>>> +endif
>> 
>> Best regards,
>> 
>> Julien.
> 
> 
> Links:
> ------
> [1] http://libcppconnman.mk
> [2] http://www.amarulasolutions.com/

Best regards,

Julien.
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

  reply	other threads:[~2026-07-21 19:59 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-07-20 12:22 [Buildroot] [PATCH v3] package/libcppconnman: add new package Andrea Ricchi
2026-07-20 19:22 ` Julien Olivain via buildroot
2026-07-21  7:06   ` Andrea Ricchi
2026-07-21 19:59     ` Julien Olivain via buildroot [this message]
2026-07-21 20:47       ` Andrea Ricchi

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=ddab76e125902757f19b380a6cf1b3ac@free.fr \
    --to=buildroot@buildroot.org \
    --cc=andrea.ricchi@amarulasolutions.com \
    --cc=ju.o@free.fr \
    /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