Skip Menu |
 

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Perl-Critic-Policy-logicLAB-ProhibitShellDispatch CPAN distribution.

Report information
The Basics
Id: 91542
Status: resolved
Priority: 0/
Queue: Perl-Critic-Policy-logicLAB-ProhibitShellDispatch

People
Owner: jonasbn [...] cpan.org
Requestors: johantheolive [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in:
  • 0.01
  • 0.02
Fixed in: 0.03



Subject: System function in module
Date: Fri, 20 Dec 2013 12:03:57 -0600
To: bug-Perl-Critic-logicLAB [...] rt.cpan.org
From: "johantheolive [...] gmail.com" <johantheolive [...] gmail.com>
Download (untitled) / with headers
text/plain 261b
I have the following code: $startSSH -> system( {async => 1} , "$setRemoteProfile $patCmd 1> /dev/null" ); where $startSSH is a Net::OpenSSH object. Perl::Critic::Policy::logicLAB::ProhibitShellDispatch erroneously identifies this as a system/exec statement.
Subject: Re: [rt.cpan.org #91542] System function in module
Date: Sat, 21 Dec 2013 16:29:36 +0100
To: bug-Perl-Critic-logicLAB [...] rt.cpan.org
From: "Jonas B. Nielsen" <jonasbn [...] cpan.org>
Download (untitled) / with headers
text/plain 852b
Thanks for the bug report, I will look into it. jonasbn On Fri, Dec 20, 2013 at 7:04 PM, johantheolive@gmail.com via RT < bug-Perl-Critic-logicLAB@rt.cpan.org> wrote: Show quoted text
> Fri Dec 20 13:04:06 2013: Request 91542 was acted upon. > Transaction: Ticket created by johantheolive@gmail.com > Queue: Perl-Critic-logicLAB > Subject: System function in module > Broken in: (no value) > Severity: (no value) > Owner: Nobody > Requestors: johantheolive@gmail.com > Status: new > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=91542 > > > > I have the following code: > > $startSSH -> system( {async => 1} , "$setRemoteProfile $patCmd 1> > /dev/null" ); > > where $startSSH is a Net::OpenSSH > object. Perl::Critic::Policy::logicLAB::ProhibitShellDispatch erroneously > identifies this as a system/exec statement. > >
Download (untitled) / with headers
text/plain 240b
Hello, I have just uploaded release 0.03 to CPAN, please provide feedback if this release still bugs you and you code. I implemented a test case based on the example you provided and fixed accordingly. Thanks for the bug report, jonasbn
Subject: Re: [rt.cpan.org #91542] System function in module
Date: Fri, 27 Dec 2013 08:31:14 -0600
To: bug-Perl-Critic-logicLAB [...] rt.cpan.org
From: "johantheolive [...] gmail.com" <johantheolive [...] gmail.com>
Download (untitled) / with headers
text/plain 464b
Works great. Thanks for the quick fix! On Fri, Dec 27, 2013 at 4:16 AM, Jonas B. Nielsen via RT < bug-Perl-Critic-logicLAB@rt.cpan.org> wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=91542 > > > Hello, > > I have just uploaded release 0.03 to CPAN, please provide feedback if this > release still bugs you and you code. I implemented a test case based on the > example you provided and fixed accordingly. > > Thanks for the bug report, > > jonasbn >


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.