David Relson writes:

> In /etc/BackupPC/config.pl is the following:
> 
> $Conf{BackupFilesOnly} = {
>   '/' => [],
>   'main' => [
>     '/boot',
>     '/',
>     '/home'
>   ]
> };
> 
> These setting show up under Include/Exclude.  As the first group is the
> result of experimenting, I now want to delete it.  Clicking the
> "Delete" button enables the "Save" button.  Clicking the "Save" button
> _does_ rewrite /etc/BackupPC/config.pl, but the new file _still_ has
> the unwanted setting.
> 
> This seems like a defect.  Is it?

Yes, it's a bug that I've fixed.  Should be checked into cvs in the
next few days.  Here's the patch if you want to make the fix.

Craig

--- lib/BackupPC/CGI/EditConfig.pm       2006-11-18 22:07:58.000000000 -0800
+++ lib/BackupPC/CGI/EditConfig.pm      2006-11-26 16:44:12.000000000 -0800
@@ -1100,7 +1100,7 @@
 
         if ( !$isError && !$type->{noKeyEdit}
                         && $In{deleteVar} !~ /^\Q${varName}_z_\E.*_z_/
-                        && $In{deleteVar} =~ /^\Q${varName}_z_\E(\w+)$/ ) {
+                        && $In{deleteVar} =~ /^\Q${varName}_z_\E(.*)$/ ) {
             #
             # User deleted entry in this hash
             #

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
BackupPC-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/backuppc-users
http://backuppc.sourceforge.net/

Reply via email to