Tod Detre <tod.de...@maine.edu> wrote on 03/16/2011 04:41:05 PM:

> > I'd have to look at the rest of the code, but if gethostbyname 
> fails, and if you remove the nmb stuff, you're left with a hostname.
>  The question is, does the rest of the code work with a host name 
> string rather than an IP address?
> 
> It did when I tried it, but it was on my home network with 2-3
> machines and I did not do a lot of testing.

Interesting.  If that's the case, it would seem that rather than exit when 
the NMB lookup fails, it should simply log the issue (possibly strongly) 
and continue.

FWICT, Perl does not give you a way to use getipaddrbyname unless you are 
using IPv6-specific modules.  (The same is true for every *other* aspect 
of IPv6 with Perl, including sockets.)  However, if passing hostnames 
through to the proper daemons does work, why not take advantage of it?

Craig, do you know of any issues that might come from simply removing the 
exit() on line 506 (and maybe adding some logging between 507-508)?  If it 
blows up later, it's at least potentially farther than it was if we simply 
exit.

The only downside to this is if the server is both IPv4 and IPv6 enabled, 
the system will default to IPv4 first:  the code will do an IPv4-only 
lookup first, and if it works, it'll use the IPv4 address it gets.  I 
don't see that as a major issue.

Also, I hope that native IPv6 support is a design requirement for BackupPC 
4.0?  :)

Timothy J. Massey

 
Out of the Box Solutions, Inc. 
Creative IT Solutions Made Simple!
http://www.OutOfTheBoxSolutions.com
tmas...@obscorp.com 
 
22108 Harper Ave.
St. Clair Shores, MI 48080
Office: (800)750-4OBS (4627)
Cell: (586)945-8796 

------------------------------------------------------------------------------
Colocation vs. Managed Hosting
A question and answer guide to determining the best fit
for your organization - today and in the future.
http://p.sf.net/sfu/internap-sfd2d
_______________________________________________
BackupPC-users mailing list
BackupPC-users@lists.sourceforge.net
List:    https://lists.sourceforge.net/lists/listinfo/backuppc-users
Wiki:    http://backuppc.wiki.sourceforge.net
Project: http://backuppc.sourceforge.net/

Reply via email to