On Tue, 2010-12-21 at 12:46 +0800, Tay Ray Chuan wrote: > Allow the vendor to be specified on the client class. That way, when > users wish to use a separate vendor, they can just subclass > SSHGitClient, instead of replacing (globally) client.get_ssh_vendor. > --- > dulwich/client.py | 7 +++---- > dulwich/tests/compat/test_client.py | 25 +++++++++++++------------ > 2 files changed, 16 insertions(+), 16 deletions(-) > > diff --git a/dulwich/client.py b/dulwich/client.py > index 2b69852..e89879f 100644 > --- a/dulwich/client.py > +++ b/dulwich/client.py > @@ -338,12 +338,11 @@ class SSHVendor(object): > stdout=subprocess.PIPE) > return SubprocessWrapper(proc) > > -# Can be overridden by users > -get_ssh_vendor = SSHVendor > - > > class SSHGitClient(GitClient): > > + ssh_vendor_class = SSHVendor > + My main concern is about keeping compatibility with the Bazaar SSHVendor interface - both so bzr-git can easily hook it in, but also to keep the option open of stealing that code easily in the future. This patch doesn't change the SSHVendor class, so that's not really an issue here.
I'm happy to merge this but would prefer to keep calling this variable get_vendor as ideally it selects a vendor (openssh, sshcorp, plink, lsh, paramiko, etc) and returns an instance of that rather than being hardcoded to a single class. We should also document in NEWS that get_ssh_vendor has moved. Cheers, Jelmer
signature.asc
Description: This is a digitally signed message part
_______________________________________________ Mailing list: https://launchpad.net/~dulwich-users Post to : [email protected] Unsubscribe : https://launchpad.net/~dulwich-users More help : https://help.launchpad.net/ListHelp
