All of lore.kernel.org
 help / color / mirror / Atom feed
From: Thomas Petazzoni via buildroot <buildroot@buildroot.org>
To: "Michael Nosthoff" <buildroot@heine.tech>
Cc: "Michael Nosthoff via buildroot" <buildroot@buildroot.org>
Subject: Re: [Buildroot] [PATCH] package/boost: remove dependencies on Boost.System
Date: Fri, 5 Sep 2025 08:37:42 +0200	[thread overview]
Message-ID: <20250905083742.3fada1ae@windsurf> (raw)
In-Reply-To: <35281-68b9ee00-5-70e30080@92065202>

On Thu, 04 Sep 2025 21:52:00 +0200
"Michael Nosthoff" <buildroot@heine.tech> wrote:

> > Thanks for the patch. It's great to have someone understanding the
> > dependencies between the different Boost modules, because they are not
> > obvious, and not easy to maintain, so thanks a lot.
> 
> The boost guys are currently working on better modularization support. So at least now each
> boost subproject has a dependency list in its build.jam file which helps a lot.
> e.g., https://github.com/boostorg/json/blob/develop/build.jam

Sounds very nice indeed to then have a better representation of those
dependencies in Buildroot.

> I checked all packages that depend on at least one of the boost modules
> where I removed the Boost.System dependency. Only two don't explicitly
> select BOOST_SYSTEM. libcpprestsdk nedded it so I fixed it. The other one
> is log4cxx which really doesn't depends on it in it's CMakeLists.

Great work! You could have mentioned this in the commit log.

> > Note that the fix of package/libcpprestsdk and other packages could be
> > done as a separate, preliminary patch.  
> 
> I will split them and resubmit.

Excellent, thanks!

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

      reply	other threads:[~2025-09-05  6:37 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-03 13:37 [Buildroot] [PATCH] package/boost: remove dependencies on Boost.System Michael Nosthoff via buildroot
2025-09-03 19:34 ` Thomas Petazzoni via buildroot
2025-09-04 19:52   ` Michael Nosthoff via buildroot
2025-09-05  6:37     ` Thomas Petazzoni via buildroot [this message]

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=20250905083742.3fada1ae@windsurf \
    --to=buildroot@buildroot.org \
    --cc=buildroot@heine.tech \
    --cc=thomas.petazzoni@bootlin.com \
    /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.