blob: 9d0088f804741e1e4b7cb409add8c829952d55a5 [file] [log] [blame]
Joey Armstrong28eddda2023-01-10 03:09:34 -05001# -*- makefile -*-
2# -----------------------------------------------------------------------
Joey Armstrong518f3572024-02-11 07:56:25 -05003# Copyright 2017-2024 Open Networking Foundation (ONF) and the ONF Contributors
Joey Armstrong28eddda2023-01-10 03:09:34 -05004#
Joey Armstrong980e37f2023-02-28 18:57:41 -05005# Licensed under the Apache License, Version 2.0 (the "License")
Joey Armstrong28eddda2023-01-10 03:09:34 -05006# you may not use this file except in compliance with the License.
7# You may obtain a copy of the License at
8#
9# http://www.apache.org/licenses/LICENSE-2.0
10#
11# Unless required by applicable law or agreed to in writing, software
12# distributed under the License is distributed on an "AS IS" BASIS,
13# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
14# See the License for the specific language governing permissions and
15# limitations under the License.
16# -----------------------------------------------------------------------
17
Joey Armstrong980e37f2023-02-28 18:57:41 -050018$(if $(DEBUG),$(warning ENTER))
Joey Armstrong28eddda2023-01-10 03:09:34 -050019
Joey Armstrong980e37f2023-02-28 18:57:41 -050020##-------------------##
21##---] GLOBALS [---##
22##-------------------##
23.PHONY: lint-json lint-json-all lint-json-modified
Joey Armstrong28eddda2023-01-10 03:09:34 -050024
Joey Armstrong980e37f2023-02-28 18:57:41 -050025have-json-files := $(if $(strip $(JSON_FILES)),true)
26JSON_FILES ?= $(error JSON_FILES= required)
Joey Armstrong28eddda2023-01-10 03:09:34 -050027
28## -----------------------------------------------------------------------
Joey Armstrong980e37f2023-02-28 18:57:41 -050029## Intent: Use the json command to perform syntax checking.
30## o If UNSTABLE=1 syntax check all sources
31## o else only check sources modified by the developer.
32## Usage:
33## % make lint UNSTABLE=1
34## % make lint-json-all
Joey Armstrong28eddda2023-01-10 03:09:34 -050035## -----------------------------------------------------------------------
Joey Armstrong980e37f2023-02-28 18:57:41 -050036ifndef NO-LINT-JSON
37 lint-json-mode := $(if $(have-json-files),modified,all)
38 lint : lint-json-$(lint-json-mode)
39endif# NO-LINT-JSON
Joey Armstrong28eddda2023-01-10 03:09:34 -050040
Joey Armstrong980e37f2023-02-28 18:57:41 -050041## -----------------------------------------------------------------------
42## Intent: exhaustive json syntax checking
43## -----------------------------------------------------------------------
44json-find-args := $(null)
45json-find-args += -name '$(venv-name)'
Joey Armstrongdaa1f0a2024-04-03 18:07:59 -040046lint-json-all:
Joey Armstrong980e37f2023-02-28 18:57:41 -050047 $(HIDE)$(MAKE) --no-print-directory lint-json-install
48
49 $(activate)\
50 && find . \( $(json-find-args) \) -prune -o -name '*.json' -print0 \
51 | $(xargs-n1) python -m json.tool > /dev/null ;\
52
53## -----------------------------------------------------------------------
54## Intent: check deps for format and python3 cleanliness
55## Note:
56## json --py3k option no longer supported
57## -----------------------------------------------------------------------
58lint-json-modified: $(venv-activate-script)
59 $(HIDE)$(MAKE) --no-print-directory lint-json-install
60
61 $(activate)\
62 && for jsonfile in $(JSON_FILES); do \
63 echo "Validating json file: $$jsonfile" ;\
64 python -m json.tool $$jsonfile > /dev/null ;\
65 done
66
67## -----------------------------------------------------------------------
68## Intent:
69## -----------------------------------------------------------------------
70.PHONY: lint-json-install
71lint-json-install: $(venv-activate-script)
72 @echo
73 @echo "** -----------------------------------------------------------------------"
74 @echo "** json syntax checking"
75 @echo "** -----------------------------------------------------------------------"
76# $(activate) && pip install --upgrade json.tool
77# $(activate) && python -m json.tool --version (?-howto-?)
78 @echo
79
80## -----------------------------------------------------------------------
81## Intent: Display command usage
82## -----------------------------------------------------------------------
Joey Armstrong28eddda2023-01-10 03:09:34 -050083help::
Joey Armstrong980e37f2023-02-28 18:57:41 -050084 @echo ' lint-json Syntax check python using the json command'
85 ifdef VERBOSE
86 @echo ' lint-json-all json checking: exhaustive'
87 @echo ' lint-json-modified json checking: only modified'
88 endif
89
90$(if $(DEBUG),$(warning LEAVE))
Joey Armstrong28eddda2023-01-10 03:09:34 -050091
92# [EOF]