You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cocoon.apache.org by Leszek Gawron <lg...@apache.org> on 2005/04/14 12:01:03 UTC

Re: svn commit: r159493 - cocoon/trunk/src/webapp/WEB-INF/logkit.xconf

vgritsenko@apache.org wrote:
>    <!--+
> -      | Factories are responsible to create the consumers of the log events, the targets.
> -      | Here we have configured just a few, the main cocoon target factory (that prints to a file)
> -      | and the servlet target factory (that prints back to the servlet container log stream) but
> -      | for more info on the available logkit factories, please consult
> -      |  http://excalibur.apache.org/apidocs/org/apache/avalon/excalibur/logger/factory/package-summary.html
> +      | Factories are responsible to create the consumers of the log events,
> +      | the targets. Here we have configured just a few, the main cocoon
> +      | target factory (that prints to a file) and the servlet target factory
> +      | (that prints back to the servlet container log stream) but  for more
> +      | info on the available logkit factories, please consult
> +      | http://excalibur.apache.org/apidocs/org/apache/avalon/excalibur/logger/factory/package-summary.html
>        +-->
>    <factories>
>      <factory type="cocoon" class="org.apache.cocoon.util.log.CocoonTargetFactory"/>
>      <factory type="servlet" class="org.apache.avalon.excalibur.logger.factory.ServletTargetFactory"/>
> +    <factory type="stream" class="org.apache.avalon.excalibur.logger.factory.StreamTargetFactory"/>
>    </factories>
>  
>    <!--+
> @@ -53,39 +70,43 @@
>            | the error stacktrace (which tend to be very verbose and for little value).
>            | If you need the stacktraces, uncomment the second format below.
>            +-->
> -      <format type="cocoon">%7.7{priority} %{time} [%{category}] (%{uri}) %{thread}/%{class:short}: %{message}\n</format>
> -      <!--format type="cocoon">%7.7{priority} %{time} [%{category}] (%{uri}) %{thread}/%{class:short}: %{message}\n%{throwable}</format-->
> +      <format type="cocoon">%5.5{priority} %{time} [%{category}] (%{uri}) %{thread}/%{class:short}: %{message}\n</format>
> +      <!--format type="cocoon">%5.5{priority} %{time} [%{category}] (%{uri}) %{thread}/%{class:short}: %{message}\n%{throwable}</format-->
>  
>        <!--+
>            | <append> if set to 'true' will make cocoon append the events
>            | to the existing file, if set to 'false' cocoon will override
>            | the existing ones at every new start.
>            +-->
> -      <append>true</append>
> +      <append>false</append>

wouldn't it be better to append data by default? or at least make it 
configurable during build just like log level?

-- 
Leszek Gawron                                                 MobileBox
lgawron@apache.org                              http://www.mobilebox.pl

Re: svn commit: r159493 - cocoon/trunk/src/webapp/WEB-INF/logkit.xconf

Posted by Leszek Gawron <lg...@mobilebox.pl>.
Carsten Ziegeler wrote:
> Leszek Gawron wrote:
> 
>>>With 2.2 we can make it configurable using properties; so you can
>>>overwrite the value on startup.
>>>
>>>Carsten
>>>
>>>
>>
>>how ? :)
>>
> 
> With Cocoon 2.2 you can provide a cocoon-settings.properties file in the
> context directory of Cocoon and specify any property in this file, like
> hello=world
> 
> Then you can use these properties throughout the cocoon.xconf and logkit
> configuration: ${hello}.
> 
> With the cocoon-settings.properties you provide the default value and
> via commandline you can override the values "-Dhello=cocoon".
> 
> [1] has a short description; I will write some config docs as soon as I
> have more time :(
> 
> Carsten
> 
> [1] http://marc.theaimsgroup.com/?l=xml-cocoon-dev&m=110857478808645&w=2
That is a very nice feature. I really think we should start a multi user 
blog. It would be much easier for you to report that to the world. We 
could introduce some blog entry states like
- "to be documented"
- "ported to official docs"

Then you just put the blog post on the web and after sometime even 
someone else may just pick it up and put into official documentation.

-- 
Leszek Gawron                                      lgawron@mobilebox.pl
IT Manager                                         MobileBox sp. z o.o.
+48 (61) 855 06 67                              http://www.mobilebox.pl
mobile: +48 (501) 720 812                       fax: +48 (61) 853 29 65

Re: svn commit: r159493 - cocoon/trunk/src/webapp/WEB-INF/logkit.xconf

Posted by Carsten Ziegeler <cz...@apache.org>.
Leszek Gawron wrote:
>>
>>With 2.2 we can make it configurable using properties; so you can
>>overwrite the value on startup.
>>
>>Carsten
>>
>>
> 
> how ? :)
> 
With Cocoon 2.2 you can provide a cocoon-settings.properties file in the
context directory of Cocoon and specify any property in this file, like
hello=world

