]> xenbits.xensource.com Git - libvirt.git/commitdiff
maint: fix syntax-check sc_prohibit_int_ijk exclude rule
authorPavel Hrdina <phrdina@redhat.com>
Mon, 23 May 2016 23:00:07 +0000 (01:00 +0200)
committerPavel Hrdina <phrdina@redhat.com>
Tue, 24 May 2016 11:36:11 +0000 (13:36 +0200)
Fix the regex for excluding files for this syntax-rule.  The rule "include/"
will not work, because we are matching the whole line like this
"^(...|include/|...)$ so we need to use "include/libvirt/libvirt.+".  The second
issue is that we are using only one '$' but there should be two of those at the
end.  The last small adjustment is to escape dots '.' so it match only dot.

Signed-off-by: Pavel Hrdina <phrdina@redhat.com>
cfg.mk

diff --git a/cfg.mk b/cfg.mk
index c19f615f9a40ff9739db957bf75532c6577ef231..a7b726664d3c5f808522e413e2b8b486a6b43f7a 100644 (file)
--- a/cfg.mk
+++ b/cfg.mk
@@ -1259,7 +1259,7 @@ exclude_file_name_regexp--sc_prohibit_include_public_headers_brackets = \
   ^(tools/|examples/|include/libvirt/(virterror|libvirt(-(admin|qemu|lxc))?)\.h$$)
 
 exclude_file_name_regexp--sc_prohibit_int_ijk = \
-  ^(src/remote_protocol-structs|src/remote/remote_protocol.x|cfg.mk|include/|src/admin_protocol-structs|src/admin/admin_protocol.x)$
+  ^(src/remote_protocol-structs|src/remote/remote_protocol\.x|cfg\.mk|include/libvirt/libvirt.+|src/admin_protocol-structs|src/admin/admin_protocol\.x)$$
 
 exclude_file_name_regexp--sc_prohibit_getenv = \
   ^tests/.*\.[ch]$$