[FFmpeg-devel,1/2] configure: properly honor --valgrind option

Submitted by Clément Bœsch on April 9, 2017, 4:46 p.m.

Details

Message ID 20170409164651.31151-1-u@pkh.me
State New
Headers show

Commit Message

Clément Bœsch April 9, 2017, 4:46 p.m.
Setting --valgrind implies the valgrind-memcheck toolchain. Before this
commit, the toolchain handling was overriding the target exec with an
arbitrary "valgrind" setting not taking into account the user setting.
---
 configure | 1 +
 1 file changed, 1 insertion(+)

Patch hide | download patch | download mbox

diff --git a/configure b/configure
index 8ad0bda76c..539a126656 100755
--- a/configure
+++ b/configure
@@ -3615,6 +3615,7 @@  case "$toolchain" in
     ;;
     valgrind-*)
         target_exec_default="valgrind"
+        test -n "$valgrind" && target_exec_default="$valgrind"
         case "$toolchain" in
             valgrind-massif)
                 target_exec_args="--tool=massif --alloc-fn=av_malloc --alloc-fn=av_mallocz --alloc-fn=av_calloc --alloc-fn=av_fast_padded_malloc --alloc-fn=av_fast_malloc --alloc-fn=av_realloc_f --alloc-fn=av_fast_realloc --alloc-fn=av_realloc"