* undefined reference to `set_breakpoint' @ 2010-03-17 17:34 Gene Smith 2010-03-17 17:57 ` Tom Tromey 0 siblings, 1 reply; 13+ messages in thread From: Gene Smith @ 2010-03-17 17:34 UTC (permalink / raw) To: insight With CVS head, see this error when, I think, linking gdb: rm -f gdb gcc -g -O2 \ -o gdb gdb.o libgdb.a \ ../sim/arm/libsim.a ../readline/libreadline.a ../opcodes/libopcodes.a ../bfd/libbfd.a ../libiberty/libiberty.a ../libdecnumber/libdecnumber.a ../libgui/src/libgui.a -L/home/gene/addons/insight-cvs/insight_build/tk/unix -ltk8.4 -L/home/gene/addons/insight-cvs/insight_build/tcl/unix -ltcl8.4 -lX11 -ldl -lieee -lm -lncurses -lz -lm -lpython2.6 -lexpat ../libiberty/libiberty.a gnulib/libgnu.a libgdb.a(gdbtk-bp.o): In function `gdb_set_bp': /home/gene/addons/insight-cvs/insight_build/gdb/../../insight_sources/gdb/gdbtk/generic/gdbtk-bp.c:533: undefined reference to `set_breakpoint' Configured like this: configure -v --quiet --prefix=$PREFIX \ --target=$TARGET --enable-interwork --enable-multilib \ --with-gnu-ld --with-gnu-as \ --disable-werror Is this due to a missing package? This is on 64bit fedora 12. tk and tcl dev packages installed. Thanks, -gene ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: undefined reference to `set_breakpoint' 2010-03-17 17:34 undefined reference to `set_breakpoint' Gene Smith @ 2010-03-17 17:57 ` Tom Tromey 2010-03-18 3:39 ` Keith Seitz 0 siblings, 1 reply; 13+ messages in thread From: Tom Tromey @ 2010-03-17 17:57 UTC (permalink / raw) To: Gene Smith; +Cc: insight >>>>> "Gene" == Gene Smith <gds@chartertn.net> writes: Gene> /home/gene/addons/insight-cvs/insight_build/gdb/../../insight_sources/gdb/gdbtk/generic/gdbtk-bp.c:533: Gene> undefined reference to `set_breakpoint' Gene> Is this due to a missing package? This is on 64bit fedora 12. tk and Gene> tcl dev packages installed. No, this is due to a change to gdb without a corresponding fix to insight. This happens from time to time. Tom ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: undefined reference to `set_breakpoint' 2010-03-17 17:57 ` Tom Tromey @ 2010-03-18 3:39 ` Keith Seitz 2010-03-18 4:58 ` Gene Smith 2010-03-19 4:32 ` Gene Smith 0 siblings, 2 replies; 13+ messages in thread From: Keith Seitz @ 2010-03-18 3:39 UTC (permalink / raw) To: Gene Smith; +Cc: insight [-- 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) ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: undefined reference to `set_breakpoint' 2010-03-18 3:39 ` Keith Seitz @ 2010-03-18 4:58 ` Gene Smith 2010-03-19 4:32 ` Gene Smith 1 sibling, 0 replies; 13+ messages in thread From: Gene Smith @ 2010-03-18 4:58 UTC (permalink / raw) To: insight Keith Seitz wrote, On 03/17/2010 01:57 PM: > > 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 It builds and runs. Still need to try it in embedded mode using openocd. Thanks, -gene ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: undefined reference to `set_breakpoint' 2010-03-18 3:39 ` Keith Seitz 2010-03-18 4:58 ` Gene Smith @ 2010-03-19 4:32 ` Gene Smith 2010-03-20 5:33 ` Hui Zhu 1 sibling, 1 reply; 13+ messages in thread From: Gene Smith @ 2010-03-19 4:32 UTC (permalink / raw) To: insight Keith Seitz wrote, On 03/17/2010 01:57 PM: > 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 Built OK on/for f12-64bit native. Tried my build for mingw32 (using f12 cross compiler pkg that works OK for insight 6.8) and see this error when entering the itcl directory and apparently trying to "make all": i686-pc-mingw32-windres -o wish.res.o --include "../../../insight_sources/tk/win/../generic" --include "/home/gene/addons/insight-cvs/insight_sources/tcl/generic" --include "../../../insight_sources/tk/win/rc" ../../../insight_sources/tk/win/rc/wish.rc i686-pc-mingw32-gcc -g -O2 -D__USE_MINGW_ACCESS winMain.o "/home/gene/addons/insight-cvs/insight_build/tcl/win/libtcl84.a" libtk84.a -lgdi32 -lcomdlg32 -limm32 -lcomctl32 -lshell32 \ wish.res.o -o wish84.exe -mwindows -mwin32 make[3]: Leaving directory `/home/gene/addons/insight-cvs/insight_build/tk/win' make[2]: Leaving directory `/home/gene/addons/insight-cvs/insight_build/tk' make[2]: Entering directory `/home/gene/addons/insight-cvs/insight_build/itcl' Making all in itcl make[3]: Entering directory `/home/gene/addons/insight-cvs/insight_build/itcl/itcl' make[3]: *** No rule to make target `all'. Stop. make[3]: Leaving directory `/home/gene/addons/insight-cvs/insight_build/itcl/itcl' Making all in itk make[3]: Entering directory `/home/gene/addons/insight-cvs/insight_build/itcl/itk' make[3]: *** No rule to make target `all'. Stop. make[3]: Leaving directory `/home/gene/addons/insight-cvs/insight_build/itcl/itk' Making all in iwidgets make[3]: Entering directory `/home/gene/addons/insight-cvs/insight_build/itcl/iwidgets' make[3]: *** No rule to make target `all'. Stop. make[3]: Leaving directory `/home/gene/addons/insight-cvs/insight_build/itcl/iwidgets' make[3]: Entering directory `/home/gene/addons/insight-cvs/insight_build/itcl' make[3]: Nothing to be done for `all-am'. make[3]: Leaving directory `/home/gene/addons/insight-cvs/insight_build/itcl' make[2]: *** [all-recursive] Error 1 make[2]: Leaving directory `/home/gene/addons/insight-cvs/insight_build/itcl' make[1]: *** [all-itcl] Error 2 make[1]: Leaving directory `/home/gene/addons/insight-cvs/insight_build' make: *** [all] Error 2 Configured like this: ./configure -v --quiet --prefix=$PREFIX \ --target=$TARGET --enable-interwork --enable-multilib \ --with-gnu-ld --with-gnu-as \ --disable-werror \ --host=i686-pc-mingw32 --build=i686-pc-linux-gnu \ CC="i686-pc-mingw32-gcc" ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: undefined reference to `set_breakpoint' 2010-03-19 4:32 ` Gene Smith @ 2010-03-20 5:33 ` Hui Zhu 2010-03-30 8:29 ` Hui Zhu 0 siblings, 1 reply; 13+ messages in thread From: Hui Zhu @ 2010-03-20 5:33 UTC (permalink / raw) To: Gene Smith, Keith Seitz; +Cc: insight It looks like another issue. Does this patch can check in? Thanks, Hui On Thu, Mar 18, 2010 at 12:58, Gene Smith <gds@chartertn.net> wrote: > Keith Seitz wrote, On 03/17/2010 01:57 PM: >> >> 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 > > Built OK on/for f12-64bit native. Tried my build for mingw32 (using f12 > cross compiler pkg that works OK for insight 6.8) and see this error when > entering the itcl directory and apparently trying to "make all": > > i686-pc-mingw32-windres -o wish.res.o --include > "../../../insight_sources/tk/win/../generic" --include > "/home/gene/addons/insight-cvs/insight_sources/tcl/generic" --include > "../../../insight_sources/tk/win/rc" > ../../../insight_sources/tk/win/rc/wish.rc > i686-pc-mingw32-gcc -g -O2 -D__USE_MINGW_ACCESS winMain.o > "/home/gene/addons/insight-cvs/insight_build/tcl/win/libtcl84.a" libtk84.a > -lgdi32 -lcomdlg32 -limm32 -lcomctl32 -lshell32 \ > wish.res.o -o wish84.exe -mwindows -mwin32 > make[3]: Leaving directory > `/home/gene/addons/insight-cvs/insight_build/tk/win' > make[2]: Leaving directory `/home/gene/addons/insight-cvs/insight_build/tk' > make[2]: Entering directory > `/home/gene/addons/insight-cvs/insight_build/itcl' > Making all in itcl > make[3]: Entering directory > `/home/gene/addons/insight-cvs/insight_build/itcl/itcl' > make[3]: *** No rule to make target `all'. Stop. > make[3]: Leaving directory > `/home/gene/addons/insight-cvs/insight_build/itcl/itcl' > Making all in itk > make[3]: Entering directory > `/home/gene/addons/insight-cvs/insight_build/itcl/itk' > make[3]: *** No rule to make target `all'. Stop. > make[3]: Leaving directory > `/home/gene/addons/insight-cvs/insight_build/itcl/itk' > Making all in iwidgets > make[3]: Entering directory > `/home/gene/addons/insight-cvs/insight_build/itcl/iwidgets' > make[3]: *** No rule to make target `all'. Stop. > make[3]: Leaving directory > `/home/gene/addons/insight-cvs/insight_build/itcl/iwidgets' > make[3]: Entering directory > `/home/gene/addons/insight-cvs/insight_build/itcl' > make[3]: Nothing to be done for `all-am'. > make[3]: Leaving directory > `/home/gene/addons/insight-cvs/insight_build/itcl' > make[2]: *** [all-recursive] Error 1 > make[2]: Leaving directory > `/home/gene/addons/insight-cvs/insight_build/itcl' > make[1]: *** [all-itcl] Error 2 > make[1]: Leaving directory `/home/gene/addons/insight-cvs/insight_build' > make: *** [all] Error 2 > > Configured like this: > ./configure -v --quiet --prefix=$PREFIX \ > --target=$TARGET --enable-interwork --enable-multilib \ > --with-gnu-ld --with-gnu-as \ > --disable-werror \ > --host=i686-pc-mingw32 --build=i686-pc-linux-gnu \ > CC="i686-pc-mingw32-gcc" > > > > > > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: undefined reference to `set_breakpoint' 2010-03-20 5:33 ` Hui Zhu @ 2010-03-30 8:29 ` Hui Zhu 2010-03-30 16:48 ` Hui Zhu 0 siblings, 1 reply; 13+ messages in thread From: Hui Zhu @ 2010-03-30 8:29 UTC (permalink / raw) To: Keith Seitz; +Cc: insight gcc -g -O2 -I. -I../../src/gdb -I../../src/gdb/common -I../../src/gdb/config -DLOCALEDIR="\"/usr/local/share/locale\"" -DHAVE_CONFIG_H -I../../src/gdb/../include/opcode -I../../src/gdb/../opcodes/.. -I../../src/gdb/../readline/.. -I../bfd -I../../src/gdb/../bfd -I../../src/gdb/../include -I../libdecnumber -I../../src/gdb/../libdecnumber -I../../src/gdb/gnulib -Ignulib -DMI_OUT=1 -DTUI=1 -DGDBTK -Wall -Wdeclaration-after-statement -Wpointer-arith -Wformat-nonliteral -Wno-pointer-sign -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Werror -c -o gdbtk-bp.o -MT gdbtk-bp.o -MMD -MP -MF .deps/gdbtk-bp.Tpo -I../../src/gdb/../libgui/src -I/home/teawater/gdb/cvs/src/tcl/generic -I/home/teawater/gdb/cvs/src/tk/generic -DGDBTK_LIBRARY=\"/usr/local/share/insight1.0\" -DSRC_DIR=\"/home/teawater/gdb/cvs/src/gdb\" ../../src/gdb/gdbtk/generic/gdbtk-bp.c cc1: warnings being treated as errors ../../src/gdb/gdbtk/generic/gdbtk-bp.c: In function 'gdb_get_breakpoint_info': ../../src/gdb/gdbtk/generic/gdbtk-bp.c:365: warning: passing argument 1 of 'get_breakpoint_commands' from incompatible pointer type ../../src/gdb/gdbtk/generic/gdbtk-bp.c: In function 'gdb_actions_command': ../../src/gdb/gdbtk/generic/gdbtk-bp.c:688: warning: passing argument 1 of 'free_command_lines' from incompatible pointer type ../../src/gdb/gdbtk/generic/gdbtk-bp.c: In function 'gdb_get_tracepoint_info': ../../src/gdb/gdbtk/generic/gdbtk-bp.c:776: warning: assignment from incompatible pointer type make[2]: *** [gdbtk-bp.o] Error 1 make[2]: Leaving directory `/home/teawater/gdb/cvs/bgdball/gdb' make[1]: *** [all-gdb] Error 2 make[1]: Leaving directory `/home/teawater/gdb/cvs/bgdball' make: *** [all] Error 2 On Fri, Mar 19, 2010 at 17:01, Hui Zhu <teawater@gmail.com> wrote: > It looks like another issue. > > Does this patch can check in? > > Thanks, > Hui > > On Thu, Mar 18, 2010 at 12:58, Gene Smith <gds@chartertn.net> wrote: >> Keith Seitz wrote, On 03/17/2010 01:57 PM: >>> >>> 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 >> >> Built OK on/for f12-64bit native. Tried my build for mingw32 (using f12 >> cross compiler pkg that works OK for insight 6.8) and see this error when >> entering the itcl directory and apparently trying to "make all": >> >> i686-pc-mingw32-windres -o wish.res.o --include >> "../../../insight_sources/tk/win/../generic" --include >> "/home/gene/addons/insight-cvs/insight_sources/tcl/generic" --include >> "../../../insight_sources/tk/win/rc" >> ../../../insight_sources/tk/win/rc/wish.rc >> i686-pc-mingw32-gcc -g -O2 -D__USE_MINGW_ACCESS winMain.o >> "/home/gene/addons/insight-cvs/insight_build/tcl/win/libtcl84.a" libtk84.a >> -lgdi32 -lcomdlg32 -limm32 -lcomctl32 -lshell32 \ >> wish.res.o -o wish84.exe -mwindows -mwin32 >> make[3]: Leaving directory >> `/home/gene/addons/insight-cvs/insight_build/tk/win' >> make[2]: Leaving directory `/home/gene/addons/insight-cvs/insight_build/tk' >> make[2]: Entering directory >> `/home/gene/addons/insight-cvs/insight_build/itcl' >> Making all in itcl >> make[3]: Entering directory >> `/home/gene/addons/insight-cvs/insight_build/itcl/itcl' >> make[3]: *** No rule to make target `all'. Stop. >> make[3]: Leaving directory >> `/home/gene/addons/insight-cvs/insight_build/itcl/itcl' >> Making all in itk >> make[3]: Entering directory >> `/home/gene/addons/insight-cvs/insight_build/itcl/itk' >> make[3]: *** No rule to make target `all'. Stop. >> make[3]: Leaving directory >> `/home/gene/addons/insight-cvs/insight_build/itcl/itk' >> Making all in iwidgets >> make[3]: Entering directory >> `/home/gene/addons/insight-cvs/insight_build/itcl/iwidgets' >> make[3]: *** No rule to make target `all'. Stop. >> make[3]: Leaving directory >> `/home/gene/addons/insight-cvs/insight_build/itcl/iwidgets' >> make[3]: Entering directory >> `/home/gene/addons/insight-cvs/insight_build/itcl' >> make[3]: Nothing to be done for `all-am'. >> make[3]: Leaving directory >> `/home/gene/addons/insight-cvs/insight_build/itcl' >> make[2]: *** [all-recursive] Error 1 >> make[2]: Leaving directory >> `/home/gene/addons/insight-cvs/insight_build/itcl' >> make[1]: *** [all-itcl] Error 2 >> make[1]: Leaving directory `/home/gene/addons/insight-cvs/insight_build' >> make: *** [all] Error 2 >> >> Configured like this: >> ./configure -v --quiet --prefix=$PREFIX \ >> --target=$TARGET --enable-interwork --enable-multilib \ >> --with-gnu-ld --with-gnu-as \ >> --disable-werror \ >> --host=i686-pc-mingw32 --build=i686-pc-linux-gnu \ >> CC="i686-pc-mingw32-gcc" >> >> >> >> >> >> > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: undefined reference to `set_breakpoint' 2010-03-30 8:29 ` Hui Zhu @ 2010-03-30 16:48 ` Hui Zhu 2010-03-31 3:20 ` Keith Seitz 0 siblings, 1 reply; 13+ messages in thread From: Hui Zhu @ 2010-03-30 16:48 UTC (permalink / raw) To: Keith Seitz; +Cc: insight Ping. Did I miss something? Thanks, Hui On Thu, Mar 25, 2010 at 15:06, Hui Zhu <teawater@gmail.com> wrote: > gcc -g -O2 -I. -I../../src/gdb -I../../src/gdb/common > -I../../src/gdb/config -DLOCALEDIR="\"/usr/local/share/locale\"" > -DHAVE_CONFIG_H -I../../src/gdb/../include/opcode > -I../../src/gdb/../opcodes/.. -I../../src/gdb/../readline/.. -I../bfd > -I../../src/gdb/../bfd -I../../src/gdb/../include -I../libdecnumber > -I../../src/gdb/../libdecnumber -I../../src/gdb/gnulib -Ignulib > -DMI_OUT=1 -DTUI=1 -DGDBTK -Wall -Wdeclaration-after-statement > -Wpointer-arith -Wformat-nonliteral -Wno-pointer-sign -Wno-unused > -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts > -Werror -c -o gdbtk-bp.o -MT gdbtk-bp.o -MMD -MP -MF > .deps/gdbtk-bp.Tpo -I../../src/gdb/../libgui/src > -I/home/teawater/gdb/cvs/src/tcl/generic > -I/home/teawater/gdb/cvs/src/tk/generic > -DGDBTK_LIBRARY=\"/usr/local/share/insight1.0\" > -DSRC_DIR=\"/home/teawater/gdb/cvs/src/gdb\" > ../../src/gdb/gdbtk/generic/gdbtk-bp.c > cc1: warnings being treated as errors > ../../src/gdb/gdbtk/generic/gdbtk-bp.c: In function 'gdb_get_breakpoint_info': > ../../src/gdb/gdbtk/generic/gdbtk-bp.c:365: warning: passing argument > 1 of 'get_breakpoint_commands' from incompatible pointer type > ../../src/gdb/gdbtk/generic/gdbtk-bp.c: In function 'gdb_actions_command': > ../../src/gdb/gdbtk/generic/gdbtk-bp.c:688: warning: passing argument > 1 of 'free_command_lines' from incompatible pointer type > ../../src/gdb/gdbtk/generic/gdbtk-bp.c: In function 'gdb_get_tracepoint_info': > ../../src/gdb/gdbtk/generic/gdbtk-bp.c:776: warning: assignment from > incompatible pointer type > make[2]: *** [gdbtk-bp.o] Error 1 > make[2]: Leaving directory `/home/teawater/gdb/cvs/bgdball/gdb' > make[1]: *** [all-gdb] Error 2 > make[1]: Leaving directory `/home/teawater/gdb/cvs/bgdball' > make: *** [all] Error 2 > > > On Fri, Mar 19, 2010 at 17:01, Hui Zhu <teawater@gmail.com> wrote: >> It looks like another issue. >> >> Does this patch can check in? >> >> Thanks, >> Hui >> >> On Thu, Mar 18, 2010 at 12:58, Gene Smith <gds@chartertn.net> wrote: >>> Keith Seitz wrote, On 03/17/2010 01:57 PM: >>>> >>>> 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 >>> >>> Built OK on/for f12-64bit native. Tried my build for mingw32 (using f12 >>> cross compiler pkg that works OK for insight 6.8) and see this error when >>> entering the itcl directory and apparently trying to "make all": >>> >>> i686-pc-mingw32-windres -o wish.res.o --include >>> "../../../insight_sources/tk/win/../generic" --include >>> "/home/gene/addons/insight-cvs/insight_sources/tcl/generic" --include >>> "../../../insight_sources/tk/win/rc" >>> ../../../insight_sources/tk/win/rc/wish.rc >>> i686-pc-mingw32-gcc -g -O2 -D__USE_MINGW_ACCESS winMain.o >>> "/home/gene/addons/insight-cvs/insight_build/tcl/win/libtcl84.a" libtk84.a >>> -lgdi32 -lcomdlg32 -limm32 -lcomctl32 -lshell32 \ >>> wish.res.o -o wish84.exe -mwindows -mwin32 >>> make[3]: Leaving directory >>> `/home/gene/addons/insight-cvs/insight_build/tk/win' >>> make[2]: Leaving directory `/home/gene/addons/insight-cvs/insight_build/tk' >>> make[2]: Entering directory >>> `/home/gene/addons/insight-cvs/insight_build/itcl' >>> Making all in itcl >>> make[3]: Entering directory >>> `/home/gene/addons/insight-cvs/insight_build/itcl/itcl' >>> make[3]: *** No rule to make target `all'. Stop. >>> make[3]: Leaving directory >>> `/home/gene/addons/insight-cvs/insight_build/itcl/itcl' >>> Making all in itk >>> make[3]: Entering directory >>> `/home/gene/addons/insight-cvs/insight_build/itcl/itk' >>> make[3]: *** No rule to make target `all'. Stop. >>> make[3]: Leaving directory >>> `/home/gene/addons/insight-cvs/insight_build/itcl/itk' >>> Making all in iwidgets >>> make[3]: Entering directory >>> `/home/gene/addons/insight-cvs/insight_build/itcl/iwidgets' >>> make[3]: *** No rule to make target `all'. Stop. >>> make[3]: Leaving directory >>> `/home/gene/addons/insight-cvs/insight_build/itcl/iwidgets' >>> make[3]: Entering directory >>> `/home/gene/addons/insight-cvs/insight_build/itcl' >>> make[3]: Nothing to be done for `all-am'. >>> make[3]: Leaving directory >>> `/home/gene/addons/insight-cvs/insight_build/itcl' >>> make[2]: *** [all-recursive] Error 1 >>> make[2]: Leaving directory >>> `/home/gene/addons/insight-cvs/insight_build/itcl' >>> make[1]: *** [all-itcl] Error 2 >>> make[1]: Leaving directory `/home/gene/addons/insight-cvs/insight_build' >>> make: *** [all] Error 2 >>> >>> Configured like this: >>> ./configure -v --quiet --prefix=$PREFIX \ >>> --target=$TARGET --enable-interwork --enable-multilib \ >>> --with-gnu-ld --with-gnu-as \ >>> --disable-werror \ >>> --host=i686-pc-mingw32 --build=i686-pc-linux-gnu \ >>> CC="i686-pc-mingw32-gcc" >>> >>> >>> >>> >>> >>> >> > ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: undefined reference to `set_breakpoint' 2010-03-30 16:48 ` Hui Zhu @ 2010-03-31 3:20 ` Keith Seitz 2010-03-31 4:12 ` Hui Zhu 0 siblings, 1 reply; 13+ messages in thread From: Keith Seitz @ 2010-03-31 3:20 UTC (permalink / raw) To: Hui Zhu; +Cc: insight [-- Attachment #1: Type: text/plain, Size: 748 bytes --] On 03/30/2010 01:29 AM, Hui Zhu wrote: > Ping. Did I miss something? > Nope -- I did. After the breakpoint commands API was fiddled with, I worked up the original patch. Some (brief) time after committing my patch, the API was fiddled with again. I apologize about the confusion. Try this patch, which I've committed. Keith ChangeLog 2010-03-30 Keith Seitz <keiths@redhat.com> * generic/gdbtk-bp.c (gdb_get_breakpoint_info): Breakpoint commands are now reference counted. Adjust data pointers. (gdb_get_tracepoint_info): Likewise. (gdb_actions_command): Don't free previous actions. If the commands aren't valid, we would prefer to keep the old ones any way. break_point_set_commands will also clobber any previous results anyway. [-- Attachment #2: more-bp-cmd-fixes.patch --] [-- Type: text/plain, Size: 1678 bytes --] Index: generic/gdbtk-bp.c =================================================================== RCS file: /cvs/src/src/gdb/gdbtk/generic/gdbtk-bp.c,v retrieving revision 1.34 diff -u -p -r1.34 gdbtk-bp.c --- generic/gdbtk-bp.c 24 Mar 2010 20:18:28 -0000 1.34 +++ generic/gdbtk-bp.c 30 Mar 2010 16:45:30 -0000 @@ -362,7 +362,7 @@ gdb_get_breakpoint_info (ClientData clie Tcl_NewIntObj (b->ignore_count)); Tcl_ListObjAppendElement (NULL, result_ptr->obj_ptr, - get_breakpoint_commands (b->commands)); + get_breakpoint_commands (b->commands ? b->commands->commands : NULL)); Tcl_ListObjAppendElement (NULL, result_ptr->obj_ptr, Tcl_NewStringObj (b->cond_string, -1)); @@ -684,9 +684,6 @@ gdb_actions_command (ClientData clientDa return TCL_ERROR; } - /* Free any existing actions. */ - free_command_lines (&tp->commands); - /* Validate and set new tracepoint actions. */ Tcl_ListObjGetElements (interp, objv[2], &gdbtk_obj_array_cnt, &gdbtk_obj_array); @@ -773,12 +770,15 @@ gdb_get_tracepoint_info (ClientData clie /* Append a list of actions */ action_list = Tcl_NewObj (); - for (cl = tp->commands; cl != NULL; cl = cl->next) + if (tp->commands != NULL) { - Tcl_ListObjAppendElement (interp, action_list, - Tcl_NewStringObj (cl->line, -1)); + for (cl = tp->commands->commands; cl != NULL; cl = cl->next) + { + Tcl_ListObjAppendElement (interp, action_list, + Tcl_NewStringObj (cl->line, -1)); + } + Tcl_ListObjAppendElement (interp, result_ptr->obj_ptr, action_list); } - Tcl_ListObjAppendElement (interp, result_ptr->obj_ptr, action_list); return TCL_OK; } ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: undefined reference to `set_breakpoint' 2010-03-31 3:20 ` Keith Seitz @ 2010-03-31 4:12 ` Hui Zhu 2010-03-31 4:28 ` Keith Seitz 0 siblings, 1 reply; 13+ messages in thread From: Hui Zhu @ 2010-03-31 4:12 UTC (permalink / raw) To: Keith Seitz; +Cc: insight [-- Attachment #1: Type: text/plain, Size: 1047 bytes --] On Wed, Mar 31, 2010 at 00:48, Keith Seitz <keiths@redhat.com> wrote: > On 03/30/2010 01:29 AM, Hui Zhu wrote: >> >> Ping. Did I miss something? >> > > Nope -- I did. After the breakpoint commands API was fiddled with, I worked > up the original patch. Some (brief) time after committing my patch, the API > was fiddled with again. > > I apologize about the confusion. Try this patch, which I've committed. > > Keith > > ChangeLog > 2010-03-30 Keith Seitz <keiths@redhat.com> > > * generic/gdbtk-bp.c (gdb_get_breakpoint_info): Breakpoint > commands are now reference counted. Adjust data pointers. > (gdb_get_tracepoint_info): Likewise. > (gdb_actions_command): Don't free previous actions. If the > commands aren't valid, we would prefer to keep the old ones > any way. break_point_set_commands will also clobber any > previous results anyway. > Hi Keith, I try this patch bug cannot patch to cvs-head. The attachment is the rej file that I got. Thanks, Hui [-- Attachment #2: gdbtk-bp.c.rej --] [-- Type: application/x-reject, Size: 2002 bytes --] ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: undefined reference to `set_breakpoint' 2010-03-31 4:12 ` Hui Zhu @ 2010-03-31 4:28 ` Keith Seitz 2010-03-31 5:33 ` Hui Zhu 0 siblings, 1 reply; 13+ messages in thread From: Keith Seitz @ 2010-03-31 4:28 UTC (permalink / raw) To: Hui Zhu; +Cc: insight On 03/30/2010 08:19 PM, Hui Zhu wrote: > I try this patch bug cannot patch to cvs-head. The attachment is the > rej file that I got. Wow, it looks like the whole patch was rejected! I've committed this patch to CVS HEAD: can you pull it into your tree from there? Keith ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: undefined reference to `set_breakpoint' 2010-03-31 4:28 ` Keith Seitz @ 2010-03-31 5:33 ` Hui Zhu 2010-03-31 5:37 ` Keith Seitz 0 siblings, 1 reply; 13+ messages in thread From: Hui Zhu @ 2010-03-31 5:33 UTC (permalink / raw) To: Keith Seitz; +Cc: insight On Wed, Mar 31, 2010 at 12:12, Keith Seitz <keiths@redhat.com> wrote: > On 03/30/2010 08:19 PM, Hui Zhu wrote: >> >> I try this patch bug cannot patch to cvs-head. The attachment is the >> rej file that I got. > > Wow, it looks like the whole patch was rejected! > > I've committed this patch to CVS HEAD: can you pull it into your tree from > there? > > Keith > Hi Keith, I tried to patch this patch to cvs-head that already have this patch. Sorry for my mistake. My part is OK now. Thanks for your work. Best, Hui ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: undefined reference to `set_breakpoint' 2010-03-31 5:33 ` Hui Zhu @ 2010-03-31 5:37 ` Keith Seitz 0 siblings, 0 replies; 13+ messages in thread From: Keith Seitz @ 2010-03-31 5:37 UTC (permalink / raw) To: Hui Zhu; +Cc: insight On 03/30/2010 10:25 PM, Hui Zhu wrote: > I tried to patch this patch to cvs-head that already have this patch. That would certainly explain why the whole patch failed to apply! :-) Keith ^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2010-03-31 5:33 UTC | newest] Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2010-03-17 17:34 undefined reference to `set_breakpoint' Gene Smith 2010-03-17 17:57 ` Tom Tromey 2010-03-18 3:39 ` Keith Seitz 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
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).