[Buildroot] [PATCH 1/4] gdb: use consistent version on host and target for microblaze and arc
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Thu Jan 9 23:35:46 UTC 2014
Dear Spenser Gilliland,
On Wed, 8 Jan 2014 10:33:02 -0600, Spenser Gilliland wrote:
> Currently, the version of gdb used on the host and the target are different for
> microblaze and arc. This is fixed by using a different method to specify the
> version.
>
> This fixes autobuild failure
>
> http://autobuild.buildroot.net/results/554/5547456a42cdd21892bba665a15d96cdbb5b1b26//build-end.log
> Signed-off-by: Spenser Gilliland <spenser at gillilanding.com>
Missing empty line before the Signed-off-by line. Also, some comment
below.
> diff --git a/package/gdb/Config.in.host b/package/gdb/Config.in.host
> index de0e0be..f446384 100644
> --- a/package/gdb/Config.in.host
> +++ b/package/gdb/Config.in.host
> @@ -59,7 +59,5 @@ config BR2_GDB_VERSION
> default "7.3.1" if BR2_GDB_VERSION_7_3
> default "7.4.1" if BR2_GDB_VERSION_7_4
> default "7.5.1" if BR2_GDB_VERSION_7_5
> - default "f25a1952afd054205f9471e449c1f7ca5b271b7c" if BR2_arc
> - default "6be65fb56ea6694a9260733a536a023a1e2d4d57" if BR2_microblaze
>
> endif
> diff --git a/package/gdb/gdb.mk b/package/gdb/gdb.mk
> index d06b61b..94d8302 100644
> --- a/package/gdb/gdb.mk
> +++ b/package/gdb/gdb.mk
> @@ -16,6 +16,10 @@ ifeq ($(BR2_bfin),y)
> GDB_VERSION = 6.6a
> else ifeq ($(BR2_avr32),y)
> GDB_VERSION = 6.7.1-avr32-2.1.5
> +else ifeq ($(BR2_microblaze),y)
> +GDB_VERSION = 0d0b89a88104785cee93d27074e15118bf14fac1
> +else ifeq ($(BR2_arc),y)
> +GDB_VERSION = f25a1952afd054205f9471e449c1f7ca5b271b7c
> else
> GDB_VERSION = 7.5.1
> endif
I am not entirely happy with this change, because it handles
Microblaze/ARC in a different way than AVR32, while they are really the
same case.
And also because the code you're touching in gdb.mk has the following
comment on top of it:
# When no version is defined, it means that cross-gdb for the host has
# not been enabled, and we will only build gdbserver or gdb for the
# target. In this case, use the latest available version
# automatically.
However, this comment is no longer true: with your patch, for certain
architectures (Microblaze/ARC), we are *never* setting BR2_GDB_VERSION,
and therefore this piece of gdb.mk code is also used when cross-gdb is
enabled.
Therefore maybe we should:
1/ Remove the version string for AVR32 in the definition of
BR2_GDB_VERSION, like you did for Microblaze and ARC
2/ Update the comment above the code you're changing in gdb.mk to
explain what's going on.
3/ Add a comment on top of BR2_GDB_VERSION that explains why we don't
define a version string for some of the gdb versions.
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
More information about the buildroot
mailing list