Re: [PATCH v6 1/8] distro/include: rework debug build

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Wed, 2025-12-17 at 23:24 +0800, Hongxu Jia wrote:
> The problem with DEBUG_BUILD is that we have a lot of debug information
> in builds anyway and this is now unclear what it means. The
> documentation.conf entry mentions packages which is now incorrect too.
> The variable is a very old one and the interface is poorly designed
> compared to other areas now.
> 
> This commit drop DEBUG_BUILD, and add debug_build.inc to instead.
> In bitbake.conf, use ??= to set *_OPTIMIZATION, in debug_build.inc to
> use ?= to override *_OPTIMIZATION
> 
> Make a config fragment to use include_all debug_build.inc to allow other
> layers to add their own debug build configurations, user should use the
> following ways to enable debug build
> 
>   $ bitbake-config-build enable-fragment core/yocto/debug-build
> 
> Or
> 
>   $ echo 'OE_FRAGMENTS += "core/yocto/debug-build"' >> conf/local.conf
> 
> Suggested-by: Peter Kjellerstedt <peter.kjellerstedt@axis.com>
> Suggested-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com>

Thanks, this series does look much better, I was really happy to see it
and I really wanted to merge it.

Unfortunately, Peter does have a point that this makes it hard to
change this on a per recipe basis and that is a valid use case. I think
we may need to work out how to address that.

I also still don't like the name "debug-build" when it really means
"build without compiler optimisations". I think we need to find a
better name too...

Cheers,

Richard