mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2025-01-03 11:35:19 +00:00
build: enforce preprocessor indentation
Since cppi is not part of Fedora Core 12, the check is conditional: without cppi, running 'make syntax-check' merely warns: $ make sc_preprocessor_indentation preprocessor_indentation maint.mk: skipping test sc_preprocessor_indentation: cppi not installed * cfg.mk (sc_preprocessor_indentation): New syntax-check rule. (preprocessor_exempt): New macro, with first exemption.
This commit is contained in:
parent
12023373cd
commit
89a08ba63a
13
cfg.mk
13
cfg.mk
@ -251,6 +251,19 @@ sc_prohibit_trailing_blank_lines:
|
|||||||
rm -f $@-t; \
|
rm -f $@-t; \
|
||||||
test $$found = 0
|
test $$found = 0
|
||||||
|
|
||||||
|
# Regex for grep -E that exempts generated files from style rules.
|
||||||
|
preprocessor_exempt = (remote_(driver|protocol)\.h)$$
|
||||||
|
# Enforce recommended preprocessor indentation style.
|
||||||
|
sc_preprocessor_indentation:
|
||||||
|
@if (cppi --version >/dev/null 2>&1); then \
|
||||||
|
$(VC_LIST_EXCEPT) | grep '\.[ch]$$' \
|
||||||
|
| grep -vE '$(preprocessor_exempt)' | xargs cppi -a -c \
|
||||||
|
|| { echo '$(ME): incorrect preprocessor indentation' 1>&2; \
|
||||||
|
exit 1; }; \
|
||||||
|
else \
|
||||||
|
echo '$(ME): skipping test $@: cppi not installed' 1>&2; \
|
||||||
|
fi
|
||||||
|
|
||||||
# We don't use this feature of maint.mk.
|
# We don't use this feature of maint.mk.
|
||||||
prev_version_file = /dev/null
|
prev_version_file = /dev/null
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user