Re: [Buildroot] [PATCH 0/7] Add package/python-memray and dependencies

[Thomas Petazzoni via buildroot <buildroot@buildroot.org>]

Hello Fiona,

On Sun, 10 Aug 2025 00:32:22 +0200
Fiona Klute <fiona.klute@gmx.de> wrote:

> This series adds the Memray memory profiler for Python [1] and its
> dependencies. I needed it to help reduce memory usage of some custom
> tools, and figure it might be useful to others for
> debugging/optimization, too.
> 
> The individual patches are fairly simple, just a bunch of new Python
> packages. Memray itself includes a binary module that needs to be
> cross-compiled.

Thanks for the patch series! Since it's mostly made if Python packages,
it would be nice to have some tests in support/testing/ for those
packages, or at least for the "top-level" Memray package.

> One thing I'm not sure about is whether there should be a separate
> option to enable building libdebuginfod in package/elfutils, instead
> of enabling it if the dependencies are met. Having an option would
> allow people who need package/json-c and package/libcurl, but don't
> build anything that links against libdebuginfod to save a few kB (in
> my arm64 build /usr/lib/libdebuginfod-0.192.so is about 38 kB), at the
> price of slightly more complexity in the config system. I'm happy to
> add the option if that's preferred.

In this particular case, I believe I would create an explicit
sub-option. Not because 38 KiB matters, but because it's rather
impossible to "guess" that json-c and libcurl are needed to get
libdebuginfod to be built.

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