[dpdk-dev] [PATCH v2] tools: add tags and cscope index file generation support
Thomas Monjalon
thomas.monjalon at 6wind.com
Mon Feb 27 15:18:52 CET 2017
2017-01-17 14:11, Jerin Jacob:
> 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=)
What is the purpose of selecting a configuration?
Is it to go quicker in the implementation you are interested in?
In that case, I think we need a catch-all option, because I like
being prompted by vim that several implementations exist and I can
choose one of them.
> 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.jacob at caviumnetworks.com>
> Reviewed-by: Yuanhan Liu <yuanhan.liu at linux.intel.com>
> Reviewed-by: Ferruh Yigit <ferruh.yigit at intel.com>
[...]
> .gitignore | 8 ++
> devtools/tags.sh | 251 ++++++++++++++++++++++++++++++++++++++++++++++++++++++
> mk/rte.sdkroot.mk | 4 +
> 3 files changed, 263 insertions(+)
I think build-tags.sh would be a better name.
On the implementation, I have few comments:
- is there a way to re-use the skip list when including the related files?
- you can remove tile from this patch
And for the details:
- why a bash shebang where /bin/sh would be wider?
- verbose option should be -v (with getopts)
- please use $() instead of backquotes
- please avoid one-line functions used only once like doctags
I can help you with this script if needed.
More information about the dev
mailing list