Re: [Buildroot] [PATCH v6 03/10] package/newlib-bare-metal: new package

From: Thomas Petazzoni via buildroot <buildroot@buildroot.org>
To: "Frager, Neal" <neal.frager@amd.com>
Cc: "Simek, Michal" <michal.simek@amd.com>,
	"Erkiaga Elorza, Ibai" <ibai.erkiaga-elorza@amd.com>,
	"luca.ceresoli@bootlin.com" <luca.ceresoli@bootlin.com>,
	Neal Frager via buildroot <buildroot@buildroot.org>
Subject: Re: [Buildroot] [PATCH v6 03/10] package/newlib-bare-metal: new package
Date: Wed, 1 Nov 2023 12:50:48 +0100	[thread overview]
Message-ID: <20231101125048.01553f76@windsurf> (raw)
In-Reply-To: <CH2PR12MB5004C8F27820E2EC98FC5232F0A7A@CH2PR12MB5004.namprd12.prod.outlook.com>

Hello,

On Wed, 1 Nov 2023 11:43:34 +0000
"Frager, Neal" <neal.frager@amd.com> wrote:

> Are you sure that the Config.in files can be dropped?  
> 
> When I drop them, I am stuck with the following build error:
> Makefile:576: *** newlib-bare-metal is in the dependency chain of toolchain-bare-metal-buildroot that has added it to its _DEPENDENCIES variable without selecting it or depending on it from Config.in.  Stop.
> 
> How can we deal with this without selecting the packages and having Config.in files?

Ah, yes, I asked to change the newlib-bare-metal package to be a target
package. In this case indeed you do need a Config.in file. However, its
option doesn't have to be visible, it should simply be selected by the
toolchain-bare-metal-buildroot package.

config BR2_PACKAGE_NEWLIB_BARE_METAL
	bool

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