U++ forum: Welcome to the forum

Re: Patches - Fixes - Bazaar addition [message #15889 is a reply to message #15881]

Wed, 14 May 2008 10:36

mdelfede
Messages: 1307
Registered: September 2007

Ultimate Contributor

luzr wrote on Wed, 14 May 2008 08:14

mdelfede wrote on Mon, 12 May 2008 17:41

File 'Valgrind.cpp' uses a wrong (or not compatible with default ubuntu Valgrind version) flag calling valgrind executable.
The line 20, which is :

	cmdline << "valgrind --xml=yes --num-callers=40 --log-file-exactly=" << fn << ' ';

Should be :

	cmdline << "valgrind --xml=yes --num-callers=40 --log-file=" << fn << ' ';

Ops. This will not work with my copy of valgrind. "--log-file" produces non-fixed filenames there... (valgrind appends process number to extension - that is why it has -exactly option....)

Mirek

Uhmmm... that's weird. From where did you download valgrind ? I've got the latest ubuntu package and it doesn't have "exactly" stuff.
BTW, I think I can patch valgrind.cpp to check wether the option is available or not.... should I ?

Max

Report message to a moderator