All of lore.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.