Re: [PATCH] Adding support for file copy to marvin's remoteSSHClient

2012-06-10 Thread David Nalley
Prasanna, Sorry for taking so long to circle back to this - I'll commit this shortly. --David

[PATCH] Adding support for file copy to marvin's remoteSSHClient

2012-06-05 Thread Prasanna Santhanam
Signed-off-by: Prasanna Santhanam prasanna.santha...@citrix.com --- tools/marvin/marvin/remoteSSHClient.py | 11 ++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/tools/marvin/marvin/remoteSSHClient.py b/tools/marvin/marvin/remoteSSHClient.py index 8836844..4063b33

Re: [PATCH] Adding support for file copy to marvin's remoteSSHClient

2012-06-05 Thread David Nalley
On Tue, Jun 5, 2012 at 6:38 AM, Prasanna Santhanam prasanna.santha...@citrix.com wrote: Signed-off-by: Prasanna Santhanam prasanna.santha...@citrix.com ---  tools/marvin/marvin/remoteSSHClient.py |   11 ++-  1 file changed, 10 insertions(+), 1 deletion(-) diff --git

RE: [PATCH] Adding support for file copy to marvin's remoteSSHClient

2012-06-05 Thread Edison Su
-Original Message- From: David Nalley [mailto:da...@gnsa.us] Sent: Tuesday, June 05, 2012 8:27 AM To: cloudstack-dev@incubator.apache.org Subject: Re: [PATCH] Adding support for file copy to marvin's remoteSSHClient On Tue, Jun 5, 2012 at 6:38 AM, Prasanna Santhanam

Re: [PATCH] Adding support for file copy to marvin's remoteSSHClient

2012-06-05 Thread prasanna
This inherently looks ok, but Paramiko that you use in the above patch is licensed LGPLv2 - and needs to be removed[1]. Before we perpetuate its use, would you take a look at: http://bugs.cloudstack.org/browse/CS-14998 http://bugs.cloudstack.org/browse/CS-14999