dev-libs/librelp: Don't enable valgrind in tests
authorThomas Deutschmann <whissi@gentoo.org>
Thu, 17 May 2018 14:54:28 +0000 (16:54 +0200)
committerThomas Deutschmann <whissi@gentoo.org>
Thu, 17 May 2018 14:54:43 +0000 (16:54 +0200)
Closes: https://bugs.gentoo.org/655952
Package-Manager: Portage-2.3.36, Repoman-2.3.9

dev-libs/librelp/files/librelp-1.2.16-fix-valgrind-usage.patch [new file with mode: 0644]
dev-libs/librelp/librelp-1.2.16.ebuild

diff --git a/dev-libs/librelp/files/librelp-1.2.16-fix-valgrind-usage.patch b/dev-libs/librelp/files/librelp-1.2.16-fix-valgrind-usage.patch
new file mode 100644 (file)
index 0000000..15f605d
--- /dev/null
@@ -0,0 +1,27 @@
+Upstream bug: https://github.com/rsyslog/librelp/issues/100
+
+--- a/configure.ac
++++ b/configure.ac
+@@ -176,16 +176,20 @@ fi
+ # valgrind
+ AC_ARG_ENABLE(valgrind,
+-        [AS_HELP_STRING([--enable-valgrind],[Enable valgrind tests@<:@default=yes@:>@])],
++        [AS_HELP_STRING([--enable-valgrind],[Enable valgrind tests@<:@default=no@:>@])],
+         [case "${enableval}" in
+          yes) enable_valgrind="yes" ;;
+           no) enable_valgrind="no" ;;
+            *) AC_MSG_ERROR(bad value ${enableval} for --enable-valgrind) ;;
+          esac],
+-        [enable_valgrind="yes"]
++        [enable_valgrind="no"]
+ )
+ if test "$enable_valgrind" = "yes"; then
+         AC_CHECK_PROG(VALGRIND, [valgrind], [valgrind], [no])
++
++        if test "x$VALGRIND" = "xno"; then
++                AC_MSG_ERROR([valgrind is missing but forced with --enable-valgrind. Either install valgrind or remove the option!])
++        fi
+ fi
+ AM_CONDITIONAL([HAVE_VALGRIND], test "$enable_valgrind" == "yes")
index e82777d8801bfe53b7274a4e066db57827707f8e..62f7452fb6a6fe4537344ee7720ef3aa6b80e784 100644 (file)
@@ -26,6 +26,8 @@ DEPEND="
        virtual/pkgconfig
 "
 
+PATCHES=( "${FILESDIR}"/${P}-fix-valgrind-usage.patch )
+
 src_prepare() {
        sed -i \
                -e 's/ -g"/"/g' \