Patchwork local.conf.sample: change valgrind support architecture comment

login
register
mail settings
Submitter ml@communistcode.co.uk
Date Sept. 13, 2012, 11:20 a.m.
Message ID <1347535204-25960-1-git-send-email-ml@communistcode.co.uk>
Download mbox | patch
Permalink /patch/36473/
State New
Headers show

Comments

ml@communistcode.co.uk - Sept. 13, 2012, 11:20 a.m.
From: Jack Mitchell <jack.mitchell@dbbroadcast.co.uk>

The comment for debug-tools states valgrind will only be installed for x86
targets. This is not true as valgrind now supports x86*, PPC* and ARMv7a;
alter the comment to reflect this.

Signed-off-by: Jack Mitchell <jack.mitchell@dbbroadcast.co.uk>
---
 meta/conf/local.conf.sample | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Paul Eggleton - Sept. 13, 2012, 12:21 p.m.
On Thursday 13 September 2012 12:20:04 Jack Mitchell wrote:
> From: Jack Mitchell <jack.mitchell@dbbroadcast.co.uk>
> 
> The comment for debug-tools states valgrind will only be installed for x86
> targets. This is not true as valgrind now supports x86*, PPC* and ARMv7a;
> alter the comment to reflect this.

I have to say I think we can just drop this part of the comment entirely given 
how wide the support is now.

Cheers,
Paul

Patch

diff --git a/meta/conf/local.conf.sample b/meta/conf/local.conf.sample
index 5679c03..cf35310 100644
--- a/meta/conf/local.conf.sample
+++ b/meta/conf/local.conf.sample
@@ -119,7 +119,7 @@  PACKAGE_CLASSES ?= "package_ipk"
 #                     (useful if you want to develop against libs in the image)
 #  "tools-sdk"      - add development tools (gcc, make, pkgconfig etc.)
 #  "tools-debug"    - add debugging tools (gdb, strace)
-#  "tools-profile"  - add profiling tools (oprofile, exmap, lttng valgrind (x86 only))
+#  "tools-profile"  - add profiling tools (oprofile, exmap, lttng, valgrind(x86*, armv7a, ppc*))
 #  "tools-testapps" - add useful testing tools (ts_print, aplay, arecord etc.)
 #  "debug-tweaks"   - make an image suitable for development
 #                     e.g. ssh root access has a blank password