[pacman-dev] [PATCH] Add valgrind usage for testing purpose into the HACKING guide
djszapi at archlinux.us
Sat Nov 14 00:54:40 EST 2009
On Sat, Nov 14, 2009 at 6:41 AM, Allan McRae <allan at archlinux.org> wrote:
> Laszlo Papp wrote:
>> Signed-off-by: Laszlo Papp <djszapi at archlinux.us>
>> HACKING | 10 ++++++++++
>> 1 files changed, 10 insertions(+), 0 deletions(-)
>> diff --git a/HACKING b/HACKING
>> index 09e782d..954d4b7 100644
>> --- a/HACKING
>> +++ b/HACKING
>> @@ -133,6 +133,16 @@ For pacman:
>> #include "anythingelse.h"
>> +How to test pacman more using valgrind?
>> +Try to run some commands using valgrind with a command like:
>> +valgrind --leak-check=full --trace-children=yes --
>> src/pacman/.libs/pacman -Syu
>> +Be careful, valgrind reports lots of errors, even some errors that may
>> not be related to
>> +pacman. You should have a little experience of valgrind to ignore the
>> vim: set ts=2 sw=2 syntax=asciidoc et:
> Why would we provide instructions on how to use valgrind? That seems to be
> a job for the valgrind authors...
Yeah, but git usage (e.g.) is described too on the wiki and in the
> Anyway, I believe the pactest suite is set up to use valgrind so you would
> be better using that to detect leaks. Also, we have a valgrind.supp file
> to suppress a lot of the false positives.
I didn't know pactest suite can check all the facilities that valgrind
can services, but maybe it's better for valgrind users to use it, as
they use it daily, than debug/test which pactest suite does this task.
Maybe it's quicker to use directly valgrind in a given situation
sometimes than doing a full pactest suite, or deciding which one is
good for the current requirement.
In summary it's an attention for the developers/users they can use
valgrind too, so it's not a change for pactest suites, but more
facility to debug/test the application. It's not a pactest suite vs.
valgrind question. The person who aren't interested in using/learning
pactest suite, but he knows valgrind, he can take valgrind-like
testings/debuggings too, which could be very useful too.
More information about the pacman-dev