| 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:phlegm6^@krackel.dreamhost.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.