Bug #33821 for Tk-GraphViz: avoid the shell in calling the layout command

Wed Mar 05 05:21:28 2008 reinpost [...] win.tue.nl - Ticket created

Subject:

avoid the shell in calling the layout command

The dot command is invoked as a string. It's safer (and in the long run more flexible) to build it as an array instead. A patch is attached.

Subject:

TkGraphViz.pm.diff-urd

Download TkGraphViz.pm.diff-urd
application/octet-stream 1.1k

Message body not shown because it is not plain text.

Sat Mar 29 23:20:32 2008 jeremy [...] jkslade.net - Correspondence added

Subject:	Re: [rt.cpan.org #33821] avoid the shell in calling the layout command
Date:	Sat, 29 Mar 2008 21:19:23 -0600
To:	bug-Tk-GraphViz [...] rt.cpan.org
From:	Jeremy Slade <jeremy [...] jkslade.net>

Thanks for the feedback. I included your patch in the latest release: http://search.cpan.org/~jgs/Tk-GraphViz-1.00/ Jeremy Reinier Post via RT wrote: Show quoted text

> Wed Mar 05 05:21:28 2008: Request 33821 was acted upon. > Transaction: Ticket created by rpost > Queue: Tk-GraphViz > Subject: avoid the shell in calling the layout command > Broken in: 0.90 > Severity: Important > Owner: Nobody > Requestors: reinpost@win.tue.nl > Status: new > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=33821 > > > > The dot command is invoked as a string. It's safer (and in the long run > more flexible) to build it as an array instead. A patch is attached.

Sat Mar 29 23:20:33 2008 The RT System itself - Status changed from 'new' to 'open'

Fri Nov 06 17:17:49 2020 ETJ [...] cpan.org - Status changed from 'open' to 'resolved'

Fri Nov 06 17:17:50 2020 ETJ [...] cpan.org - Fixed in 1.00 added