On 10/23/2014 06:58 AM, Daniel P. Berrange wrote:
The cfg.mk rule for checking preprocessor indentation was mistakenly missing the libvirt.h.in file due to bad file extension matching rule. Fix that and the resolve the incorrect indentation thats identified.
s/thats/that's/
--- cfg.mk | 2 +- include/libvirt/libvirt.h.in | 76 ++++++++++++++++++++++---------------------- 2 files changed, 39 insertions(+), 39 deletions(-)
diff --git a/cfg.mk b/cfg.mk index 501530f..682687a 100644 --- a/cfg.mk +++ b/cfg.mk @@ -680,7 +680,7 @@ sc_require_whitespace_in_translation: # Enforce recommended preprocessor indentation style. sc_preprocessor_indentation: @if cppi --version >/dev/null 2>&1; then \ - $(VC_LIST_EXCEPT) | grep '\.[ch]$$' | xargs cppi -a -c \ + $(VC_LIST_EXCEPT) | grep -E '\.[ch](\.in)?$$' | xargs cppi -a -c \
Nice catch. ACK. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org