> -----Original Message-----
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Jerin Jacob
> Sent: Friday, January 13, 2017 2:50 AM
> To: Yigit, Ferruh <ferruh.yi...@intel.com>
> Cc: dev@dpdk.org; thomas.monja...@6wind.com
> Subject: Re: [dpdk-dev] [PATCH] tools: add tags and cscope index file
> generation support
> 
> On Thu, Jan 12, 2017 at 02:19:44PM +0000, Ferruh Yigit wrote:
> > On 11/27/2016 12:12 AM, Jerin Jacob wrote:
> > > This script generates cscope, gtags, and tags index files based on
> > > EAL environment.
> > > (architecture and OS(linux/bsd))
> > >
> > > Selection of the architecture and OS environment is based on dpdk
> > > configuration target(T=)
> > >
> > > example usage:
> > > make tags T=x86_64-native-linuxapp-gcc make cscope
> > > T=x86_64-native-linuxapp-gcc make gtags T=x86_64-native-linuxapp-gcc
> > >
> > > Signed-off-by: Jerin Jacob <jerin.ja...@caviumnetworks.com>
> >
> > <...>
> >
> > > +
> > > +if [ -n "$T" ]; then
> > <...>
> > > + usage $1
> > > + exit
> >
> > I would prefer fallback to RTE_TARGET parsing, or processing all files
> > ( cscope -Rqn ), instead of exiting here, but that is your call.
> 
> I would prefer avoid any fallback(sometimes RTE_TARGET can be "build"
> also) and print the usage and exit error if the requirements are not meet.
> 
> >
> >
> > Also "tags.sh" needs to be moved into "devtools" after latest updates.
> 
> Sure. I will send v2 to address this.

Very handy. +1.

Could you also add an etags target, for Emacs. Either separately like the 
gtags target or just add it to the tags target:
 
    doetags()
    {
        all_sources | xargs etags -a
    }
    
    # or:

    doctags()
    {
        all_sources | xargs ctags -a
        all_sources | xargs etags -a
    }

It create a TAGS file which should be added to the gitignore rules.

Thanks,

John


Reply via email to