From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 19918 invoked by alias); 21 Sep 2005 14:36:59 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 19835 invoked by uid 22791); 21 Sep 2005 14:36:32 -0000 Received: from qnxmail.qnx.com (HELO nimbus.ott.qnx.com) (209.226.137.76) by sourceware.org (qpsmtpd/0.30-dev) with ESMTP; Wed, 21 Sep 2005 14:36:32 +0000 Received: from [10.42.102.103] (STIMPY [10.42.102.103]) by nimbus.ott.qnx.com with SMTP (Microsoft Exchange Internet Mail Service Version 5.5.2653.13) id R7M0FSDG; Wed, 21 Sep 2005 10:36:30 -0400 Message-ID: <43317105.9040201@qnx.com> Date: Wed, 21 Sep 2005 14:36:00 -0000 From: Kris Warkentin User-Agent: Mozilla Thunderbird 1.0 (Windows/20041206) MIME-Version: 1.0 To: Nick Roberts CC: gdb-patches@sources.redhat.com Subject: Re: [PATCH] Get make TAGS to work in gdb directory References: <17200.36038.631429.268643@farnswood.snap.net.nz> In-Reply-To: <17200.36038.631429.268643@farnswood.snap.net.nz> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2005-09/txt/msg00188.txt.bz2 Wow! I had no idea that even existed. I always just did ctags `find . -name "*.[ch]"`. I suppose that the make TAGS is smart enough to only generate tags for the source files that have been configured for your target? How handy! cheers, Kris Nick Roberts wrote: > > make TAGS doesn't seem to work in the gdb directory for two reason: > > 1) minimon.h, symfile-mem.h, amdfbsd-nat.c, irix4-nat.c, m3-nat.c, > mipsm3-nat.c, ns32k-tdep.c, symm-tdep.c and symm-nat.c have been > removed. > > 2) It doesn't find mi-*.c tui-*.c because they are in different > directories. > > > It's been bust for quite a long time so either people don't use it much, > they have a local patch or there's something wrong with my setup. > Assuming > its not the latter, I attach a patch. > > Nick > > > 2005-09-21 Nick Roberts > > * Makefile.in (HFILES_NO_SRCDIR): Remove minimon.h, > symfile-mem.h, > amdfbsd-nat.c, irix4-nat.c, m3-nat.c, mipsm3-nat.c, ns32k-tdep.c, > symm-tdep.c and symm-nat.c. > (TAGFILES_NO_SRCDIR): Add SUBDIR_MI_SRCS SUBDIR_TUI_SRCS. > (DEPFILES): Remove CONFIG_OBS and include it explicitly where > needed. > > > *** /home/nickrob/src/gdb/Makefile.in.~1.753.~ 2005-09-16 > 18:00:51.000000000 +1200 > --- /home/nickrob/src/gdb/Makefile.in 2005-09-21 09:44:57.617994472 > +1200 > *************** > *** 852,860 **** > HFILES_NO_SRCDIR = bcache.h buildsym.h call-cmds.h coff-solib.h > defs.h \ > exceptions.h \ > environ.h $(gdbcmd_h) gdb.h gdbcore.h \ > ! gdb-stabs.h $(inferior_h) language.h minimon.h monitor.h \ > objfiles.h parser-defs.h serial.h solib.h \ > ! symfile.h symfile-mem.h stabsread.h target.h terminal.h > typeprint.h \ > xcoffsolib.h \ > macrotab.h macroexp.h macroscope.h \ > ada-lang.h c-lang.h f-lang.h \ > --- 852,860 ---- > HFILES_NO_SRCDIR = bcache.h buildsym.h call-cmds.h coff-solib.h > defs.h \ > exceptions.h \ > environ.h $(gdbcmd_h) gdb.h gdbcore.h \ > ! gdb-stabs.h $(inferior_h) language.h monitor.h \ > objfiles.h parser-defs.h serial.h solib.h \ > ! symfile.h stabsread.h target.h terminal.h typeprint.h \ > xcoffsolib.h \ > macrotab.h macroexp.h macroscope.h \ > ada-lang.h c-lang.h f-lang.h \ > *************** > *** 881,896 **** > # Makefile.in > > DEPFILES = $(TDEPFILES) $(SER_HARDWIRE) $(NATDEPFILES) \ > ! $(REMOTE_OBS) $(SIM_OBS) $(CONFIG_OBS) > > SOURCES = $(SFILES) $(ALLDEPFILES) $(YYFILES) $(CONFIG_SRCS) > # Don't include YYFILES (*.c) because we already include *.y in SFILES, > # and it's more useful to see it in the .y file. > TAGFILES_NO_SRCDIR = $(SFILES) $(HFILES_NO_SRCDIR) $(ALLDEPFILES) \ > ! $(SUBDIR_CLI_SRCS) > TAGFILES_WITH_SRCDIR = $(HFILES_WITH_SRCDIR) > > ! COMMON_OBS = $(DEPFILES) $(YYOBJ) \ > version.o \ > annotate.o \ > auxv.o \ > --- 881,896 ---- > # Makefile.in > > DEPFILES = $(TDEPFILES) $(SER_HARDWIRE) $(NATDEPFILES) \ > ! $(REMOTE_OBS) $(SIM_OBS) > > SOURCES = $(SFILES) $(ALLDEPFILES) $(YYFILES) $(CONFIG_SRCS) > # Don't include YYFILES (*.c) because we already include *.y in SFILES, > # and it's more useful to see it in the .y file. > TAGFILES_NO_SRCDIR = $(SFILES) $(HFILES_NO_SRCDIR) $(ALLDEPFILES) \ > ! $(SUBDIR_CLI_SRCS) $(SUBDIR_MI_SRCS) $(SUBDIR_TUI_SRCS) > TAGFILES_WITH_SRCDIR = $(HFILES_WITH_SRCDIR) > > ! COMMON_OBS = $(DEPFILES) $(CONFIG_OBJS) $(YYOBJ) \ > version.o \ > annotate.o \ > auxv.o \ > *************** > *** 1232,1238 **** > @etags $(srcdir)/$(DEPRECATED_TM_FILE) \ > $(srcdir)/$(XM_FILE) \ > $(srcdir)/$(NAT_FILE) \ > ! `(for i in $(DEPFILES) $(TAGFILES_NO_SRCDIR); do \ > echo $(srcdir)/$$i ; \ > done ; for i in $(TAGFILES_WITH_SRCDIR); do \ > echo $$i ; \ > --- 1232,1238 ---- > @etags $(srcdir)/$(DEPRECATED_TM_FILE) \ > $(srcdir)/$(XM_FILE) \ > $(srcdir)/$(NAT_FILE) \ > ! `(for i in $(TAGFILES_NO_SRCDIR); do \ > echo $(srcdir)/$$i ; \ > done ; for i in $(TAGFILES_WITH_SRCDIR); do \ > echo $$i ; \ > *************** > *** 1354,1360 **** > > lint: $(LINTFILES) > $(LINT) $(INCLUDE_CFLAGS) $(LINTFLAGS) $(LINTFILES) \ > ! `echo $(DEPFILES) | sed 's/\.o /\.c /g'` > > gdb.cxref: $(SFILES) > cxref -I. $(SFILES) >gdb.cxref > --- 1354,1360 ---- > > lint: $(LINTFILES) > $(LINT) $(INCLUDE_CFLAGS) $(LINTFLAGS) $(LINTFILES) \ > ! `echo $(DEPFILES) $(CONFIG_OBJS) | sed 's/\.o /\.c /g'` > > gdb.cxref: $(SFILES) > cxref -I. $(SFILES) >gdb.cxref > *************** > *** 1378,1384 **** > alpha-tdep.c alpha-linux-tdep.c alphabsd-tdep.c > alphanbsd-tdep.c \ > alpha-osf1-tdep.c alphafbsd-tdep.c alpha-mdebug-tdep.c \ > amd64-nat.c amd64-tdep.c \ > ! amd64bsd-nat.c amdfbsd-nat.c amd64fbsd-tdep.c \ > amd64nbsd-nat.c amd64nbsd-tdep.c \ > amd64obsd-nat.c amd64obsd-tdep.c \ > amd64-linux-nat.c amd64-linux-tdep.c \ > --- 1378,1384 ---- > alpha-tdep.c alpha-linux-tdep.c alphabsd-tdep.c > alphanbsd-tdep.c \ > alpha-osf1-tdep.c alphafbsd-tdep.c alpha-mdebug-tdep.c \ > amd64-nat.c amd64-tdep.c \ > ! amd64bsd-nat.c amd64fbsd-tdep.c \ > amd64nbsd-nat.c amd64nbsd-tdep.c \ > amd64obsd-nat.c amd64obsd-tdep.c \ > amd64-linux-nat.c amd64-linux-tdep.c \ > *************** > *** 1408,1416 **** > i386gnu-nat.c i386gnu-tdep.c \ > ia64-linux-nat.c ia64-linux-tdep.c ia64-tdep.c \ > inf-ptrace.c inf-ttrace.c \ > ! infptrace.c inftarg.c irix4-nat.c irix5-nat.c \ > libunwind-frame.c \ > ! lynx-nat.c m3-nat.c \ > m68hc11-tdep.c \ > m32r-tdep.c \ > m32r-linux-nat.c m32r-linux-tdep.c \ > --- 1408,1416 ---- > i386gnu-nat.c i386gnu-tdep.c \ > ia64-linux-nat.c ia64-linux-tdep.c ia64-tdep.c \ > inf-ptrace.c inf-ttrace.c \ > ! infptrace.c inftarg.c irix5-nat.c \ > libunwind-frame.c \ > ! lynx-nat.c \ > m68hc11-tdep.c \ > m32r-tdep.c \ > m32r-linux-nat.c m32r-linux-tdep.c \ > *************** > *** 1418,1431 **** > m68kbsd-nat.c m68kbsd-tdep.c \ > m68klinux-nat.c m68klinux-tdep.c \ > m88k-tdep.c m88kbsd-nat.c \ > - mcore-tdep.c \ > mips-linux-nat.c mips-linux-tdep.c \ > mips-irix-tdep.c \ > ! mips-tdep.c mipsm3-nat.c mipsv4-nat.c \ > mipsnbsd-nat.c mipsnbsd-tdep.c \ > mips64obsd-nat.c mips64obsd-tdep.c \ > nbsd-tdep.c \ > ! ns32k-tdep.c solib-osf.c \ > somread.c solib-som.c $(HPREAD_SOURCE) \ > ppc-sysv-tdep.c ppc-linux-nat.c ppc-linux-tdep.c \ > ppcnbsd-nat.c ppcnbsd-tdep.c \ > --- 1418,1430 ---- > m68kbsd-nat.c m68kbsd-tdep.c \ > m68klinux-nat.c m68klinux-tdep.c \ > m88k-tdep.c m88kbsd-nat.c \ > mips-linux-nat.c mips-linux-tdep.c \ > mips-irix-tdep.c \ > ! mips-tdep.c mipsv4-nat.c \ > mipsnbsd-nat.c mipsnbsd-tdep.c \ > mips64obsd-nat.c mips64obsd-tdep.c \ > nbsd-tdep.c \ > ! solib-osf.c \ > somread.c solib-som.c $(HPREAD_SOURCE) \ > ppc-sysv-tdep.c ppc-linux-nat.c ppc-linux-tdep.c \ > ppcnbsd-nat.c ppcnbsd-tdep.c \ > *************** > *** 1447,1453 **** > sparc64-tdep.c sparc64fbsd-nat.c sparc64fbsd-tdep.c \ > sparc64nbsd-nat.c sparc64nbsd-tdep.c sparc64obsd-tdep.c \ > sparcnbsd-nat.c sparcnbsd-tdep.c sparcobsd-tdep.c \ > - symm-tdep.c symm-nat.c \ > v850-tdep.c \ > vax-nat.c vax-tdep.c vaxbsd-nat.c vaxnbsd-tdep.c \ > win32-nat.c \ > --- 1446,1451 ---- > > Diff finished. Wed Sep 21 10:01:41 2005 > -- Stay up-to-date on all the QNX news! Register at http://www.qnx.com/news/forms/newsletter.html to receive our newsletter.