losif,

Thanks for discovering and debugging this issue.  Yes, it's a long-time
latent bug because the 2nd argument is missing a qw() wrapper.  Your
solution is the right one.

I created an issue <https://github.com/backuppc/backuppc/issues/466> and
have pushed a fix
<https://github.com/backuppc/backuppc/commit/2c9270b9b849b2c86ae6301dd722c97757bc9256>
.

Craig

On Sat, Apr 9, 2022 at 9:35 AM Iosif Fettich <ifett...@netsoft.ro> wrote:

> Hi Ged,
>
> > $ grep -n 'my $d = Data::Dumper->new'
> > /usr/local/BackupPC/lib/BackupPC/Storage/Text.pm
> > 379:                    my $d = Data::Dumper->new([$newConf->{$var}],
> [*value]);
> > 413:    my $d = Data::Dumper->new([$newConf->{$var}], [*value]);
>
> Ouch. I haven't even noticed that there are two occurencies, the one I
> patched was enough for what I was doing.
>
> Both should be corrected in the same way, i.e. no second call argument.
>
> Thanks,
>
> Iosif
>
>
> _______________________________________________
> BackupPC-users mailing list
> BackupPC-users@lists.sourceforge.net
> List:    https://lists.sourceforge.net/lists/listinfo/backuppc-users
> Wiki:    https://github.com/backuppc/backuppc/wiki
> Project: https://backuppc.github.io/backuppc/
>
_______________________________________________
BackupPC-users mailing list
BackupPC-users@lists.sourceforge.net
List:    https://lists.sourceforge.net/lists/listinfo/backuppc-users
Wiki:    https://github.com/backuppc/backuppc/wiki
Project: https://backuppc.github.io/backuppc/

Reply via email to