lapapaye / parallel-ssh

Automatically exported from code.google.com/p/parallel-ssh
Other
0 stars 0 forks source link

parallel-rsync fails under Ubuntu 11.10 (Oneiric) with python error #58

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. parallel-rsync -z -r -O 'IdentityFile=/home/me/control.pem' -l control -H 
host1.com /mydir1 /mydir2

What is the expected output? What do you see instead?

Traceback (most recent call last):
  File "/usr/bin/parallel-rsync", line 125, in <module>
    do_prsync(hosts, local, remote, opts)
  File "/usr/bin/parallel-rsync", line 78, in do_prsync
    cmd = ['rsync', '-e', ' '.join(ssh)]
TypeError: sequence item 2: expected string, list found

ssh has value: ['ssh', '-o', ['IdentityFile=/home/me/control.pem']]

What version of the product are you using? On what operating system?

Ubuntu 11.10 (GNU/Linux 3.0.0-12-generic-pae i686)
pssh (2.2.2-0ubuntu1) oneiric; urgency=low
python-defaults (2.7.2-7ubuntu2) oneiric; urgency=low

Please provide any additional information below.

Is it because string.join() has been deprecated?

Original issue reported on code.google.com by b...@beyond2000.eu on 21 Oct 2011 at 9:39

GoogleCodeExporter commented 8 years ago
Just to mention. 
Everything works just fine under Ubuntu 10.04 LTS.

Original comment by b...@beyond2000.eu on 21 Oct 2011 at 2:56

GoogleCodeExporter commented 8 years ago
Thank you very much for raising the issue.  I think I've found and fixed the 
problem.  I've committed a two-line fix, and I'm also attaching a patch in case 
this is easier for you than pulling from the git repo.

Would you please test the change and let me know whether it works for you or 
whether you continue to have problems?  Thanks!

Original comment by amcna...@gmail.com on 21 Oct 2011 at 4:45

Attachments:

GoogleCodeExporter commented 8 years ago
I applied the patch and it worked fine.
Thanks for the prompt fix.

Original comment by b...@beyond2000.eu on 24 Oct 2011 at 9:34

GoogleCodeExporter commented 8 years ago
Great.  Feel free to contact the Debian maintainers to see if they can update 
their package to include the patch.

Original comment by amcna...@gmail.com on 24 Oct 2011 at 3:47