On 7 June 2013 20:29, Christian Grobmeier <[email protected]> wrote:

> Hi,
>
> these docs:
> http://logging.apache.org/log4php/docs/appenders/rolling-file.html
>
> say, one must include a %s:
> "Should contain a %s which gets substituted by the date."
>
> I have tried that, but it seems I don't need to do that. From the code
> I didn't see anything which would point into that direction. Anybody
> knows anything about it? Otherwise i remove this refence.
>

You're right. It's a copy/paste leftover from the LoggerAppenderDailyFile.
Please delete it.



> Also I found out one needs to give a full path to the rolling file.
> Like /var/www/xyz.log.
> Otherwise filesize() would return with a warning. We could disable
> that warning with @filesize, but i am not sure if that is a good idea.
>
> Here is the warn:
>
>  Warning: filesize(): stat failed for log/cicada.log in
>
> /var/www/cicada/vendor/apache/log4php/src/main/php/appenders/LoggerAppenderRollingFile.php
> on line 223
>
> The other option is to document a full path is needed.
>

I'll try it out and see what's up with that.

Regards,
Ivan

Reply via email to