Re: [BackupPC-users] Needed patch for BackupPC::Storage::Text.pm

2022-04-15 Thread Craig Barratt via BackupPC-users
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 and have pushed a fix

Re: [BackupPC-users] Needed patch for BackupPC::Storage::Text.pm

2022-04-09 Thread Iosif Fettich
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

Re: [BackupPC-users] Needed patch for BackupPC::Storage::Text.pm

2022-04-09 Thread G.W. Haywood via BackupPC-users
Hi Iosif, On Sat, 9 Apr 2022, Iosif Fettich wrote: just to let you know: BackupPC::Storage::Text.pm needs a patch. in sub ConfigFileMerge, the line reading my $d = Data::Dumper->new([$newConf->{$var}], [*value]); should ... Which one? I'm not sure what you're asking... $

Re: [BackupPC-users] Needed patch for BackupPC::Storage::Text.pm

2022-04-09 Thread Iosif Fettich
Hi Ged, just to let you know: BackupPC::Storage::Text.pm needs a patch. in sub ConfigFileMerge, the line reading my $d = Data::Dumper->new([$newConf->{$var}], [*value]); should ... Which one? I'm not sure what you're asking... I thought giving the full line would be better

Re: [BackupPC-users] Needed patch for BackupPC::Storage::Text.pm

2022-04-09 Thread Norman J. Goldstein
Thank you for the heads-up.  This may have been what happened to me a few weeks ago (on Fedora 35), when I was modifying the ping parameters to ensure ping used IPv4. However, I would like to hear back from others, too, before making the code change. Norm On 2022-04-09 03:00, Iosif Fettich

Re: [BackupPC-users] Needed patch for BackupPC::Storage::Text.pm

2022-04-09 Thread G.W. Haywood via BackupPC-users
Hi there, On Sat, 9 Apr 2022, Iosif Fettich wrote: just to let you know: BackupPC::Storage::Text.pm needs a patch. in sub ConfigFileMerge, the line reading my $d = Data::Dumper->new([$newConf->{$var}], [*value]); should ... Which one? I've tried to get in touch with Craig Barrat

[BackupPC-users] Needed patch for BackupPC::Storage::Text.pm

2022-04-09 Thread Iosif Fettich
Hi BackupPC list members, just to let you know: BackupPC::Storage::Text.pm needs a patch. To be more exact, within sub ConfigFileMerge, the line reading my $d = Data::Dumper->new([$newConf->{$var}], [*value]); should become my $d = Data::Dumper->new([$newConf->{$var}]); (just omit