Message ID | 20180624135310.16514-4-thomas.petazzoni@bootlin.com |
---|---|
State | Accepted |
Commit | 8f3d8623145505f751c2235a4ae40705cff6e05d |
Headers | show |
Series | Add support for gdb 8.1 | expand |
>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@bootlin.com> writes: > From: Romain Naour <romain.naour@gmail.com> > 8.1 is around, 8.0 has already seen a point release, so it's time to > make 8.0 the default version for gdb. > Signed-off-by: Romain Naour <romain.naour@gmail.com> > Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com> > --- > package/gdb/Config.in.host | 15 +++++++++------ > 1 file changed, 9 insertions(+), 6 deletions(-) > diff --git a/package/gdb/Config.in.host b/package/gdb/Config.in.host > index 333da6bad2..41016f6ccd 100644 > --- a/package/gdb/Config.in.host > +++ b/package/gdb/Config.in.host > @@ -45,7 +45,7 @@ config BR2_PACKAGE_HOST_GDB_SIM > choice > prompt "GDB debugger Version" > - default BR2_GDB_VERSION_7_11 > + default BR2_GDB_VERSION_8_0 > depends on !BR2_arc > help > Select the version of gdb you wish to use. > @@ -75,14 +75,17 @@ endif > # Tells whether the currently selected gdb version requires C++11 > # support in the toolchain. When host-gdb is not enabled, the target > -# gdb built is 7.11, which doesn't require C++11 support. So it's only > -# if host-gdb is built, with the version set to 8.0 or 8.1 that C++11 > -# support is needed in the toolchain to build gdb for the target. > +# gdb built is 8.0, which requires C++11 support, which is why > +# BR2_PACKAGE_GDB_NEEDS_CXX11 is 'y' when BR2_PACKAGE_HOST_GDB is not > +# enabled. When host-gdb is built, with the version set to 8.0 or 8.1 > +# that C++11 support is needed in the toolchain to build gdb for the s/that/then/ Committed, thanks.
diff --git a/package/gdb/Config.in.host b/package/gdb/Config.in.host index 333da6bad2..41016f6ccd 100644 --- a/package/gdb/Config.in.host +++ b/package/gdb/Config.in.host @@ -45,7 +45,7 @@ config BR2_PACKAGE_HOST_GDB_SIM choice prompt "GDB debugger Version" - default BR2_GDB_VERSION_7_11 + default BR2_GDB_VERSION_8_0 depends on !BR2_arc help Select the version of gdb you wish to use. @@ -75,14 +75,17 @@ endif # Tells whether the currently selected gdb version requires C++11 # support in the toolchain. When host-gdb is not enabled, the target -# gdb built is 7.11, which doesn't require C++11 support. So it's only -# if host-gdb is built, with the version set to 8.0 or 8.1 that C++11 -# support is needed in the toolchain to build gdb for the target. +# gdb built is 8.0, which requires C++11 support, which is why +# BR2_PACKAGE_GDB_NEEDS_CXX11 is 'y' when BR2_PACKAGE_HOST_GDB is not +# enabled. When host-gdb is built, with the version set to 8.0 or 8.1 +# that C++11 support is needed in the toolchain to build gdb for the +# target. # # Even though this option is related to target gdb dependencies, we # keep it next to the BR2_GDB_VERSION so that they are kept in sync. config BR2_PACKAGE_GDB_NEEDS_CXX11 bool + default y if !BR2_PACKAGE_HOST_GDB default y if BR2_GDB_VERSION_8_0 default y if BR2_GDB_VERSION_8_1 @@ -91,8 +94,8 @@ config BR2_GDB_VERSION string default "arc-2018.03-rc2-gdb" if BR2_arc default "7.10.1" if BR2_GDB_VERSION_7_10 - default "7.11.1" if BR2_GDB_VERSION_7_11 || !BR2_PACKAGE_HOST_GDB + default "7.11.1" if BR2_GDB_VERSION_7_11 default "7.12.1" if BR2_GDB_VERSION_7_12 - default "8.0.1" if BR2_GDB_VERSION_8_0 + default "8.0.1" if BR2_GDB_VERSION_8_0 || !BR2_PACKAGE_HOST_GDB default "8.1" if BR2_GDB_VERSION_8_1 depends on BR2_PACKAGE_GDB || BR2_PACKAGE_HOST_GDB