make help target cleanup
Makefile
makefiles/help/include.mk
makefiles/lint/include.mk
makefiles/lint/makefile.mk
makefiles/lint/.........mk
-------------------------
o Implement topic help:: using 3 distinct targets.
o help-summary - displays one-line topic help.
o help-simple - displays common topic help (lint, build, test)
o help-verbose - exhaustive topic help for targets.
o help (default) help-summary + help-simple (see {gerrit,lint}/help.mk)
o Align all help text on string '[target]' from make cmd usage.
o Restore JJB_VERSION ?= 2.8.0 in case of variant conflict with v4.1.0 in use.
makefiles/gerrit/help.mk
makefiles/gerrit/include.mk
---------------------------
o Add gerrit command line convenience targets.
o replication-status target shows health of gerrit-to-github mirror.
jjb/voltha-scale.yaml
---------------------
o Lint cleanups, remove duplicate definitions.
Change-Id: I1c40fc0b5a61e3962e3481f0eae7266c5a6f2b2e
diff --git a/makefiles/lint/yaml/python.mk b/makefiles/lint/yaml/python.mk
index 868d9b7..316afd3 100644
--- a/makefiles/lint/yaml/python.mk
+++ b/makefiles/lint/yaml/python.mk
@@ -1,6 +1,6 @@
# -*- makefile -*-
# -----------------------------------------------------------------------
-# Copyright 2022 Open Networking Foundation (ONF) and the ONF Contributors
+# Copyright 2022-2023 Open Networking Foundation (ONF) and the ONF Contributors
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
@@ -14,7 +14,7 @@
# See the License for the specific language governing permissions and
# limitations under the License.
#
-# SPDX-FileCopyrightText: 2022 Open Networking Foundation (ONF) and the ONF Contributors
+# SPDX-FileCopyrightText: 2022-2023 Open Networking Foundation (ONF) and the ONF Contributors
# SPDX-License-Identifier: Apache-2.0
# -----------------------------------------------------------------------
@@ -32,10 +32,16 @@
lint-yaml: $(venv-activate)
lint-yaml: $(lint-yaml-dep)
+## -----------------------------------------------------------------------
+## Intent: Perform a lint check on yaml sources
+## -----------------------------------------------------------------------
$(lint-yaml-dep):
$(vst-env) && yamllint -s $(call lint-yaml-src,$@)
-help::
- @echo " lint-yaml Syntax check yaml sources"
+## -----------------------------------------------------------------------
+## Intent: Display command help
+## -----------------------------------------------------------------------
+help-summary ::
+ @echo ' lint-yaml Syntax check yaml sources (python -M yaml)'
# [EOF]
diff --git a/makefiles/lint/yaml/yamllint.mk b/makefiles/lint/yaml/yamllint.mk
index 6782c74..dc4e015 100644
--- a/makefiles/lint/yaml/yamllint.mk
+++ b/makefiles/lint/yaml/yamllint.mk
@@ -1,6 +1,6 @@
# -*- makefile -*-
# -----------------------------------------------------------------------
-# Copyright 2022 Open Networking Foundation (ONF) and the ONF Contributors
+# Copyright 2022-2023 Open Networking Foundation (ONF) and the ONF Contributors
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
@@ -14,7 +14,7 @@
# See the License for the specific language governing permissions and
# limitations under the License.
#
-# SPDX-FileCopyrightText: 2022 Open Networking Foundation (ONF) and the ONF Contributors
+# SPDX-FileCopyrightText: 2022-2023 Open Networking Foundation (ONF) and the ONF Contributors
# SPDX-License-Identifier: Apache-2.0
# -----------------------------------------------------------------------
@@ -41,15 +41,18 @@
yamllint-find += -print0
## -----------------------------------------------------------------------
+## Intent: Perform a lint check on yaml sources
## -----------------------------------------------------------------------
lint lint-yaml:
$(HIDE)$(env-clean) $(yamllint-find) \
- | xargs -0 --no-run-if-empty -t -n1 $(yamllint) $(yamllint-args)
+ | $(xargs-n1-clean) -t $(yamllint) $(yamllint-args)
+# | xargs -0 --no-run-if-empty -t -n1 $(yamllint) $(yamllint-args)
## -----------------------------------------------------------------------
+## Intent: Display command help
## -----------------------------------------------------------------------
help::
- @echo " lint-yaml Syntax check yaml sources"
+ @echo " lint-yaml Syntax check yaml sources (yamllint)"
$(if $(DEBUG),$(warning ENTER))