Skip Menu |
 

This queue is for tickets about the Dackup CPAN distribution.

Report information
The Basics
Id: 44405
Status: new
Priority: 0/
Queue: Dackup

People
Owner: Nobody in particular
Requestors: adamk [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Unimportant
Broken in: 0.41
Fixed in: (no value)



Subject: Allow optional ssh string param for the SSH backend.
Download (untitled) / with headers
text/plain 961b
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.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.