diff mbox

[1/3] package/gdb: add support for gdb 7.12

Message ID 20161214134058.29826-1-joerg.krause@embedded.rocks
State Accepted
Commit 4ef7a0b76dc62aeae71180af2d58bceac750e80b
Headers show

Commit Message

Jörg Krause Dec. 14, 2016, 1:40 p.m. UTC
gdb 7.12 was released on October 2016, it's time to include it in
Buildroot, which this commit does.

Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks>
---
 package/gdb/Config.in.host | 4 ++++
 package/gdb/gdb.hash       | 1 +
 2 files changed, 5 insertions(+)

Comments

Thomas Petazzoni Dec. 14, 2016, 2:20 p.m. UTC | #1
Hello,

On Wed, 14 Dec 2016 14:40:56 +0100, Jörg Krause wrote:
> gdb 7.12 was released on October 2016, it's time to include it in
> Buildroot, which this commit does.
> 
> Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks>

We have one patch on gdb 7.11.1. Is it still needed? I know I reported
the problem to Mike Frysinger, so he might have fixed it upstream, but
I haven't followed. It's worth double checing.

Thanks,

Thomas
Jörg Krause Dec. 14, 2016, 2:48 p.m. UTC | #2
Hi Thomas,

On Wed, 2016-12-14 at 15:20 +0100, Thomas Petazzoni wrote:
> Hello,
> 
> On Wed, 14 Dec 2016 14:40:56 +0100, Jörg Krause wrote:
> > gdb 7.12 was released on October 2016, it's time to include it in
> > Buildroot, which this commit does.
> > 
> > Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks>
> 
> We have one patch on gdb 7.11.1. Is it still needed? I know I
> reported
> the problem to Mike Frysinger, so he might have fixed it upstream,
> but
> I haven't followed. It's worth double checing.

The issue address by the patch 'Revert "sim: unify SIM_CPU definition"'
should be fixed by this commit [1].

I double checked by building gdb 7.12 without the mentioned patch for
the target bf512 with this [2] Buildroot toolchain configuration
successfully.

[1] https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=commitdi
ff;h=474a2d9f5f8a080e30812525729c3a8b7baa61d6
[2] http://autobuild.buildroot.org/toolchains/configs/br-bfin-full.conf
ig

Jörg
Thomas Petazzoni Dec. 14, 2016, 2:52 p.m. UTC | #3
Hello,

On Wed, 14 Dec 2016 15:48:09 +0100, Jörg Krause wrote:

> The issue address by the patch 'Revert "sim: unify SIM_CPU definition"'
> should be fixed by this commit [1].

Indeed, seems related.

> I double checked by building gdb 7.12 without the mentioned patch for
> the target bf512 with this [2] Buildroot toolchain configuration
> successfully.

OK. Note that just building gdb was not sufficient to trigger the
problem, you had to enable the GDB simulator (in host gdb), if I
remember correctly.

But that's fine, the patch you point at is clearly addressing the
original problem, so we're good.

Thanks a lot!

Thomas
Peter Korsgaard Dec. 15, 2016, 1:59 p.m. UTC | #4
>>>>> "Jörg" == Jörg Krause <joerg.krause@embedded.rocks> writes:

 > gdb 7.12 was released on October 2016, it's time to include it in
 > Buildroot, which this commit does.

 > Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks>

Committed all 3, thanks.
diff mbox

Patch

diff --git a/package/gdb/Config.in.host b/package/gdb/Config.in.host
index 4c8d902..2d4631e 100644
--- a/package/gdb/Config.in.host
+++ b/package/gdb/Config.in.host
@@ -48,6 +48,9 @@  choice
 	config BR2_GDB_VERSION_7_11
 		bool "gdb 7.11.x"
 
+	config BR2_GDB_VERSION_7_12
+		bool "gdb 7.12.x"
+
 endchoice
 
 endif
@@ -61,3 +64,4 @@  config BR2_GDB_VERSION
 	default "7.9.1"    if BR2_GDB_VERSION_7_9
 	default "7.10.1"   if BR2_GDB_VERSION_7_10 || !BR2_PACKAGE_HOST_GDB
 	default "7.11.1"   if BR2_GDB_VERSION_7_11
+	default "7.12"     if BR2_GDB_VERSION_7_12
diff --git a/package/gdb/gdb.hash b/package/gdb/gdb.hash
index c91373b..a44aaa3 100644
--- a/package/gdb/gdb.hash
+++ b/package/gdb/gdb.hash
@@ -2,6 +2,7 @@ 
 sha512	eebdf88b24e52e792b8a4b89ea85790de72b462a7810b44975fdf4232c068f353b15506071f450102a9d4bcecdde8e93dc3748a10699b7f73f3e04fb2d9d8414	gdb-7.9.1.tar.xz
 sha512	17a5138277a31685a5c2a841cb47ed9bc4626ea617b8ca77750513b300299f4fbbffe504958b5372de610dcb952c679cf8fa9c1bdadd380294fbf59b6e366010	gdb-7.10.1.tar.xz
 sha512  f80ec6c8a0f0b54c8b945666e875809174402b7e121efb378ebac931a91f9a1cc0048568f8e2f42ae8ae2392ff8d144c2e51d41c7398935017450aaf29838360	gdb-7.11.1.tar.xz
+sha512  f610338b3a24dfb387c295431d806a0c1375e8ce51b8dd8bbfd2efc4612d0c0ed40ad4393508afca0071eca70688f62bc4f199da5e75ad16f64d91c3bc046033	gdb-7.12.tar.xz
 
 # Locally calculated (fetched from Github)
 sha512  0a467091d4b01fbecabb4b8da1cb743025c70e7f4874a0b5c8fa2ec623569a39bde6762b91806de0be6e63711aeb6909715cfbe43860de73d8aec6159a9f10a7	gdb-6be65fb56ea6694a9260733a536a023a1e2d4d57.tar.gz