Re: [Buildroot] [PATCH 1/3] package/gdb: add support for GDB 14.1

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

Hello,

On Wed, 27 Dec 2023 22:40:35 +0100
"Yann E. MORIN" <yann.morin.1998@free.fr> wrote:

> > +	select BR2_PACKAGE_MPFR if BR2_GDB_VERSION_14  
> 
> This is a bit fragile: when we add gdb 15, and drop gdb 12, gdb 14 will
> become the default version. Then, if the host variant is not selected,
> BR2_GDB_VERSION_14 will not be set. If the user enables the full gdb n
> target, mpfr will not be selected.

Yes, it requires some care. We had things like that in the past (gmp
became a mandatory dependency), and we were able to make it work, but I
agree it requires being careful when doing the version bumps.

> I've added a litle blurb in the config.in.host, pointing to this part of
> the code for an appropriate update.

Thanks!

> Alternatively, I wonder if we shouldn't jsut make mpfr unconditional for
> all versions of gdb, now.

No strong opinion on this: whenever possible I try to not build
unnecessary things, hence the proposed solution. But admittedly, mpfr
is becoming a mandatory dependency for gdb, so it means that anyway in
the not too distant future, everyone will have to build mpfr as a
pre-requisite of building gdb.

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