[Koha-bugs] [Bug 26854] stage-marc-import.pl does not properly fork

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26854

--- Comment #15 from Jonathan Druart <jonathan.druart at bugs.koha-community.org> ---
(In reply to David Cook from comment #13)
> (In reply to Jonathan Druart from comment #12)
> > Shouldn't we use the log dir defined in the log4perl config file?
> 
> I am not sure that I understand your question.
> 
> logdir in etc/koha-conf.xml is set by __LOG_DIR__ and the log dir in
> etc/log4perl.conf is also set by __LOG_DIR__, so it's the same log directory.

At the instance creation, yes. But you can modify it.

> The log4perl config file is also not able to be parsed by anything except
> Log4Perl, and using Log4Perl isn't really an option here without rewriting
> tools/stage-marc-import.pl to use Log4Perl instead of using the warn
> built-in.

It seems that the 'logdir' from the Koha config file is only used by
installer/install.pl and misc/cronjobs/edi_cron.pl
And so I guess we should remove it in favour or the Log4Perl one. With that in
mind I don't think it's a good idea to add another occurrence.

I think we should push this without the log, it's what we are doing for all
other background jobs anyway.

-- 
You are receiving this mail because:
You are watching all bug changes.