This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[RFA(7.8)+NEWS] Remove "set/show remotebaud" (deprecated) commands.
- From: Joel Brobecker <brobecker at adacore dot com>
- To: gdb-patches at sourceware dot org
- Date: Mon, 19 May 2014 11:56:48 -0700
- Subject: [RFA(7.8)+NEWS] Remove "set/show remotebaud" (deprecated) commands.
- Authentication-results: sourceware.org; auth=none
Hello,
This patch removes support for the "set/show remotebaud" command,
which were deprecated in GDB 7.7, and should be now be removed
ahead of cutting the GDB 7.8 branch. This should take care of
the one item that's on my list pre- branching 7.8.
gdb/ChangeLog:
* serial.c (_initialize_serial): Remove support for
the "set remotebaud" and "show remotebaud" commands.
* NEWS: Add entry documenting the removal of that command.
gdb/testsuite/ChangeLog:
* config/monitor.exp (gdb_target_monitor): Replace use of
"set remotebaud" by "set serial baud".
Tested on x86_64-linux. The testsuite change was tested by hand
by copy/pasting the command and making sure that it works...
OK to commit?
Thanks,
--
Joel
---
gdb/NEWS | 4 ++++
gdb/serial.c | 28 ----------------------------
gdb/testsuite/config/monitor.exp | 2 +-
3 files changed, 5 insertions(+), 29 deletions(-)
diff --git a/gdb/NEWS b/gdb/NEWS
index 00ec8b9..fc5ef45 100644
--- a/gdb/NEWS
+++ b/gdb/NEWS
@@ -113,6 +113,10 @@ PowerPC64 GNU/Linux little-endian powerpc64le-*-linux*
and "assf"), have been deprecated. Use the "sharedlibrary" command, or
its alias "share", instead.
+* The commands "set remotebaud" and "show remotebaud" are no longer
+ supported. Use "set serial baud" and "show serial baud" (respectively)
+ instead.
+
*** Changes in GDB 7.7
* Improved support for process record-replay and reverse debugging on
diff --git a/gdb/serial.c b/gdb/serial.c
index 7471a6b..e780bbe 100644
--- a/gdb/serial.c
+++ b/gdb/serial.c
@@ -671,34 +671,6 @@ using remote targets."),
serial_baud_show_cmd,
&serial_set_cmdlist, &serial_show_cmdlist);
- /* The commands "set/show serial baud" used to have a different name.
- Add aliases to those names to facilitate the transition, and mark
- them as deprecated, in order to make users aware of the fact that
- the command names have been changed. */
- {
- const char *cmd_name;
- struct cmd_list_element *cmd;
-
- /* FIXME: There is a limitation in the deprecation mechanism,
- and the warning ends up not being displayed for prefixed
- aliases. So use a real command instead of an alias. */
- add_setshow_zinteger_cmd ("remotebaud", class_alias, &baud_rate, _("\
-Set baud rate for remote serial I/O."), _("\
-Show baud rate for remote serial I/O."), _("\
-This value is used to set the speed of the serial port when debugging\n\
-using remote targets."),
- NULL,
- serial_baud_show_cmd,
- &setlist, &showlist);
- cmd_name = "remotebaud";
- cmd = lookup_cmd (&cmd_name, setlist, "", -1, 1);
- deprecate_cmd (cmd, "set serial baud");
- cmd_name
- = "remotebaud"; /* needed because lookup_cmd updates the pointer */
- cmd = lookup_cmd (&cmd_name, showlist, "", -1, 1);
- deprecate_cmd (cmd, "show serial baud");
- }
-
add_setshow_filename_cmd ("remotelogfile", no_class, &serial_logfile, _("\
Set filename for remote session recording."), _("\
Show filename for remote session recording."), _("\
diff --git a/gdb/testsuite/config/monitor.exp b/gdb/testsuite/config/monitor.exp
index bddcdd5..a85aaee 100644
--- a/gdb/testsuite/config/monitor.exp
+++ b/gdb/testsuite/config/monitor.exp
@@ -86,7 +86,7 @@ proc gdb_target_monitor { exec_file } {
return -1
}
if [target_info exists baud] {
- gdb_test "set remotebaud [target_info baud]" "" ""
+ gdb_test "set serial baud [target_info baud]" "" ""
}
if [target_info exists binarydownload] {
gdb_test "set remotebinarydownload [target_info binarydownload]" "" ""
--
1.9.1