Linaro Image Tools

Code review comment for lp:~salgado/linaro-image-tools/cmd-runner

  1. cmd-runner
  2. Merge into trunk
Revision history for this message
James Westby (james-w) wrote :

On Tue, 30 Nov 2010 20:27:28 -0000, James Westby <email address hidden> wrote:
> Do we need shell=True? I assume we do, but using it as little as possible
> would be good.

Sorry, seems I wasn't really paying attention to that part of the code.

Thanks,

James

« Back to merge proposal