Re: [review] Load system gdbinit files from a directory

[Eli Zaretskii <eliz at gnu dot org>]

> Date: Wed, 16 Oct 2019 18:18:34 -0400
> From: "Christian Biesinger (Code Review)" <gerrit@gnutoolchain-gerrit.osci.io>
> Cc: Tom de Vries <tdevries@suse.de>,	Simon Marchi <simon.marchi@polymtl.ca>
> 
> gdb/ChangeLog:
> 
> 2019-08-20  Christian Biesinger  <cbiesinger@google.com>
> 
> 	* NEWS: Mention new --with-system-gdbinit-dir option.
> 	* config.in: Regenerate.
> 	* configure: Regenerate.
> 	* configure.ac: Add new option --with-system-gdbinit-dir.
> 	* extension.c (get_ext_lang_of_file): Return extension_language_gdb
> 	for a ".gdb" suffix.
> 	* main.c (get_init_files): Change system_gdbinit argument to
> 	a vector and return the files in SYSTEM_GDBINIT_DIR in
> 	addition to SYSTEM_GDBINIT.
> 	(captured_main_1): Update.
> 	(print_gdb_help): Update.
> 	* top.c (print_gdb_configuration): Also print the value of
> 	SYSTEM_GDBINIT_DIR.
> 
> gdb/doc/ChangeLog:
> 
> 2019-08-25  Christian Biesinger  <cbiesinger@google.com>
> 
> 	* Makefile.in: Also set SYSTEM_GDBINIT_DIR for the info manual
> 	generation.
> 	* gdb.texinfo (many sections): Document new --with-system-gdbinit-dir
> 	option.

I think I already approved the changes to the documentation, didn't I?

> diff --git a/gdb/NEWS b/gdb/NEWS
> index 25e67e4..7613598 100644
> --- a/gdb/NEWS
> +++ b/gdb/NEWS
> @@ -46,6 +46,12 @@
>    The 'outer_function::' prefix is only needed if 'inner_function' is
>    not visible in the current scope.
>  
> +* In addition to the system-wide gdbinit file, if configured with
> + --with-system-gdbinit-dir, GDB will now also load files in that directory
> + as system gdbinit files, unless the -nx or -n flag is provided.  Files
> + with extensions .gdb, .py and .scm are supported as long as GDB was
> + compiled with support for that language.

OK for this part.

> +@item @file{system.gdbinit.d}
> +This is the system-wide init directory.
> +Its location is specified with the @code{--with-system-gdbinit-dir}
> +configure option (@pxref{System-wide configuration}).
> +Files in this directory are loaded in alphabetical order immediately after
> +system.gdbinit (if enabled) when @value{GDBN} starts, before command line
> +options have been processed.  Files need to have a recognized scripting
> +language extension (@code{.py}/@code{.scm}) or be named with a @code{.gdb}

File-name extensions should have the @file markup.

> @@ -1315,8 +1325,11 @@
>  @cindex init file
>  Reads the system-wide @dfn{init file} (if @option{--with-system-gdbinit} was
>  used when building @value{GDBN}; @pxref{System-wide configuration,
> - ,System-wide configuration and settings}) and executes all the commands in
> -that file.
> + ,System-wide configuration and settings}) and the files in the system-wide
> +gdbinit directory (if @option{--with-system-gdbinit-dir} was used) and executes
> +all the commands in those files.  The files need to be named with a @code{.gdb}
> +extension to be interpreted as @value{GDBN} commands, or they can be written
> +in a supported scripting language with an appropriate file extension.

Likewise here, and elsewhere in the patch.

Thanks.