public inbox for insight@sourceware.org
 help / color / mirror / Atom feed
From: Keith Seitz <keiths@redhat.com>
To: Gene Smith <gds@chartertn.net>
Cc: insight@sources.redhat.com
Subject: Re: undefined reference to `set_breakpoint'
Date: Thu, 18 Mar 2010 03:39:00 -0000	[thread overview]
Message-ID: <4BA11802.8050909@redhat.com> (raw)
In-Reply-To: <m3vdcu6ef6.fsf@fleche.redhat.com>

[-- Attachment #1: Type: text/plain, Size: 562 bytes --]

On 03/17/2010 10:33 AM, Tom Tromey wrote:
>>>>>> "Gene" == Gene Smith<gds@chartertn.net>  writes:
> No, this is due to a change to gdb without a corresponding fix to
> insight.  This happens from time to time.

Do you want to test the attach patch? [I'm off the rest of the week, or 
I would probably have occasion to put it through its paces.]

Let me know if this works -- it was okay in preliminary testing.

Keith

ChangeLog
2010-03-17  Keith Seitz  <keiths@redhat.com>

	* generic/gdbtk-bp.c (gdb_set_bp): set_breakpoint is no more.
	Use create_breakpoint.

[-- Attachment #2: set_breakpoint.patch --]
[-- Type: text/plain, Size: 1181 bytes --]

Index: generic/gdbtk-bp.c
===================================================================
RCS file: /cvs/src/src/gdb/gdbtk/generic/gdbtk-bp.c,v
retrieving revision 1.33
diff -u -p -r1.33 gdbtk-bp.c
--- generic/gdbtk-bp.c	15 Jan 2010 22:37:18 -0000	1.33
+++ generic/gdbtk-bp.c	17 Mar 2010 17:56:54 -0000
@@ -1,5 +1,5 @@
 /* Tcl/Tk command definitions for Insight - Breakpoints.
-   Copyright (C) 2001, 2002, 2008, 2009 Free Software Foundation, Inc.
+   Copyright (C) 2001, 2002, 2008, 2009, 2010 Free Software Foundation, Inc.
 
    This file is part of GDB.
 
@@ -530,9 +530,16 @@ gdb_set_bp (ClientData clientData, Tcl_I
 
   TRY_CATCH (e, RETURN_MASK_ALL)
     {
-      set_breakpoint (get_current_arch (), address, condition,
-		      0 /* hardwareflag */, temp, thread, ignore_count,
-		      pending, enabled);
+      create_breakpoint (get_current_arch (), address, condition, thread,
+			 0	/* condition and thread are valid */,
+			 temp,
+			 0	/* hardware flag */,
+			 0	/* trace flag */,
+			 ignore_count,
+			 (pending ? AUTO_BOOLEAN_TRUE : AUTO_BOOLEAN_FALSE),
+			 NULL	/* breakpoint ops */,
+			 0	/* from_tty */,
+			 enabled);
     }
 
   if (e.reason < 0)

  reply	other threads:[~2010-03-17 17:57 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-17 17:34 Gene Smith
2010-03-17 17:57 ` Tom Tromey
2010-03-18  3:39   ` Keith Seitz [this message]
2010-03-18  4:58     ` Gene Smith
2010-03-19  4:32     ` Gene Smith
2010-03-20  5:33       ` Hui Zhu
2010-03-30  8:29         ` Hui Zhu
2010-03-30 16:48           ` Hui Zhu
2010-03-31  3:20             ` Keith Seitz
2010-03-31  4:12               ` Hui Zhu
2010-03-31  4:28                 ` Keith Seitz
2010-03-31  5:33                   ` Hui Zhu
2010-03-31  5:37                     ` Keith Seitz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4BA11802.8050909@redhat.com \
    --to=keiths@redhat.com \
    --cc=gds@chartertn.net \
    --cc=insight@sources.redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).