|Subject:||Allow optional ssh string param for the SSH backend.|
So far when using Dackup for backing up on my local machine, I've been able to nest the constructor quite happily. my $ssh = Net::OpenSSH->new('nbackup:email@example.com'); $ssh->error and die("Failed to create Net::OpenSSH object"); Dackup->new( directory => '/home/adam/dackup/', source => Dackup::Target::Filesystem->new( prefix => '/var/lib/svn/', ), destination => Dackup::Target::Filesystem->new(, prefix => '/var/lib/svn2/', ), delete => 0, dry_run => 0, verbose => 1, )->backup; This is clean, elegant and easier to read. I can't do the same for the SSH target, because I'm supposed to pass in a Net::OpenSSH object that I've error-checked myself. I'd like to see Dackup::Target::Filesystem accept a string for the 'ssh' parameter, which it would then connect to and error check for me. This would allow the same elegance for that backend as for the filesystem ones.