Skip to contentSkip to conversation

jemalloc: add a check for valgrind being configured as well

PATCH New Charles Hardin 2 participants 2 messages Sep 8, 2016 – Sep 11, 2016

lore
Share & tools
lore

Charles Hardin

author
#0 lore raw

PATCH 1/1 jemalloc: add a check for valgrind being configured as well

jemalloc can be compiled with valgrind support being enabled to produce proper output, so when the valgrind package has been configured configure jemalloc appropriately.

Signed-off-by Charles Hardin <[email protected]>
---
 package/jemalloc/jemalloc.mk | 9 +++++++++
 1 file changed, 9 insertions(+)
package/jemalloc/jemalloc.mk +9 -0
index 7086dfa..f1c9ffe 100644
@@ -11,5 +11,14 @@ JEMALLOC_LICENSE = BSD-2c
 JEMALLOC_LICENSE_FILES = COPYING
 JEMALLOC_INSTALL_STAGING = YES
 
+ifeq ($(BR2_PACKAGE_VALGRIND),y)
+JEMALLOC_DEPENDENCIES += valgrind
+JEMALLOC_CONF_OPTS += --enable-valgrind
+else
+JEMALLOC_CONF_OPTS += --disable-valgrind
+endif
+
+HOST_JEMALLOC_CONF_OPTS += --disable-valgrind
+
 $(eval $(autotools-package))
 $(eval $(host-autotools-package))
Signature · 1 line
2.7.4 (Apple Git-66)
Applied
#1 lore raw

PATCH 1/1 jemalloc: add a check for valgrind being configured as well

Hello,

Quoted 8 lines · Charles Hardin

jemalloc can be compiled with valgrind support being enabled to produce proper output, so when the valgrind package has been configured configure jemalloc appropriately.

Signed-off-by Charles Hardin <[email protected]>
---
 package/jemalloc/jemalloc.mk | 9 +++++++++
 1 file changed, 9 insertions(+)

Applied to master, thanks.

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com