shekkbuilder / parallel-ssh

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

Change hosts file format #13

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
As suggested in issue 7, I think it should be better to have the following
format in the hosts file :

[user@]host[:port]

instead of :

host[:port] [user]

I think it's more "natural", and would be more consistent with the new
output file names (see issue 7).

Thanks in advance,
Marc

Original issue reported on code.google.com by dooblem...@positon.org on 22 Feb 2010 at 10:35

GoogleCodeExporter commented 8 years ago
I agree that this would be a better format, but to start with, we'll need to 
support 
both syntaxes (before deprecating the old syntax).  This isn't a really high 
priority 
issue, but I agree that it would be an improvement.  I'll get to it eventually, 
but it 
will probably be in PSSH 2.2 rather than PSSH 2.1.

Original comment by amcna...@gmail.com on 22 Feb 2010 at 5:47

GoogleCodeExporter commented 8 years ago
I've implemented the [user@]host[:port] in a new commit (which is _not_ 
included in 
PSSH 2.1).  Anyway, feel free to give it a try and see what you think.

Eventually, we'll need to deprecate the old line format, too, but for now the 
new line 
format should work.  Let me know if you have any problems.

Original comment by amcna...@gmail.com on 25 Feb 2010 at 6:00

GoogleCodeExporter commented 8 years ago
Since I haven't heard back from you, I'm going to close this issue.  However, it
would still be great if you could test it and give feedback.  Thanks.

Original comment by amcna...@gmail.com on 5 Mar 2010 at 3:55

GoogleCodeExporter commented 8 years ago
Sorry.. seems I missed your comment 2 :)

I tested it : it works perfectly with no problem.

I tested both new and old hosts file format, and also check the output file 
names.

All is OK.. Thanks a lot.

Original comment by dooblem...@positon.org on 5 Mar 2010 at 9:33

GoogleCodeExporter commented 8 years ago
Great.  I'm glad it works.  Starting in PSSH 2.2, [user@]host[:port] will be 
the 
documented syntax for the file, and we'll probably wait a few versions to 
deprecate 
the old syntax.  Thanks for your help.

Original comment by amcna...@gmail.com on 5 Mar 2010 at 5:13

GoogleCodeExporter commented 8 years ago
Where is the commit mentioned for this? I seem to have missed it. 

Original comment by rouss...@gmail.com on 30 Nov 2010 at 3:56

GoogleCodeExporter commented 8 years ago
This is in commit bda32356.

Original comment by amcna...@gmail.com on 2 Dec 2010 at 6:42