This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH 3/3] Load system gdbinit files from a directory
On Wed, Aug 21, 2019 at 1:15 PM Sergio Durigan Junior
<sergiodj@redhat.com> wrote:
>
> On Tuesday, August 20 2019, Christian Biesinger via gdb-patches wrote:
>
> > diff --git a/gdb/configure b/gdb/configure
> > index cb71bbf057..e5aa2e6b3b 100755
> > --- a/gdb/configure
> > +++ b/gdb/configure
> > @@ -693,6 +693,7 @@ WIN32LIBS
> > SER_HARDWIRE
> > WERROR_CFLAGS
> > WARN_CFLAGS
> > +SYSTEM_GDBINIT_DIR
> > SYSTEM_GDBINIT
> > TARGET_SYSTEM_ROOT
> > CONFIG_LDFLAGS
> > @@ -824,6 +825,7 @@ infodir
> > docdir
> > oldincludedir
> > includedir
> > +runstatedir
> > localstatedir
> > sharedstatedir
> > sysconfdir
> > @@ -884,6 +886,7 @@ with_libipt_prefix
> > with_included_regex
> > with_sysroot
> > with_system_gdbinit
> > +with_system_gdbinit_dir
> > enable_werror
> > enable_build_warnings
> > enable_gdb_build_warnings
> > @@ -956,6 +959,7 @@ datadir='${datarootdir}'
> > sysconfdir='${prefix}/etc'
> > sharedstatedir='${prefix}/com'
> > localstatedir='${prefix}/var'
> > +runstatedir='${localstatedir}/run'
> > includedir='${prefix}/include'
> > oldincludedir='/usr/include'
> > docdir='${datarootdir}/doc/${PACKAGE}'
> > @@ -1208,6 +1212,15 @@ do
> > | -silent | --silent | --silen | --sile | --sil)
> > silent=yes ;;
> >
> > + -runstatedir | --runstatedir | --runstatedi | --runstated \
> > + | --runstate | --runstat | --runsta | --runst | --runs \
> > + | --run | --ru | --r)
> > + ac_prev=runstatedir ;;
> > + -runstatedir=* | --runstatedir=* | --runstatedi=* | --runstated=* \
> > + | --runstate=* | --runstat=* | --runsta=* | --runst=* | --runs=* \
> > + | --run=* | --ru=* | --r=*)
> > + runstatedir=$ac_optarg ;;
> > +
>
> Something else I forgot to comment: this 'runstatedir' addition seems
> unrelated to the patch; it's probably due to the autoconf version you're
> using to regenerate these files.
>
> Make sure you have the correct versions installed:
>
> https://sourceware.org/gdb/wiki/DeveloperTips#Editing_configure.ac
Turns out the debian version of autoconf has a local patch to do this
runstate thing :(
https://sources.debian.org/src/autoconf/2.69-11/debian/patches/add-runstatedir.patch/
Anyway, I'll hold off on updating this patch pending the discussion
about the approach in the other thread. (But patches 1 and 2 may still
be good to get committed)
Christian