From: Filipe Brandenburger <filbranden@gm...> - 2008-12-08 14:56:04
On Mon, Dec 8, 2008 at 07:30, Holger Parplies <wbppc@...> wrote:
> alex bodnaru wrote on 2008-12-08 05:31:04 +0200 [Re: [BackupPC-devel] Changing TopDir on config file]:
> > after an inquiry (and perl is not my language) i did the fix: append the
> > following two lines to /usr/share/backuppc/lib/BackupPC/Lib.pm:
> Apparently meaning you duplicated the assignments. Otherwise, your fix
> seems to correspond with mine, though I didn't check exactly.
Holger's patch seems to be the right one for me.
While Alex's patch should work, it is doing something very specific
(setting CPool and Pool values) in a very general function (ReadConf,
that reads any configuration value).
It seems to me that the real problem is in new, setting CPool and Pool
before the value of TopDir has been overridden by the config file, so
I think the best solution is the one proposed by Holger.
With this thread, I would like to point out that the problem has been
reported (multiple times) for around 4 months now, and it hasn't been
officially acknowledged, and although there are patches, there does
not seem to be a consensus on whether to go ahead with the patch or
not. That's why I copied the developers specifically on the original
e-mail. I would really appreciate if they could have a look at the
issue and the patch and give their opinion on them.