Then you can use these properties throughout the cocoon.xconf and logkit
configuration: ${hello}.

With the cocoon-settings.properties you provide the default value and
via commandline you can override the values "-Dhello=cocoon".

[1] has a short description; I will write some config docs as soon as I
have more time :(

Carsten

[1] http://marc.theaimsgroup.com/?l=xml-cocoon-dev&m=110857478808645&w=2

-- 
Carsten Ziegeler - Open Source Group, S&N AG
http://www.s-und-n.de
http://www.osoco.org/weblogs/rael/

Re: svn commit: r159493 - cocoon/trunk/src/webapp/WEB-INF/logkit.xconf

Posted by Leszek Gawron <lg...@mobilebox.pl>.
Carsten Ziegeler wrote:
> Vadim Gritsenko wrote:
> 
>>Leszek Gawron wrote:
>>
>>
>>>vgritsenko@apache.org wrote:
>>>
>>>
>>>
>>>>-      <append>true</append>
>>>>+      <append>false</append>
>>>
>>>
>>>wouldn't it be better to append data by default?
>>
>>
>>No.
>>
>>For production config, you want append=true, and rotation (type=unique).
>>For development config, you want append=false, and no rotation.
>>
>>Previously it was neither. Now it's development.
>>
>>
>>
>>
>>>or at least make it 
>>>configurable during build just like log level?
>>
>>
>>I've got no problem with that, +1.
>>
> 
> With 2.2 we can make it configurable using properties; so you can
> overwrite the value on startup.
> 
> Carsten
> 
> 
how ? :)

-- 
Leszek Gawron                                      lgawron@mobilebox.pl
IT Manager                                         MobileBox sp. z o.o.
+48 (61) 855 06 67                              http://www.mobilebox.pl
mobile: +48 (501) 720 812                       fax: +48 (61) 853 29 65

Re: svn commit: r159493 - cocoon/trunk/src/webapp/WEB-INF/logkit.xconf

Posted by Carsten Ziegeler <cz...@apache.org>.
Vadim Gritsenko wrote:
> Leszek Gawron wrote:
> 
>>vgritsenko@apache.org wrote:
>>
>>
>>>-      <append>true</append>
>>>+      <append>false</append>
>>
>>
>>wouldn't it be better to append data by default?
> 
> 
> No.
> 
> For production config, you want append=true, and rotation (type=unique).
> For development config, you want append=false, and no rotation.
> 
> Previously it was neither. Now it's development.
> 
> 
> 
>>or at least make it 
>>configurable during build just like log level?
> 
> 
> I've got no problem with that, +1.
> 
With 2.2 we can make it configurable using properties; so you can
overwrite the value on startup.

Carsten


-- 
Carsten Ziegeler - Open Source Group, S&N AG
http://www.s-und-n.de
http://www.osoco.org/weblogs/rael/

Re: svn commit: r159493 - cocoon/trunk/src/webapp/WEB-INF/logkit.xconf

Posted by Vadim Gritsenko <va...@reverycodes.com>.
Leszek Gawron wrote:
> vgritsenko@apache.org wrote:
> 
>> -      <append>true</append>
>> +      <append>false</append>
> 
> 
> wouldn't it be better to append data by default?

No.

For production config, you want append=true, and rotation (type=unique).
For development config, you want append=false, and no rotation.

Previously it was neither. Now it's development.


> or at least make it 
> configurable during build just like log level?

I've got no problem with that, +1.

Vadim