public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Pete Wyckoff <pw@osc.edu>
To: Linux Kernel <linux-kernel@vger.kernel.org>
Subject: Re: Makefile patch for cscope and saner Ctags
Date: Thu, 31 May 2001 13:45:30 -0400	[thread overview]
Message-ID: <20010531134530.A15302@osc.edu> (raw)
In-Reply-To: <20010530180232.A4546@somanetworks.com>
In-Reply-To: <20010530180232.A4546@somanetworks.com>; from mark@somanetworks.com on Wed, May 30, 2001 at 06:02:32PM -0400

mark@somanetworks.com said:
> The following patch generates saner Ctags and will build cscope
> output.  It's against 2.4.5
> 
> --- Makefile.old	Mon May 28 22:44:01 2001
> +++ Makefile	Wed May 30 17:50:01 2001
> @@ -334,11 +334,32 @@
>  
>  # Exuberant ctags works better with -I
>  tags: dummy
> -	CTAGSF=`ctags --version | grep -i exuberant >/dev/null && echo "-I __initdata,__exitdata,EXPORT_SYMBOL,EXPORT_SYMBOL_NOVERS"`; \
> +	CTAGSF=`ctags --version | grep -i exuberant >/dev/null && echo "--sort=no -I __initdata,__exitdata,EXPORT_SYMBOL,EXPORT_SYMBOL_NOVERS"`; \
>  	ctags $$CTAGSF `find include/asm-$(ARCH) -name '*.h'` && \
> -	find include -type d \( -name "asm-*" -o -name config \) -prune -o -name '*.h' -print | xargs ctags $$CTAGSF -a && \
> +	find include -type f -name '*.h' -mindepth 2 -maxdepth 2 \
> +	    | grep -v include/asm- | grep -v include/config \
> +	    | xargs -r ctags $$CTAGSF -a && \
> +	find include -type f -name '*.h' -mindepth 3 -maxdepth 3 \
> +	    | grep -v include/asm- | grep -v include/config \
> +	    | xargs -r ctags $$CTAGSF -a && \
> +	find include -type f -name '*.h' -mindepth 4 -maxdepth 4 \
> +	    | grep -v include/asm- | grep -v include/config \
> +	    | xargs -r ctags $$CTAGSF -a && \
> +	find include -type f -name '*.h' -mindepth 5 -maxdepth 5 \
> +	    | grep -v include/asm- | grep -v include/config \
> +	    | xargs -r ctags $$CTAGSF -a && \
>  	find $(SUBDIRS) init -name '*.c' | xargs ctags $$CTAGSF -a
> +	mv tags tags.unsorted
> +	LC_ALL=C sort -k 1,1 -s tags.unsorted > tags
> +	rm tags.unsorted
>  
> +cscope: dummy
> +	find include/asm-$(ARCH) -name '*.h' >cscope.files
> +	find include $(SUBDIRS) init -type f -name '*.[ch]' \
> +	    | grep -v include/asm- | grep -v include/config >> cscope.files
> +	cscope -b -I include
> +
> +	
>  ifdef CONFIG_MODULES
>  ifdef CONFIG_MODVERSIONS
>  MODFLAGS += -DMODVERSIONS -include $(HPATH)/linux/modversions.h

You seem not to have read my response to your earlier mail proprosing
such a thing (for tags only, not cscope):

    http://boudicca.tux.org/hypermail/linux-kernel/2001week21/1869.html

How does the patch above fix anything?  You're sorting so that
include/linux/*.h comes before include/linux/{mtd,lockd,raid,...}/*.h,
but I don't see how that can be an improvement, or how it addresses
your original complaint "ctags doesn't honour any CPP #if'ing".

		-- Pete

  reply	other threads:[~2001-05-31 17:45 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-05-30 22:02 Makefile patch for cscope and saner Ctags Mark Frazer
2001-05-31 17:45 ` Pete Wyckoff [this message]
2001-05-31 18:17   ` Mark Frazer
  -- strict thread matches above, loose matches on Subject: below --
2001-05-31 16:11 Khachaturov, Vassilii
2001-05-31 18:06 ` Mark Frazer
2001-05-31 18:56   ` george anzinger
2001-05-31 18:52 Khachaturov, Vassilii
2001-05-31 20:44 ` Mark Frazer
2001-06-01 18:39 Khachaturov, Vassilii

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=20010531134530.A15302@osc.edu \
    --to=pw@osc.edu \
    --cc=linux-kernel@vger.kernel.org \
    /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