diff mbox

Rename Solaris's target to "target child" like most other ports.

Message ID 53205797.3050309@redhat.com
State Committed
Headers show

Commit Message

Pedro Alves March 12, 2014, 12:48 p.m. UTC 
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(-)

Comments

Joel Brobecker March 13, 2014, 7:39 a.m. UTC | #1 
> 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".

FWIW, this looks good to commit to me, even if untested. In the very
unlikely even that it does break somethhing, I will take care of it
then.
Pedro Alves March 13, 2014, 12:32 p.m. UTC | #2 
On 03/13/2014 07:39 AM, Joel Brobecker wrote:
>> 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".
> 
> FWIW, this looks good to commit to me, even if untested. In the very
> unlikely even that it does break somethhing, I will take care of it
> then.

Thanks Joel.  Pushed.
diff mbox

Patch

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 {