Misc cleanups, add installer for lint-groovy
makefiles/npm/README.md
makefiles/npm/include.mk
makefiles/npm/nvm.mk
makefiles/npm/npm.mk
-------------------------
o Genreate targets and deps for installing tools nvm, npm and npm-groovy-lint.
o Tools are intended for interactive use.
o npm-groovy-lint is a dependency for the lint-groovy target
config.mk
.gitignore
----------
o Add exclusion lf/onf-make for repo:onf-make git submodule.
o Add exclusion for .tmp (makefiles/nvm/nvm.mk)
makefiles/lint/groovy/groovy.mk
makefiles/lint/groovy/include.mk
makefiles/lint/groovy/install.mk
--------------------------------
o Update targets to latest lint-{language}-{all,mod,src} syntax.
o Installer updated to depend on npm/nvm.mk target 'npm-groovy-install'.
makefiles/lint/make/makefile.mk
makefiles/lint/json.mk
makefiles/lint/python/flake8.mk
makefiles/lint/shell/shellcheck.mk
makefiles/lint/yaml.mk
------------------------
o Simplify find command exclusions.
o Use "-not -path [p]" in place of "( -name p ) -prune".
makefiles/main/tempdir/tempdir.mk
---------------------------------
o Fixed a typo $(HELP) -> $(HIDE)
makefiles/lint/doc8/doc8.ini
-------------------------------
o Add directory exclusions from onf-excl-dir=.
o Remove txt from the list of file extensions to check
repo:voltha-docs is the only repo with *.txt files that contain
restructuredText so add the extension in the local per-repository
config.
Signed-off-by: Joey Armstrong <jarmstrong@linuxfoundation.org>
Change-Id: Ia2e091d812af82063c79ed855bc7dd4f5ec8f706
diff --git a/makefiles/lint/yaml/include.mk b/makefiles/lint/yaml/include.mk
index 28fe608..7518d51 100644
--- a/makefiles/lint/yaml/include.mk
+++ b/makefiles/lint/yaml/include.mk
@@ -28,7 +28,7 @@
##---] INCLUDES [---##
##--------------------##
include $(ONF_MAKEDIR)/lint/yaml/help.mk
-include $(ONF_MAKEDIR)/lint/yaml/find_utils.mk
+# include $(ONF_MAKEDIR)/lint/yaml/find_utils.mk
include $(ONF_MAKEDIR)/lint/yaml/install.mk
# [TODO] Consolidate and refactor to support a simpler answer
diff --git a/makefiles/lint/yaml/yamllint.mk b/makefiles/lint/yaml/yamllint.mk
index ed0df2c..f8c2f7f 100644
--- a/makefiles/lint/yaml/yamllint.mk
+++ b/makefiles/lint/yaml/yamllint.mk
@@ -33,6 +33,11 @@
YAMLLINT = $(activate) && yamllint
yamllint-args += --strict
+yaml-find-args := $(null)
+yaml-find-args += $(foreach dir,$(onf-excl-dirs),-not -path './$(dir)/*')
+yaml-find-args += -a \( -iname '*.yaml' -o -iname '*.yml' \)
+yaml-find-args += -print0
+
## -----------------------------------------------------------------------
## Intent: Use the yaml command to perform syntax checking.
## -----------------------------------------------------------------------
@@ -49,7 +54,8 @@
$(call banner-enter,Target $@)
$(HIDE)$(MAKE) --no-print-directory lint-yaml-install
- $(HIDE)$(activate) && $(call gen-yaml-find-cmd) \
+
+ $(HIDE)$(activate) && find . $(yaml-find-args) \
| $(env-clean) $(xargs-cmd) -I'{}' \
bash -c "$(YAMLLINT) $(yamllint-args) {}"
$(call banner-leave,Target $@)