On Fri, Feb 10, 2012 at 02:08:42PM +0100, Pierre Habouzit wrote:
> On Thu, Feb 09, 2012 at 09:53:31AM +0100, Martin Apel wrote:
> > Package: valgrind
> > Version: 1:3.7.0-1+b1
> > Severity: important
> > Tags: patch sid
> >
> > The script callgrind_control searches for the valgrind process, but
> > Debian puts a wrapper around these,
> > so it does not find it. This makes the callgrind functionality less usable.
> >
> > The following patch fixes the problem:
> >
> > --- /home/home_dev/apel/callgrind_control 2012-02-09
> > 09:49:10.000000000 +0100
> > +++ /usr/bin/callgrind_control 2012-01-14 23:00:47.000000000 +0100
> > @@ -29,7 +29,7 @@
> > @pids = ();
> > open LIST, "vgdb -l|";
> > while(<LIST>) {
> > - if (/^use --pid=(\d+) for \S*?valgrind.bin\s+(.*?)\s*$/) {
> > + if (/^use --pid=(\d+) for \S*?valgrind\s+(.*?)\s*$/) {
> > $pid = $1;
> > $cmd = $2;
> > if (!($cmd =~ /--tool=callgrind/)) { next; }
> >
>
> I'm not sure why this should be required since /usr/bin/valgrind ends with
> # Use 'exec' to avoid having another shell process hanging around.
> exec $0.bin "$@"
>
> IOW valgrind.bin should just be here unless I'm mistaken.
Okay I've understood, your patch is inverted ;)
--
·O· Pierre Habouzit
··O madco...@debian.org
OOO http://www.madism.org
--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
