Re: [PATCH 1/1] gdb: Document C++17 build requirement.

[Lancelot SIX <lsix@lancelotsix.com>]

Hi Felix,

Thank for catching this, I missed that part of the documentation when
doing the C++17 migration.

I have updated the wiki to reflect the C++17 requirement.

Reviewed-By: Lancelot Six <lancelot.six@amd.com>

Best,
Lancelot.

On Mon, Feb 19, 2024 at 03:56:55PM +0100, Felix Willgerodt wrote:
> We require C++17 to build for a while now:
> https://sourceware.org/git/?p=binutils-gdb.git;a=commit;h=f74dc26792a0679e29db45e56367331ff48666d1
> 
> I also found another place we probably should update:
> https://sourceware.org/gdb/wiki/Internals%20GDB-C-Coding-Standards#C.2B-.2B-_dialect
> But I don't know how to do that.
> 
> ---
>  gdb/doc/gdb.texinfo | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo
> index e4b89cfccfc..79805486dd6 100644
> --- a/gdb/doc/gdb.texinfo
> +++ b/gdb/doc/gdb.texinfo
> @@ -40555,9 +40555,9 @@ Other packages will be used only if they are found.
>  
>  @heading Tools/Packages Necessary for Building @value{GDBN}
>  @table @asis
> -@item C@t{++}11 compiler
> -@value{GDBN} is written in C@t{++}11.  It should be buildable with any
> -recent C@t{++}11 compiler, e.g.@: GCC.
> +@item C@t{++}17 compiler
> +@value{GDBN} is written in C@t{++}17.  It should be buildable with any
> +recent C@t{++}17 compiler, e.g.@: GCC.
>  
>  @item GNU make
>  @value{GDBN}'s build system relies on features only found in the GNU
> -- 
> 2.34.1
> 
> Intel Deutschland GmbH
> Registered Address: Am Campeon 10, 85579 Neubiberg, Germany
> Tel: +49 89 99 8853-0, www.intel.de <http://www.intel.de>
> Managing Directors: Christin Eisenschmid, Sharon Heck, Tiffany Doon Silva  
> Chairperson of the Supervisory Board: Nicole Lau
> Registered Office: Munich
> Commercial Register: Amtsgericht Muenchen HRB 186928
>