----- Original Message -----
> From: "Bhanuprakash Bodireddy" <[email protected]>
> To: [email protected]
> Sent: Monday, June 27, 2016 12:11:40 PM
> Subject: [ovs-dev] [PATCH] Makefile.am: Add clang static analysis support
>
> Clang Static Analyzer is a source code analysis tool to find bugs. This
> patch adds make target to trigger static analysis using below commands.
>
> ./boot.sh
> ./configure --with-dpdk(for configuring DPDK datapath)
> make clang-analyze
> scan-view --host=<ip address> --port <PORT>
> $OVS_DIR>/clang-analyzer-results/yyyy-mm-dd-114251-1027-1>
> --allow-all-hosts
>
> Results can be viewed on browser: http://<ip address>:<PORT>/
>
> Signed-off-by: Bhanuprakash Bodireddy <[email protected]>
> ---
> Makefile.am | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/Makefile.am b/Makefile.am
> index 8cb8523..ac96be6 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -400,6 +400,16 @@ ovsext_clean: datapath-windows/ovsext.sln
> endif
> .PHONY: ovsext
>
> +clang-analyze: clean
> + @if which clang scan-build > /dev/null 2>&1; then \
> + $(MKDIR_P) "$(srcdir)/clang-analyzer-results" || exit 1; \
> + scan-build -o $(srcdir)/clang-analyzer-results
> --use-analyzer=/usr/bin/clang \
> + make -j || exit 1; \
> + else \
> + echo -e "Unable to find clang/scan-build, Install clang,clang-analyzer
> packages"; \
> + fi
> +.PHONY: clang-analyze
> +
> dist-hook: $(DIST_HOOKS)
> all-local: $(ALL_LOCAL)
> clean-local: $(CLEAN_LOCAL)
> --
> 2.4.11
>
> _______________________________________________
> dev mailing list
> [email protected]
> http://openvswitch.org/mailman/listinfo/dev
>
LGTM, I tried it out with no issues found.
a couple of small suggestions:
- It would be good to add some text to the "Build Requirements" section of
INSTALL.md, mentioning clang-analyzer where clang is already listed.
- It might also be nice to have a sentence or two somewhere in INSTALL.md
about
how to use this feature.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
