Add local_site_name parameters to most sshutils functions.

Review Request #2327 — Created May 1, 2011 and submitted

Information

Review Board
master

Reviewers

Add local_site_name parameters to most sshutils functions.

sshutils now knows how to factor in LocalSites when figuring out which
ssh configuration and keys to use. A LocalSite's configuration lives in
~/.ssh/<local_site_name>/.

As part of this, I've better cleaned up the path determination for getting
the .ssh directory. In some cases, we assumed ".ssh", and in others we checked
for "ssh". We also were using Unix-style paths everywhere. Now we check
more carefully and use os.path.join.
Unit tests passed.

Tested this along with my upcoming pending changes, and they work.
david
  1. 
      
  2. reviewboard/scmtools/sshutils.py (Diff revision 1)
     
     
    Can you make this use super() instead?
    1. Can't yet. It won't work with Python 2.4.
    2. Can't yet. It won't work with Python 2.4. Exceptions are old-style objects.
    3. Good point. Fix the double-comment issue and then ship it :)
  3. 
      
Loading...