BackupPC-users

Re: [BackupPC-users] Error installing BackupPC: - Bareword "compareLOGName" not allowed

2008-11-01 20:17:42
Subject: Re: [BackupPC-users] Error installing BackupPC: - Bareword "compareLOGName" not allowed
From: "Jeffrey J. Kosowsky" <backuppc AT kosowsky DOT org>
To: backuppc-users AT lists.sourceforge DOT net
Date: Sat, 01 Nov 2008 20:15:43 -0400
Holger Parplies wrote at about 02:37:55 +0100 on Saturday, November 1, 2008:
 > Hi,
 > 
 > Jeffrey J. Kosowsky wrote on 2008-10-31 13:25:20 -0400 [[BackupPC-users] 
 > Error installing BackupPC: - Bareword "compareLOGName" not allowed]:
 > > 
 > > When I run config.pl, I get the following error message:
 > > 
 > >     Error loading BackupPC::Lib: Bareword "compareLOGName" not allowed
 > >     while "strict subs" in use at lib/BackupPC/Lib.pm line 1466.
 > > [...]
 > >     
 > > I was able to fix this by changing the line near the end from:
 > >    return sort(compareLOGName @files);
 > > To:
 > >    return sort(compareLOGName( @files));
 > 
 > err, NO!
 > 
 > That one is a little subtle. If your change were correct, the Perl compiler
 > would not have complained (sub compareLOGName was declared before this point,
 > so a function call "compareLOGName @files" should not have parsed as a 
 > bareword
 > but rather as a list operator).
 > 

OK. But my change (though wrong) did stop the warning messages and
compiled ...

 > Actually, the original statement is passing both the subname "compareLOGName"
 > and @files to the sort operator. I don't know why Perl 5.10.0 complains about
 > that. I would first try leaving out the ()
 > 
 >      return sort compareLOGName @files;
 > 
This seems to work (well at least it doesn't generate the error...

 > (actually, this gets rid of a warning (!) in 5.8.8, so I would suggest this 
 > to
 > be changed anyway) and, if that doesn't help, something like
 > 
 >      my $comparison = \&compareLOGName;
 >      return sort($comparison @files);
 > 
 > Regards,
 > Holger

Well the change seems to be definitely necessary for later versions so
might as well do the (first) one you propose...

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
BackupPC-users mailing list
BackupPC-users AT lists.sourceforge DOT net
List:    https://lists.sourceforge.net/lists/listinfo/backuppc-users
Wiki:    http://backuppc.wiki.sourceforge.net
Project: http://backuppc.sourceforge.net/

<Prev in Thread] Current Thread [Next in Thread>
  • Re: [BackupPC-users] Error installing BackupPC: - Bareword "compareLOGName" not allowed, Jeffrey J. Kosowsky <=