This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Remove --xdb? (Re: [testsuite patch] Remove gdb.hp [Re: [patch] Remove HPUX])
- From: Pedro Alves <palves at redhat dot com>
- To: Jan Kratochvil <jan dot kratochvil at redhat dot com>, Yao Qi <yao at codesourcery dot com>
- Cc: gdb-patches at sourceware dot org, Joel Brobecker <brobecker at adacore dot com>
- Date: Wed, 18 Mar 2015 14:58:08 +0000
- Subject: Remove --xdb? (Re: [testsuite patch] Remove gdb.hp [Re: [patch] Remove HPUX])
- Authentication-results: sourceware.org; auth=none
- References: <20141016220028 dot GA25839 at host2 dot jankratochvil dot net> <87oas1fdej dot fsf at codesourcery dot com> <20150313191618 dot GA12286 at host1 dot jankratochvil dot net>
On 03/13/2015 07:16 PM, Jan Kratochvil wrote:
> On Fri, 21 Nov 2014 04:20:04 +0100, Yao Qi wrote:
>> Jan Kratochvil <jan.kratochvil@redhat.com> writes:
>>
>>> I did not investigate too much which other parts of GDB could be removed
>>> along, there may be some dead code now (but it always has been).
>>
>> Directory testsuite/gdb.hp should be removed,
>
> Attached, OK for check-in?
>
I read through all this, and the only things that looked like could
be salvageable and mildly interesting would be the "gdb --xdb"
tests (in gdb.compat/). But, the only reference to xdb found
in the manual is commented out:
@c @item -xdb
@c @cindex @code{--xdb}
@c Run in XDB compatibility mode, allowing the use of certain XDB commands.
@c For information, see the file @file{xdb_trans.html}, which is usually
@c installed in the directory @code{/opt/langtools/wdb/doc} on HP-UX
@c systems.
The commands that enables aren't even documented in the manual.
Judging from that, I assume that only wdb users would ever really
be using the --xdb switch.
I think it's time to drop "support" for the --xdb switch too. I
looked through the commands that that exposes, the only that looked
potentially interesting was "go", but then it's just an alias
for "tbreak+jump", which can easily be done with "define go...end".
I'd rather free up the "go" name for something potentially
more interesting (either run control, or maybe even unrelated,
e.g., for golang).
So the patch looks good to go to me.
>> as well as some target triplet checking on hpux in testsuite/lib/gdb.exp.
>
> To be done later, I have spent some time on it but it seems unclear to me what
> should / should not be deleted, GDB seems to assume in many cases hppa==hpux
> which is not always the case.
Thanks,
Pedro Alves