Hello there,

You might find it more convenient to depend on 'valgrind-if-available'
rather than 'valgrind' itself, per the attached patch.  It also skips
tests that need valgrind, for problematic arches.  What do you think?

diff --git a/debian/control b/debian/control
index 44fca69..a130223 100644
--- a/debian/control
+++ b/debian/control
@@ -13,7 +13,7 @@ Build-Depends: autoconf-archive,
                libconfuse-dev,
                pkg-config,
                uthash-dev,
-               valgrind <!nocheck>
+               valgrind-if-available <!nocheck>
 Standards-Version: 4.6.1
 Rules-Requires-Root: no
 Homepage: https://github.com/SELinuxProject/selint
diff --git a/debian/rules b/debian/rules
index 3a18158..2409916 100755
--- a/debian/rules
+++ b/debian/rules
@@ -1,5 +1,7 @@
 #!/usr/bin/make -f
 
+include /usr/share/dpkg/architecture.mk
+
 export DEB_BUILD_MAINT_OPTIONS = hardening=+all optimize=+lto
 
 ifneq (,$(filter nocheck,$(DEB_BUILD_OPTIONS)))
@@ -14,6 +16,10 @@ override_dh_auto_configure:
 	dh_auto_configure -- --enable-werror $(CONFIGURE_ARGS)
 
 execute_after_dh_auto_test:
+ifneq (,$(filter $(DEB_HOST_ARCH), mips64el mipsel))
+# Skip the tests requiring valgrind on problematic architectures
+	sed -i '/^@test "\(valgrind\|Broken config\)" {/a \	skip' end-to-end.bats
+endif
 	cd tests/functional/ && bats end-to-end.bats
 
 ifneq (, $(filter cross, $(DEB_BUILD_PROFILES)))

Reply via email to