You are viewing a plain text version of this content. The canonical link for it is here.
Posted to pluto-scm@portals.apache.org by cd...@apache.org on 2006/09/11 03:35:03 UTC
svn commit: r442062 - in /portals/pluto/trunk/pluto-portal: pom.xml
src/main/webapp/WEB-INF/classes/logging.properties
Author: cdoremus
Date: Sun Sep 10 18:35:02 2006
New Revision: 442062
URL: http://svn.apache.org/viewvc?view=rev&rev=442062
Log:
Added to support built-in Tomcat logging using java.util.logging.
Added:
portals/pluto/trunk/pluto-portal/src/main/webapp/WEB-INF/classes/logging.properties
Modified:
portals/pluto/trunk/pluto-portal/pom.xml
Modified: portals/pluto/trunk/pluto-portal/pom.xml
URL: http://svn.apache.org/viewvc/portals/pluto/trunk/pluto-portal/pom.xml?view=diff&rev=442062&r1=442061&r2=442062
==============================================================================
--- portals/pluto/trunk/pluto-portal/pom.xml (original)
+++ portals/pluto/trunk/pluto-portal/pom.xml Sun Sep 10 18:35:02 2006
@@ -73,14 +73,6 @@
</exclusions>
</dependency>
- <!-- Log4j as the underlying logging impl for commons-logging -->
- <dependency>
- <groupId>log4j</groupId>
- <artifactId>log4j</artifactId>
- <version>${log4j.version}</version>
- <scope>runtime</scope>
- </dependency>
-
<dependency>
<artifactId>junit</artifactId>
<groupId>junit</groupId>
Added: portals/pluto/trunk/pluto-portal/src/main/webapp/WEB-INF/classes/logging.properties
URL: http://svn.apache.org/viewvc/portals/pluto/trunk/pluto-portal/src/main/webapp/WEB-INF/classes/logging.properties?view=auto&rev=442062
==============================================================================
--- portals/pluto/trunk/pluto-portal/src/main/webapp/WEB-INF/classes/logging.properties (added)
+++ portals/pluto/trunk/pluto-portal/src/main/webapp/WEB-INF/classes/logging.properties Sun Sep 10 18:35:02 2006
@@ -0,0 +1,34 @@
+#
+# Copyright 2004-2006 The Apache Software Foundation
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
+# implied.
+#
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+handlers = org.apache.juli.FileHandler
+
+############################################################
+#
+# JDK 1.4+ logging handler specific properties for Pluto portal
+# using Tomcat's custom implementation of LogManager.
+# Logging level can be set to any one of:
+# SEVERE, WARN, INFO, CONFIG, FINE, FINEST or ALL.
+# See the Tomcat documentation for more details.
+#
+############################################################
+
+org.apache.juli.FileHandler.level = FINE
+org.apache.juli.FileHandler.directory = ${catalina.base}/logs
+#Log file will be named pluto.<yyyy-mm-dd>.log
+org.apache.juli.FileHandler.prefix = pluto.
+
+org.apache.pluto.level=FINE
\ No newline at end of file
Re: svn commit: r442062 - in /portals/pluto/trunk/pluto-portal: pom.xml
src/main/webapp/WEB-INF/classes/logging.properties
Posted by Craig Doremus <cd...@apache.org>.
David H. DeWolf wrote:
> Craig,
>
> I would really prefer that we DON'T add any logging configuration to
> the webapp. This should be configured outside the application so
> people can choose to configure it how they want to.
>
> Would you please revert these changes. If you'd like, you can add an
> example properties file into src/resources and make sure it doesn't
> get bundled with the app.
>
>
> Thanks,
>
>
> David
>
Hi David,
Adding the logging.properties file does not prevent any of our users
from configuring other logging implementations. It does enforce logging
to a file important information needed for bug reports. In Windows, the
default stdout logging often scrolls off the console. I'm only making it
easier for our users to give us complete information about bugs. In
addition, Tomcat's built-in juli (java.util.logging) logging is very low
impact in that it does not require any extra jars. We did this in Pluto
1.0.1, and I have not heard any complaints about it.
If you still want to get rid of the logging, I'll do that, but I would
like to add this file (logging.properties) to the binary distribution
built with Tomcat. However, I'm not sure how to contingently add this
file to the pluto war file using Maven's packaging goal, so I'll need
some help with this configuration.
TIA
/Craig
> cdoremus@apache.org wrote:
>
>> Author: cdoremus
>> Date: Sun Sep 10 18:35:02 2006
>> New Revision: 442062
>>
>> URL: http://svn.apache.org/viewvc?view=rev&rev=442062
>> Log:
>> Added to support built-in Tomcat logging using java.util.logging.
>>
>> Added:
>>
>> portals/pluto/trunk/pluto-portal/src/main/webapp/WEB-INF/classes/logging.properties
>>
>> Modified:
>> portals/pluto/trunk/pluto-portal/pom.xml
>>
>> Modified: portals/pluto/trunk/pluto-portal/pom.xml
>> URL:
>> http://svn.apache.org/viewvc/portals/pluto/trunk/pluto-portal/pom.xml?view=diff&rev=442062&r1=442061&r2=442062
>>
>> ==============================================================================
>>
>> --- portals/pluto/trunk/pluto-portal/pom.xml (original)
>> +++ portals/pluto/trunk/pluto-portal/pom.xml Sun Sep 10 18:35:02 2006
>> @@ -73,14 +73,6 @@
>> </exclusions>
>> </dependency>
>>
>> - <!-- Log4j as the underlying logging impl for
>> commons-logging -->
>> - <dependency>
>> - <groupId>log4j</groupId>
>> - <artifactId>log4j</artifactId>
>> - <version>${log4j.version}</version>
>> - <scope>runtime</scope>
>> - </dependency>
>> -
>> <dependency>
>> <artifactId>junit</artifactId>
>> <groupId>junit</groupId>
>>
>> Added:
>> portals/pluto/trunk/pluto-portal/src/main/webapp/WEB-INF/classes/logging.properties
>>
>> URL:
>> http://svn.apache.org/viewvc/portals/pluto/trunk/pluto-portal/src/main/webapp/WEB-INF/classes/logging.properties?view=auto&rev=442062
>>
>> ==============================================================================
>>
>> ---
>> portals/pluto/trunk/pluto-portal/src/main/webapp/WEB-INF/classes/logging.properties
>> (added)
>> +++
>> portals/pluto/trunk/pluto-portal/src/main/webapp/WEB-INF/classes/logging.properties
>> Sun Sep 10 18:35:02 2006
>> @@ -0,0 +1,34 @@
>> +# +# Copyright 2004-2006 The Apache Software Foundation
>> +# Licensed under the Apache License, Version 2.0 (the "License");
>> +# you may not use this file except in compliance with the License.
>> +# You may obtain a copy of the License at +# +#
>> http://www.apache.org/licenses/LICENSE-2.0
>> +# +# Unless required by applicable law or agreed to in writing,
>> software
>> +# distributed under the License is distributed on an "AS IS" BASIS,
>> +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
>> +# implied.
>> +# +# See the License for the specific language governing
>> permissions and
>> +# limitations under the License.
>> +
>> +handlers = org.apache.juli.FileHandler
>> +
>> +############################################################
>> +#
>> +# JDK 1.4+ logging handler specific properties for Pluto portal
>> +# using Tomcat's custom implementation of LogManager.
>> +# Logging level can be set to any one of:
>> +# SEVERE, WARN, INFO, CONFIG, FINE, FINEST or ALL. +# See the Tomcat
>> documentation for more details.
>> +# +############################################################
>> +
>> +org.apache.juli.FileHandler.level = FINE
>> +org.apache.juli.FileHandler.directory = ${catalina.base}/logs
>> +#Log file will be named pluto.<yyyy-mm-dd>.log
>> +org.apache.juli.FileHandler.prefix = pluto.
>> +
>> +org.apache.pluto.level=FINE
>> \ No newline at end of file
>>
>>
>>
>
>
Re: svn commit: r442062 - in /portals/pluto/trunk/pluto-portal: pom.xml
src/main/webapp/WEB-INF/classes/logging.properties
Posted by "David H. DeWolf" <dd...@apache.org>.
Craig,
I would really prefer that we DON'T add any logging configuration to the
webapp. This should be configured outside the application so people can
choose to configure it how they want to.
Would you please revert these changes. If you'd like, you can add an
example properties file into src/resources and make sure it doesn't get
bundled with the app.
Thanks,
David
cdoremus@apache.org wrote:
> Author: cdoremus
> Date: Sun Sep 10 18:35:02 2006
> New Revision: 442062
>
> URL: http://svn.apache.org/viewvc?view=rev&rev=442062
> Log:
> Added to support built-in Tomcat logging using java.util.logging.
>
> Added:
> portals/pluto/trunk/pluto-portal/src/main/webapp/WEB-INF/classes/logging.properties
> Modified:
> portals/pluto/trunk/pluto-portal/pom.xml
>
> Modified: portals/pluto/trunk/pluto-portal/pom.xml
> URL: http://svn.apache.org/viewvc/portals/pluto/trunk/pluto-portal/pom.xml?view=diff&rev=442062&r1=442061&r2=442062
> ==============================================================================
> --- portals/pluto/trunk/pluto-portal/pom.xml (original)
> +++ portals/pluto/trunk/pluto-portal/pom.xml Sun Sep 10 18:35:02 2006
> @@ -73,14 +73,6 @@
> </exclusions>
> </dependency>
>
> - <!-- Log4j as the underlying logging impl for commons-logging -->
> - <dependency>
> - <groupId>log4j</groupId>
> - <artifactId>log4j</artifactId>
> - <version>${log4j.version}</version>
> - <scope>runtime</scope>
> - </dependency>
> -
> <dependency>
> <artifactId>junit</artifactId>
> <groupId>junit</groupId>
>
> Added: portals/pluto/trunk/pluto-portal/src/main/webapp/WEB-INF/classes/logging.properties
> URL: http://svn.apache.org/viewvc/portals/pluto/trunk/pluto-portal/src/main/webapp/WEB-INF/classes/logging.properties?view=auto&rev=442062
> ==============================================================================
> --- portals/pluto/trunk/pluto-portal/src/main/webapp/WEB-INF/classes/logging.properties (added)
> +++ portals/pluto/trunk/pluto-portal/src/main/webapp/WEB-INF/classes/logging.properties Sun Sep 10 18:35:02 2006
> @@ -0,0 +1,34 @@
> +#
> +# Copyright 2004-2006 The Apache Software Foundation
> +# Licensed under the Apache License, Version 2.0 (the "License");
> +# you may not use this file except in compliance with the License.
> +# You may obtain a copy of the License at
> +#
> +# http://www.apache.org/licenses/LICENSE-2.0
> +#
> +# Unless required by applicable law or agreed to in writing, software
> +# distributed under the License is distributed on an "AS IS" BASIS,
> +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
> +# implied.
> +#
> +# See the License for the specific language governing permissions and
> +# limitations under the License.
> +
> +handlers = org.apache.juli.FileHandler
> +
> +############################################################
> +#
> +# JDK 1.4+ logging handler specific properties for Pluto portal
> +# using Tomcat's custom implementation of LogManager.
> +# Logging level can be set to any one of:
> +# SEVERE, WARN, INFO, CONFIG, FINE, FINEST or ALL.
> +# See the Tomcat documentation for more details.
> +#
> +############################################################
> +
> +org.apache.juli.FileHandler.level = FINE
> +org.apache.juli.FileHandler.directory = ${catalina.base}/logs
> +#Log file will be named pluto.<yyyy-mm-dd>.log
> +org.apache.juli.FileHandler.prefix = pluto.
> +
> +org.apache.pluto.level=FINE
> \ No newline at end of file
>
>
>