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/ _______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
