apertis-abi-compare: fix sporadic issue with std headers

Due to a bug in abi-compliance-checker, std headers are occasionally
used for analyzing leading to wrongly detect an ABI breakage.
We globally exclude them to avoid this noise until it is fixed in ACC.

See: https://github.com/lvc/abi-compliance-checker/issues/85



Signed-off-by: Dylan Aïssi <dylan.aissi@collabora.com>

Makefile

@@ -4,6 +4,7 @@ all clean install:
 	$(MAKE) -C tests $@
 	$(MAKE) -C tools $@
 	$(MAKE) -C cmake $@
+	$(MAKE) -C aac $@
 
 check: all
 	$(MAKE) -C tests check

aac/Makefile

+INSTALL ?= install
+PREFIX ?= /usr
+aacdir = $(PREFIX)/share/apertis-abi-compare/
+AAC_CONFDIR ?= $(DESTDIR)$(aacdir)
+
+AAC_CONF = \
+	aac-skip-headers.conf
+
+all:
+
+clean:
+
+install:
+	$(INSTALL) -d $(AAC_CONFDIR)
+	for t in $(AAC_CONF); \
+	do \
+		$(INSTALL) $$t $(AAC_CONFDIR); \
+	done

aac/aac-skip-headers.conf

+stdio.h
+stdlib.h
+string.h
+ctype.h

debian/apertis-dev-tools.install

@@ -2,3 +2,4 @@ usr/bin/
 usr/share/man
 etc/xdg/ade/sysroot.conf
 usr/lib/cmake/
+usr/share/apertis-abi-compare/

tools/apertis-abi-compare

@@ -166,6 +166,11 @@ def run_abi_checker(lib, xml_old, ver_old, xml_new, ver_new, arch):
     acc_cmd.extend(['--arch', arch, '-lib', lib,
                     '-old', xml_old, '-v1', ver_old,
                     '-new', xml_new, '-v2', ver_new])
+    # Due to a bug in abi-compliance-checker, std headers are sporadically
+    # used for analyzing leading to wrongly detect an ABI breakage.
+    # We globally exclude them to avoid this noise until it is fixed in ACC.
+    # https://github.com/lvc/abi-compliance-checker/issues/85
+    acc_cmd.extend(['--skip-headers=/usr/share/apertis-abi-compare/aac-skip-headers.conf'])
 
     acc_process = subprocess.run(acc_cmd)
     sys.stdout.flush()