public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] [PR cli/1729] doc: Fix show remote interrupt-sequence.
@ 2021-02-14  9:21 Lancelot SIX
  2021-02-17 17:55 ` Tom Tromey
  0 siblings, 1 reply; 4+ messages in thread
From: Lancelot SIX @ 2021-02-14  9:21 UTC (permalink / raw)
  To: gdb-patches; +Cc: Lancelot SIX

Add the missing 'remote' word.

gdb/doc/ChangeLog

	PR cli/1729
	* gdb.texinfo (Remote Configuration): Fix show remote
	interrupt-sequence.
---
 gdb/doc/gdb.texinfo | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo
index 0b1deba9667..1f089de0bde 100644
--- a/gdb/doc/gdb.texinfo
+++ b/gdb/doc/gdb.texinfo
@@ -22905,7 +22905,7 @@ is high level of serial line for some certain time.
 Linux kernel prefers @samp{BREAK-g}, a.k.a Magic SysRq g.
 It is @code{BREAK} signal followed by character @code{g}.
 
-@item show interrupt-sequence
+@item show remote interrupt-sequence
 Show which of @samp{Ctrl-C}, @code{BREAK} or @code{BREAK-g}
 is sent by @value{GDBN} to interrupt the remote program.
 @code{BREAK-g} is BREAK signal followed by @code{g} and
-- 
2.30.0


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] [PR cli/1729] doc: Fix show remote interrupt-sequence.
  2021-02-14  9:21 [PATCH] [PR cli/1729] doc: Fix show remote interrupt-sequence Lancelot SIX
@ 2021-02-17 17:55 ` Tom Tromey
  2021-02-17 19:28   ` Eli Zaretskii
  2021-02-17 23:23   ` [PATCH] [PR cli/17290] doc: Fix show remote interrupt-* Lancelot SIX
  0 siblings, 2 replies; 4+ messages in thread
From: Tom Tromey @ 2021-02-17 17:55 UTC (permalink / raw)
  To: Lancelot SIX via Gdb-patches; +Cc: Lancelot SIX

>>>>> "Lancelot" == Lancelot SIX via Gdb-patches <gdb-patches@sourceware.org> writes:

Lancelot> Add the missing 'remote' word.
Lancelot> gdb/doc/ChangeLog

Lancelot> 	PR cli/1729
Lancelot> 	* gdb.texinfo (Remote Configuration): Fix show remote
Lancelot> 	interrupt-sequence.

Thanks.  This patch would qualify under the "obvious" rule.

I took a look and there's another related instance a bit further down:

    @item show interrupt-on-connect

I wonder if you'd mind fixing this up as well before checking it in?

thanks,
Tom

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] [PR cli/1729] doc: Fix show remote interrupt-sequence.
  2021-02-17 17:55 ` Tom Tromey
@ 2021-02-17 19:28   ` Eli Zaretskii
  2021-02-17 23:23   ` [PATCH] [PR cli/17290] doc: Fix show remote interrupt-* Lancelot SIX
  1 sibling, 0 replies; 4+ messages in thread
From: Eli Zaretskii @ 2021-02-17 19:28 UTC (permalink / raw)
  To: Tom Tromey; +Cc: gdb-patches, lsix

> From: Tom Tromey <tom@tromey.com>
> Date: Wed, 17 Feb 2021 10:55:50 -0700
> Cc: Lancelot SIX <lsix@lancelotsix.com>
> 
> Lancelot> 	PR cli/1729
> Lancelot> 	* gdb.texinfo (Remote Configuration): Fix show remote
> Lancelot> 	interrupt-sequence.
> 
> Thanks.  This patch would qualify under the "obvious" rule.

Right.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] [PR cli/17290] doc: Fix show remote interrupt-*.
  2021-02-17 17:55 ` Tom Tromey
  2021-02-17 19:28   ` Eli Zaretskii
@ 2021-02-17 23:23   ` Lancelot SIX
  1 sibling, 0 replies; 4+ messages in thread
From: Lancelot SIX @ 2021-02-17 23:23 UTC (permalink / raw)
  To: Tom Tromey, eliz; +Cc: Lancelot SIX via Gdb-patches

Le Wed, Feb 17, 2021 at 10:55:50AM -0700, Tom Tromey a écrit :
> >>>>> "Lancelot" == Lancelot SIX via Gdb-patches <gdb-patches@sourceware.org> writes:
> 
> Lancelot> Add the missing 'remote' word.
> Lancelot> gdb/doc/ChangeLog
> 
> Lancelot> 	PR cli/1729
> Lancelot> 	* gdb.texinfo (Remote Configuration): Fix show remote
> Lancelot> 	interrupt-sequence.
> 
> Thanks.  This patch would qualify under the "obvious" rule.
> 
> I took a look and there's another related instance a bit further down:
> 
>     @item show interrupt-on-connect
> 
> I wonder if you'd mind fixing this up as well before checking it in?
> 
> thanks,
> Tom

I just pushed the following patch.

Lancelot.

---

commit 22e6d16f9b9caff5b4bbb27727441c21480af9ad
Author: Lancelot SIX <lsix@lancelotsix.com>
Date:   Sat Feb 13 23:38:28 2021 +0000

    [PR cli/17290] gdb/doc: Fix show remote interrupt-*.
    
    Add the missing 'remote' in:
      - @item show remote interrupt-sequence
      - @item show remote interrupt-on-connect

diff --git a/gdb/doc/ChangeLog b/gdb/doc/ChangeLog
index bc5a2ea0d93..7f324dedade 100644
--- a/gdb/doc/ChangeLog
+++ b/gdb/doc/ChangeLog
@@ -1,3 +1,9 @@
+2021-02-17  Lancelot Six  <lsix@lancelotsix.com>
+
+	PR cli/17290
+	* gdb.texinfo (Remote Configuration): Fix show remote
+	interrupt-sequence and show remote interrupt-on-connect.
+
 2021-02-11  Andrew Burgess  <andrew.burgess@embecosm.com>
 
 	* gdb.texinfo (Files): Update documentation for 'maint info
diff --git a/gdb/doc/gdb.texinfo b/gdb/doc/gdb.texinfo
index 0b1deba9667..90f0c7683f6 100644
--- a/gdb/doc/gdb.texinfo
+++ b/gdb/doc/gdb.texinfo
@@ -22905,7 +22905,7 @@ is high level of serial line for some certain time.
 Linux kernel prefers @samp{BREAK-g}, a.k.a Magic SysRq g.
 It is @code{BREAK} signal followed by character @code{g}.
 
-@item show interrupt-sequence
+@item show remote interrupt-sequence
 Show which of @samp{Ctrl-C}, @code{BREAK} or @code{BREAK-g}
 is sent by @value{GDBN} to interrupt the remote program.
 @code{BREAK-g} is BREAK signal followed by @code{g} and
@@ -22918,7 +22918,7 @@ Specify whether interrupt-sequence is sent to remote target when
 Linux kernel.  Linux kernel expects @code{BREAK} followed by @code{g}
 which is known as Magic SysRq g in order to connect @value{GDBN}.
 
-@item show interrupt-on-connect
+@item show remote interrupt-on-connect
 Show whether interrupt-sequence is sent
 to remote target when @value{GDBN} connects to it.
 


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-02-17 23:23 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-14  9:21 [PATCH] [PR cli/1729] doc: Fix show remote interrupt-sequence Lancelot SIX
2021-02-17 17:55 ` Tom Tromey
2021-02-17 19:28   ` Eli Zaretskii
2021-02-17 23:23   ` [PATCH] [PR cli/17290] doc: Fix show remote interrupt-* Lancelot SIX

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).