On 12/03/2010 11:11 PM, Vincent Massol wrote:
Should this go in 2.7 or should I wait for 3.0?
IMO waiting for 3.0 (i.e. Monday for the release and the trunk branch) is best at this
stage since we should be from now on only fix bugs for the 2.x cycle.
Technically, it is a bug. Practically, it changes XWiki in an important
way. I'll leave the decision to you, but I'd be +0.1 in favor of 2.7.
Thanks
-Vincent
On Dec 3, 2010, at 10:32 AM, Vincent Massol wrote:
> Hi devs,
>
> I'd like to propose removing the log to xwiki.log by default in our
log4j.properties file.
>
> The rationale is:
> * This is causing problems with a lot of users since by default we log in the current
directory and there are lots of cases when users start xwiki from a directory where you
don't have correct permissions
> * We don't control where xwiki.log is created
>
> Proposal:
> * We only log to stdout:
> ** tomcat is already configured to capture stdout/stderr and to redirect them to
catalina.out
> ** I have a modification ready for Jetty to do the same, here it is in case you want
to see it:
>
> <!-- =========================================================== -->
> <!-- Redirect stdout& stderr to a log file -->
> <!-- =========================================================== -->
> <New id="ServerLog" class="java.io.PrintStream">
> <Arg>
> <New class="org.mortbay.util.RolloverFileOutputStream">
> <Arg><SystemProperty name="jetty.home"
default="."/>/logs/yyyy_mm_dd.stderrout.log</Arg>
> <Arg type="boolean">false</Arg>
> <Arg type="int">90</Arg>
> <Arg><Call class="java.util.TimeZone"
name="getTimeZone"><Arg>GMT</Arg></Call></Arg>
> <Get id="ServerLogName" name="datedFilename"/>
> </New>
> </Arg>
> </New>
> <Call class="org.mortbay.log.Log"
name="info"><Arg>Redirecting stderr/stdout to<Ref
id="ServerLogName"/></Arg></Call>
> <Call class="java.lang.System"
name="setErr"><Arg><Ref
id="ServerLog"/></Arg></Call>
> <Call class="java.lang.System"
name="setOut"><Arg><Ref
id="ServerLog"/></Arg></Call>
>
> WDYT?
>
> Thanks
> -Vincent
--
Sergiu Dumitriu
http://purl.org/net/sergiu/