From 9f54f4f6ceb4a6ddb8dde03544680264da570839 Mon Sep 17 00:00:00 2001
From: Fredrik Fornwall <fredrik@fornwall.net>
Date: Thu, 28 Jul 2016 17:50:42 -0400
Subject: [PATCH] Add notes for valgrind

---
 packages/valgrind/build.sh | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/packages/valgrind/build.sh b/packages/valgrind/build.sh
index 664941f26..ae87b9810 100644
--- a/packages/valgrind/build.sh
+++ b/packages/valgrind/build.sh
@@ -1,10 +1,13 @@
+# Note:
+# - Does not build on arm due to https://github.com/android-ndk/ndk/issues/132 (fixed in ndk r13)
+# - Does not build on x86_64 due to lacking upstream support of that arch on android.
 TERMUX_PKG_HOMEPAGE=http://valgrind.org/
 TERMUX_PKG_DESCRIPTION="Valgrind is an instrumentation framework for building dynamic analysis tools."
 TERMUX_PKG_VERSION=3.11.0
 TERMUX_PKG_SRCURL=http://valgrind.org/downloads/valgrind-${TERMUX_PKG_VERSION}.tar.bz2
-TERMUX_PKG_DEPENDS=""
+# TERMUX_PKG_SRCURL=https://fossies.org/linux/misc/valgrind-${TERMUX_PKG_VERSION}.tar.bz2
+TERMUX_PKG_EXTRA_CONFIGURE_ARGS="--with-tmpdir=$TERMUX_PREFIX/tmp"
 
-TERMUX_PKG_EXTRA_CONFIGURE_ARGS="--with-tmpdir=/data/data/com.termux/files/usr/tmp"
 if [ "$TERMUX_ARCH" == "arm" ]; then
   # valgrind doesn't like arm; armv7 works, though.
   TERMUX_PKG_EXTRA_CONFIGURE_ARGS+=" --host=armv7-linux-androideabi"