Revision 08366664 Makefile.am
b/Makefile.am | ||
---|---|---|
9 | 9 |
abs_top_builddir = @abs_top_builddir@ |
10 | 10 |
abs_top_srcdir = @abs_top_srcdir@ |
11 | 11 |
|
12 |
# Helper values for calling builtin functions |
|
13 |
empty := |
|
14 |
space := $(empty) $(empty) |
|
15 |
comma := , |
|
16 |
|
|
12 | 17 |
# Use bash in order to be able to use pipefail |
13 | 18 |
SHELL=/bin/bash |
14 | 19 |
|
... | ... | |
150 | 155 |
BUILT_SOURCES = \ |
151 | 156 |
ganeti \ |
152 | 157 |
stamp-srclinks \ |
153 |
lib/_autoconf.py \ |
|
154 |
lib/_vcsversion.py \ |
|
155 | 158 |
$(all_dirfiles) \ |
156 |
$(PYTHON_BOOTSTRAP) |
|
159 |
$(PYTHON_BOOTSTRAP) \ |
|
160 |
$(BUILT_PYTHON_SOURCES) |
|
157 | 161 |
|
158 |
nodist_pkgpython_PYTHON = \
|
|
162 |
BUILT_PYTHON_SOURCES = \
|
|
159 | 163 |
lib/_autoconf.py \ |
160 | 164 |
lib/_vcsversion.py |
161 | 165 |
|
166 |
nodist_pkgpython_PYTHON = \ |
|
167 |
$(BUILT_PYTHON_SOURCES) |
|
168 |
|
|
162 | 169 |
noinst_PYTHON = \ |
163 | 170 |
lib/build/__init__.py \ |
164 | 171 |
lib/build/sphinx_ext.py |
... | ... | |
365 | 372 |
# it changes |
366 | 373 |
doc/html/index.html: $(docrst) $(docpng) doc/conf.py configure.ac \ |
367 | 374 |
$(RUN_IN_TEMPDIR) lib/build/sphinx_ext.py lib/opcodes.py lib/ht.py \ |
368 |
| lib/_autoconf.py lib/_vcsversion.py
|
|
375 |
| $(BUILT_PYTHON_SOURCES)
|
|
369 | 376 |
@test -n "$(SPHINX)" || \ |
370 | 377 |
{ echo 'sphinx-build' not found during configure; exit 1; } |
371 | 378 |
@mkdir_p@ $(dir $@) |
... | ... | |
1063 | 1070 |
@rm -f test.tix |
1064 | 1071 |
./htools/test |
1065 | 1072 |
|
1073 |
# E111: indentation is not a multiple of four |
|
1074 |
# E261: at least two spaces before inline comment |
|
1075 |
# E501: line too long (80 characters) |
|
1076 |
PEP8_IGNORE = E111,E261,E501 |
|
1077 |
|
|
1078 |
# For excluding pep8 expects filenames only, not whole paths |
|
1079 |
PEP8_EXCLUDE = $(subst $(space),$(comma),$(strip $(notdir $(BUILT_PYTHON_SOURCES)))) |
|
1080 |
|
|
1066 | 1081 |
.PHONY: lint |
1067 | 1082 |
lint: $(BUILT_SOURCES) |
1068 | 1083 |
@test -n "$(PYLINT)" || { echo 'pylint' not found during configure; exit 1; } |
1084 |
if test -z "$(PEP8)"; then \ |
|
1085 |
echo '"pep8" not found during configure' >&2; \ |
|
1086 |
else \ |
|
1087 |
$(PEP8) --repeat --ignore='$(PEP8_IGNORE)' --exclude='$(PEP8_EXCLUDE)' \ |
|
1088 |
$(lint_python_code); \ |
|
1089 |
fi |
|
1069 | 1090 |
$(PYLINT) $(LINT_OPTS) $(lint_python_code) |
1070 | 1091 |
cd $(top_srcdir)/qa && \ |
1071 | 1092 |
PYTHONPATH=$(abs_top_srcdir) $(PYLINT) $(LINT_OPTS) \ |
Also available in: Unified diff