Re: [Bug win32/14529] Make gdb capable of JIT-debugging on W32

LRN
2016-06-30T18:36:34Z

[-- Attachment #1.1.1: Type: text/plain, Size: 675 bytes --]

On 30.06.2016 20:42, Eli Zaretskii wrote:
> On Thu, 30 Jun 2016 20:18:52 +0300, LRN wrote:
>> On 30.06.2016 19:23, Eli Zaretskii wrote:
>>> Thanks, but please format the ChangeLog entries according to our
>>> coding standards. You should be able to see many examples in the
>>> repository.
>>
>> I did. What did i do wrong? Had a single space (instead of two spaces)
>> after the name? Used "Fix PR gdb/14529" at the end instead of "PR
>> gdb/14529" at the beginning? No function name in parentheses (should it
>> have been signal_event_command?)?
>
> The latter.
>

Fixed patches attached.

--
O< ascii ribbon - stop html email! - www.asciiribbon.org

[-- Attachment #1.1.2: 0001-Add-signal-event-command-to-the-NEWS-file.patch --]
[-- Type: text/plain, Size: 1105 bytes --]

From 87d5a55484ef412b0c79e66ac508150d81112186 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=D0=A0=D1=83=D1=81=D0=BB=D0=B0=D0=BD=20=D0=98=D0=B6=D0=B1?=
=?UTF-8?q?=D1=83=D0=BB=D0=B0=D1=82=D0=BE=D0=B2?= <lrn1986@gmail.com>
Date: Thu, 30 Jun 2016 15:34:06 +0000
Subject: [PATCH 1/2] Add "signal-event" command to the NEWS file

---
gdb/NEWS | 6 ++++++
1 file changed, 6 insertions(+)

diff --git a/gdb/NEWS b/gdb/NEWS
index 3823f20..da04ae6 100644
--- a/gdb/NEWS
+++ b/gdb/NEWS
@@ -80,6 +80,12 @@ new-ui INTERP TTY
Start a new user interface instance running INTERP as interpreter,
using the TTY file for input/output.

+signal-event EVENTID
+ Signal ("set") the given Windows event object. This is used together
+ with Windows JIT debugging (AeDebug), where the OS suspends
+ a crashing process until a debugger can attach to it. Resuming
+ the process is done by siganlling an event.
+
* Support for tracepoints and fast tracepoints on s390-linux and s390x-linux
was added in GDBserver, including JIT compiling fast tracepoint's
conditional expression bytecode into native code.
--
2.4.0

[-- Attachment #1.1.3: 0002-Add-a-changelog-entry-for-the-signal-event-command.patch --]
[-- Type: text/plain, Size: 945 bytes --]

From 59df288924ba3714476f45dffe760ab04188e8ea Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=D0=A0=D1=83=D1=81=D0=BB=D0=B0=D0=BD=20=D0=98=D0=B6=D0=B1?=
=?UTF-8?q?=D1=83=D0=BB=D0=B0=D1=82=D0=BE=D0=B2?= <lrn1986@gmail.com>
Date: Thu, 30 Jun 2016 15:41:29 +0000
Subject: [PATCH 2/2] Add a changelog entry for the 'signal-event' command

---
gdb/ChangeLog | 8 ++++++++
1 file changed, 8 insertions(+)

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index e98a565..f1e2d9e 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,11 @@
+2016-06-30 Руслан Ижбулатов <lrn1986@gmail.com>
+
+ PR gdb/14529
+ * windows-nat.c (signal_event_command): New command
+ 'signal-event' for W32 compatibility.
+ * gdb.texinfo: document the new 'signal-event' command.
+ * NEWS: Add an entry about the new 'signal-event' command.
+
2016-06-25 Tom Tromey <tom@tromey.com>

* rust-lang.c (rust_get_disr_info, rust_print_type): Fix
--
2.4.0

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]