Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] jemalloc: add a check for valgrind being configured as well
@ 2016-09-08 23:42 Charles Hardin
  2016-09-11 20:27 ` Thomas Petazzoni
  0 siblings, 1 reply; 2+ messages in thread
From: Charles Hardin @ 2016-09-08 23:42 UTC (permalink / raw)
  To: buildroot

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 <ckhardin@exablox.com>
---
 package/jemalloc/jemalloc.mk | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/package/jemalloc/jemalloc.mk b/package/jemalloc/jemalloc.mk
index 7086dfa..f1c9ffe 100644
--- a/package/jemalloc/jemalloc.mk
+++ b/package/jemalloc/jemalloc.mk
@@ -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))
-- 
2.7.4 (Apple Git-66)

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [Buildroot] [PATCH 1/1] jemalloc: add a check for valgrind being configured as well
  2016-09-08 23:42 [Buildroot] [PATCH 1/1] jemalloc: add a check for valgrind being configured as well Charles Hardin
@ 2016-09-11 20:27 ` Thomas Petazzoni
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Petazzoni @ 2016-09-11 20:27 UTC (permalink / raw)
  To: buildroot

Hello,

On Thu,  8 Sep 2016 16:42:00 -0700, Charles Hardin wrote:
> 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 <ckhardin@exablox.com>
> ---
>  package/jemalloc/jemalloc.mk | 9 +++++++++
>  1 file changed, 9 insertions(+)

Applied to master, thanks.

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

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-09-11 20:27 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-08 23:42 [Buildroot] [PATCH 1/1] jemalloc: add a check for valgrind being configured as well Charles Hardin
2016-09-11 20:27 ` Thomas Petazzoni

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox