The clean option just needs to remove the files cscope.in.out & cscope.po.out as I am using the indexed search option which makes the lookup quicker. With non-indexed searches, the file is cscope.out. Try it out, its pretty quick. So, does not become a bottleneck.
Having an option in the makefile is better than not having anything. People may not trigger it if they don't like it. And the clean operation Is harmless. As for me, I don't know ctags nor etags so using emacs Is difficult. Very cryptic commands. So, ctags or scope is the option. The problem I have not been able to figure out is backward search.
I find this useful for two things :
1. Code tracing. 2. Bug fixing when tracking errors by using make.
This way, the user does not have to switch directories, make code change And re-compile. Option#1 only works from top level directory. i.e. if I am in directory three level deep and want to see a function prototype in A directory at 1st level, it does not pick it up.
Apart from that, its really a good thing to integrate for people who work In text mode.
-- Atul
-----Original Message----- From: Detlev Zundel [mailto:dzu@denx.de] Sent: Monday, October 24, 2005 2:17 AM To: Sabharwal, Atul Cc: frannk_m1@yahoo.com; u-boot-users@lists.sourceforge.net Subject: Re: Patch to build csope database in the makefile for u-boot-1.1.3
Hi,
Re-submit. Accessing the file over a Samba share even after UNIX to DOS conversion corrupted the file. Patch tested against latest cscope from Sourceforge. (had some problem with clipboard or file transfer over SAMBA. Ftp transfer worked. Also, cut-N-paste between edit and Outlook Did not work. Opening file with notepad and past worked.).
--- u-boot-1.1.3/Makefile 2005-10-19 18:04:22.000000000 -0700 +++ Makefile 2005-10-21 09:51:53.000000000 -0700 @@ -193,6 +193,10 @@ net disk rtc dtt drivers drivers/sk98lin common \ ( -name CVS -prune ) -o ( -name '*.[ch]' -print )`
+cscope:
find . -name "*.[cCsSh]" -print > cscope.filescscope -b -R -u -qSystem.map: u-boot $(NM) -A -l $< | \ grep -v '(compiled)|(.o$$)|( [aUw] )|(..ng$$)|(LASH[RL]DI)' | \
As stated in my other mail, I do not like that target picking up every source file - think of the function "checkboard" for example.
But apart from this the "clobber" rule of the Makefile should at least be extended to also clean up the cscope files.
Cheers Detlev