]> git.ipfire.org Git - thirdparty/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)
committerMartin Kletzander <mkletzan@redhat.com>
Tue, 4 Oct 2016 05:45:47 +0000 (07:45 +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>
(cherry picked from commit a94efa50e253cb59a2925dca2705bce801f46fb1)

cfg.mk

diff --git a/cfg.mk b/cfg.mk
index b277db184a63cfb633f82a1147b177d39399c040..f5dac32f2d61dc620a7b06f509900da05198ae9f 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]$$