aboutsummaryrefslogtreecommitdiffstats
path: root/main/abuild/0001-abuild-add-env-option-to-require-tests.patch
diff options
context:
space:
mode:
Diffstat (limited to 'main/abuild/0001-abuild-add-env-option-to-require-tests.patch')
-rw-r--r--main/abuild/0001-abuild-add-env-option-to-require-tests.patch33
1 files changed, 33 insertions, 0 deletions
diff --git a/main/abuild/0001-abuild-add-env-option-to-require-tests.patch b/main/abuild/0001-abuild-add-env-option-to-require-tests.patch
new file mode 100644
index 0000000000..49497421b8
--- /dev/null
+++ b/main/abuild/0001-abuild-add-env-option-to-require-tests.patch
@@ -0,0 +1,33 @@
+From d85a92478fc8a95bdcb4bf84c30c20ca935abc08 Mon Sep 17 00:00:00 2001
+From: Carlo Landmeter <clandmeter@gmail.com>
+Date: Tue, 24 Oct 2017 14:48:52 +0200
+Subject: [PATCH] abuild: add env option to require tests
+
+This adds an env option REQUIRE_CHECK to require testsuites to
+be run. This does not clutter getopts so it can be safely removed
+afterwards when we enforce tests globally. This will allow our CI
+infrastructure to enforce testsuites where possible.
+---
+ abuild.in | 6 ++++++
+ 1 file changed, 6 insertions(+)
+
+diff --git a/abuild.in b/abuild.in
+index 41b465d..7e3c818 100644
+--- a/abuild.in
++++ b/abuild.in
+@@ -226,6 +226,12 @@ default_sanitycheck() {
+ check_secfixes_comment || return 1
+
+ makedepends_has 'g++' && ! options_has toolchain && warning "g++ should not be in makedepends"
++
++ if ! options_has "!check" && [ -n "$REQUIRE_CHECK" ]; then
++ (unset check; . "$APKBUILD"; type check >/dev/null 2>&1) || \
++ die "Testsuites (abuild check) are required or needs to be explicitly disabled!"
++ fi
++
+ return 0
+ }
+
+--
+2.14.2
+