[PATCH] Fix some dangling references to `netbsd-tdep`

Simon Marchi simon.marchi@polymtl.ca
Mon Jul 5 13:50:39 GMT 2021


On 2021-07-01 10:46 a.m., John Ericson wrote:
> These files were renamed in 1b71cfcfdc3e13a655fefa6566b5564cec044c10,
> but evidentially a few dangling references were left behind.
> 
> N.B. Originally submitted to the binutils list by mistake as
> https://sourceware.org/pipermail/binutils/2021-June/117186.html , and
> then to the non-patch GDB list as
> https://sourceware.org/pipermail/gdb/2021-July/049547.html . I am sorry,
> still getting used to this thing.

Hi!

No worries, as long as you learn in the process.  It will go smoother
next time.

> ---
>  gdb/configure.tgt | 8 ++++----
>  gdb/netbsd-tdep.c | 4 ++--
>  gdb/sparc-tdep.h  | 2 +-
>  3 files changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/gdb/configure.tgt b/gdb/configure.tgt
> index a928c0227a0..643973917fe 100644
> --- a/gdb/configure.tgt
> +++ b/gdb/configure.tgt
> @@ -105,7 +105,7 @@ case "${targ}" in
>  *-*-freebsd* | *-*-kfreebsd*-gnu)
>  	os_obs="fbsd-tdep.o solib-svr4.o";;
>  *-*-netbsd* | *-*-knetbsd*-gnu)
> -	os_obs="nbsd-tdep.o solib-svr4.o";;
> +	os_obs="netbsd-tdep.o solib-svr4.o";;
>  *-*-openbsd*)
>  	os_obs="obsd-tdep.o solib-svr4.o";;
>  esac
> @@ -146,7 +146,7 @@ alpha*-*-netbsd* | alpha*-*-knetbsd*-gnu)
>  alpha*-*-openbsd*)
>  	# Target: OpenBSD/alpha
>  	gdb_target_obs="alpha-mdebug-tdep.o alpha-bsd-tdep.o \
> -			alpha-netbsd-tdep.o alpha-obsd-tdep.o nbsd-tdep.o"
> +			alpha-netbsd-tdep.o alpha-obsd-tdep.o netbsd-tdep.o"
>  	;;
>  
>  am33_2.0*-*-linux*)
> @@ -632,14 +632,14 @@ sparc64-*-netbsd* | sparc64-*-knetbsd*-gnu)
>  sparc-*-openbsd*)
>  	# Target: OpenBSD/sparc
>  	gdb_target_obs="sparc-tdep.o sparc-netbsd-tdep.o sparc-obsd-tdep.o \
> -			nbsd-tdep.o bsd-uthread.o \
> +			netbsd-tdep.o bsd-uthread.o \
>  			ravenscar-thread.o sparc-ravenscar-thread.o"
>  	;;
>  sparc64-*-openbsd*)
>  	# Target: OpenBSD/sparc64
>  	gdb_target_obs="sparc64-tdep.o sparc64-netbsd-tdep.o sparc64-obsd-tdep.o \
>  			sparc-tdep.o sparc-netbsd-tdep.o sparc-obsd-tdep.o \
> -			nbsd-tdep.o bsd-uthread.o \
> +			netbsd-tdep.o bsd-uthread.o \
>  			ravenscar-thread.o sparc-ravenscar-thread.o"
>  	;;
>  sparc-*-solaris2* | sparcv9-*-solaris2* | sparc64-*-solaris2*)

It sounds like the changes in configure.tgt are not like the others
(cosmetic changes in comments) but would in fact fix a build failure
when GDB is built with, say, --target=sparc64-something-openbsd (I don't
know what the actual triplets look like).  But using that gives me:

    $ make
    make: *** No rule to make target 'nbsd-tdep.c', needed by 'nbsd-tdep.o'.  Stop.

So, please mention this in the commit message, it's an important point.

Simon


More information about the Gdb-patches mailing list