valgrind support in native

Development archive
1

Hi,

I just opened a pull request for valgrind support in native: https://github.com/RIOT-OS/RIOT/pull/230

This PR makes native depend on valgrind which I guess is reasonable as valgrind should be used anyways. If you think it should rather be opt-out or even opt-in let me know and I'll ifdef around it.

Cheers, Ludwig

2

Hi,

I just opened a pull request for valgrind support in native: valgrind support for native by LudwigKnuepfer · Pull Request #230 · RIOT-OS/RIOT · GitHub

Nice!

This PR makes native depend on valgrind which I guess is reasonable as valgrind should be used anyways. If you think it should rather be opt-out or even opt-in let me know and I'll ifdef around it.

I'd definitely be for opt-in. We don't have any autoconf yet, so users will definately stumble over "cannot find valgrind.h"....

Keep dependencies to a minimum.

Why do you think "valgrind should be used anyways"?

Cheers, Kaspar

3

This PR makes native depend on valgrind which I guess is reasonable as valgrind should be used anyways. If you think it should rather be opt-out or even opt-in let me know and I'll ifdef around it.

I'd definitely be for opt-in. We don't have any autoconf yet, so users will definately stumble over "cannot find valgrind.h"....

OK, I'll update the PR accordingly.

Keep dependencies to a minimum. Why do you think "valgrind should be used anyways"?

Not using valgrind is like disabling compiler warnings, only worse :wink: For me valgrind is about as much a dependency as a compiler is...

Cheers, Ludwig

4

I just did.

Cheers, Ludwig