This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Rename Solaris's target to "target child" like most other ports.
- From: Pedro Alves <palves at redhat dot com>
- To: gdb-patches at sourceware dot org
- Date: Wed, 12 Mar 2014 12:48:23 +0000
- Subject: Rename Solaris's target to "target child" like most other ports.
- Authentication-results: sourceware.org; auth=none
- References: <1394627872-18619-1-git-send-email-palves at redhat dot com>
On 03/12/2014 12:37 PM, Pedro Alves wrote:
> I wonder whether anyone would miss this badly? The difference would be:
>
> (gdb) maint print target-stack
> The current target stack is:
> - multi-thread (multi-threaded child process.)
> - - child (Unix child process)
> + - child (Child process)
> - exec (Local exec file)
> - None (None)
>
> (gdb) help target child
> - Unix child process (started by the "run" command).
> + Child process (started by the "run" command).
>
> I find it unnecessary (and really slightly a lie) on GNU/Linux.
>
> It's also odd that e.g., the Windows port says "Unix" in reaction to
> "target child" (it was already that way before Windows used
> inf-child.c).
>
> How about we just say the same mostly everywhere?
>
> In the same vein, I'd also rename Solaris/procfs.c's "target procfs"
> to "target child".
Like this. (Untested.)
-------
[PATCH] Rename Solaris's target to "target child" like most other
ports.
Note that "target procfs" is used by QNX, but the test must be failing
there, as nto-procfs.c overrides to_open with a method that doesn't
throw the error being tested. So I'm just removing the test
completely.
gdb/
2014-03-12 Pedro Alves <palves@redhat.com>
* procfs.c (procfs_target): Don't override to_shortname,
to_longname or to_doc.
gdb/testsuite/
2014-03-12 Pedro Alves <palves@redhat.com>
* gdb.base/default.exp: Don't test "target procfs".
---
gdb/procfs.c | 4 ----
gdb/testsuite/gdb.base/default.exp | 3 ---
2 files changed, 7 deletions(-)
diff --git a/gdb/procfs.c b/gdb/procfs.c
index 8204747..f0d65d2 100644
--- a/gdb/procfs.c
+++ b/gdb/procfs.c
@@ -184,10 +184,6 @@ procfs_target (void)
{
struct target_ops *t = inf_child_target ();
- t->to_shortname = "procfs";
- t->to_longname = "Unix /proc child process";
- t->to_doc =
- "Unix /proc child process (started by the \"run\" command).";
t->to_create_inferior = procfs_create_inferior;
t->to_kill = procfs_kill_inferior;
t->to_mourn_inferior = procfs_mourn_inferior;
diff --git a/gdb/testsuite/gdb.base/default.exp b/gdb/testsuite/gdb.base/default.exp
index b63955e..a39ffcf 100644
--- a/gdb/testsuite/gdb.base/default.exp
+++ b/gdb/testsuite/gdb.base/default.exp
@@ -749,9 +749,6 @@ gdb_test "symbol-file" ".*" "symbol-file"
#test target child
gdb_test "target child" "Use the \"run\" command to start a child process.*|Undefined target command: \"child\". *Try \"help target\".*" "target child"
-#test target procfs
-gdb_test "target procfs" "Use the \"run\" command to start a child process.*|Undefined target command: \"procfs\". *Try \"help target\".*" "target procfs"
-
#test target core
send_gdb "target core\n"
gdb_expect {
--
1.7.11.7
--
Pedro Alves