You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by ke...@apache.org on 2006/05/06 06:03:31 UTC
svn commit: r400233 - in /geronimo/branches/1.1:
applications/daytrader/core/ applications/daytrader/ear/
applications/daytrader/ejb/ applications/daytrader/streamer/
applications/daytrader/web/ applications/daytrader/wsappclient/
applications/jmxdebug...
Author: kevan
Date: Fri May 5 21:03:29 2006
New Revision: 400233
URL: http://svn.apache.org/viewcvs?rev=400233&view=rev
Log:
GERONIMO-1640 Upgrade Tomcat version to 5.5.15. For compliant jsp operation, upgrade required that Jasper config option 'development' be configured to false. So, involved a bit more than simple version change...
Added:
geronimo/branches/1.1/applications/daytrader/core/
geronimo/branches/1.1/applications/daytrader/ear/
geronimo/branches/1.1/applications/daytrader/ejb/
geronimo/branches/1.1/applications/daytrader/streamer/
geronimo/branches/1.1/applications/daytrader/web/
geronimo/branches/1.1/applications/daytrader/wsappclient/
geronimo/branches/1.1/applications/jmxdebug/
geronimo/branches/1.1/modules/interop/
Modified:
geronimo/branches/1.1/configs/jetty-deployer/src/plan/plan.xml
geronimo/branches/1.1/etc/explicit_versions.properties
geronimo/branches/1.1/etc/project.properties
geronimo/branches/1.1/modules/jetty-builder/src/java/org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java
geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/geronimo-tomcat/var/catalina/conf/web.xml
Modified: geronimo/branches/1.1/configs/jetty-deployer/src/plan/plan.xml
URL: http://svn.apache.org/viewcvs/geronimo/branches/1.1/configs/jetty-deployer/src/plan/plan.xml?rev=400233&r1=400232&r2=400233&view=diff
==============================================================================
--- geronimo/branches/1.1/configs/jetty-deployer/src/plan/plan.xml (original)
+++ geronimo/branches/1.1/configs/jetty-deployer/src/plan/plan.xml Fri May 5 21:03:29 2006
@@ -93,8 +93,9 @@
<attribute name="servletClass">org.apache.jasper.servlet.JspServlet</attribute>
<attribute name="loadOnStartup">0</attribute>
<attribute name="initParams">
- logVerbosityLevel=DEBUG
+ development=false
fork=false
+ logVerbosityLevel=DEBUG
xpoweredBy=false</attribute>
<attribute name="servletMappings">*.jsp,*.jspf,*.jspx,*.xsp</attribute>
</gbean>
Modified: geronimo/branches/1.1/etc/explicit_versions.properties
URL: http://svn.apache.org/viewcvs/geronimo/branches/1.1/etc/explicit_versions.properties?rev=400233&r1=400232&r2=400233&view=diff
==============================================================================
--- geronimo/branches/1.1/etc/explicit_versions.properties (original)
+++ geronimo/branches/1.1/etc/explicit_versions.properties Fri May 5 21:03:29 2006
@@ -45,7 +45,7 @@
howl///=0.1.11
#security:
hsqldb///=1.7.2.2
-jasper///=5.5.12
+jasper///=5.5.15
javacc///=2.1
jdbm///=0.20-dev
jdom///=1.0
@@ -73,8 +73,8 @@
standard_taglibs///=1.1.1
stax/stax//jar=1.1.1-dev
stax/stax_api//jar=1.0
-tomcat_ajp///=5.5.9
-tomcat///=5.5.9
+tomcat_ajp///=5.5.15
+tomcat///=5.5.15
tomcat_servlet_examples///=5.5.15
tomcat_jsp_examples///=5.5.15
wadi///=2.0M1
Modified: geronimo/branches/1.1/etc/project.properties
URL: http://svn.apache.org/viewcvs/geronimo/branches/1.1/etc/project.properties?rev=400233&r1=400232&r2=400233&view=diff
==============================================================================
--- geronimo/branches/1.1/etc/project.properties (original)
+++ geronimo/branches/1.1/etc/project.properties Fri May 5 21:03:29 2006
@@ -155,7 +155,7 @@
howl_version=0.1.11
#security:
hsqldb_version=1.7.2.2
-jasper_version=5.5.12
+jasper_version=5.5.15
javacc_version=2.1
jdbm_version=0.20-dev
jdom_version=1.0
@@ -183,8 +183,8 @@
standard_taglibs_version=1.1.1
stax_version=1.1.1-dev
stax_api_version=1.0
-tomcat_ajp_version=5.5.9
-tomcat_version=5.5.9
+tomcat_ajp_version=5.5.15
+tomcat_version=5.5.15
tomcat_servlet_examples_version=5.5.15
tomcat_jsp_examples_version=5.5.15
wadi_version=2.0M1
Modified: geronimo/branches/1.1/modules/jetty-builder/src/java/org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java
URL: http://svn.apache.org/viewcvs/geronimo/branches/1.1/modules/jetty-builder/src/java/org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java?rev=400233&r1=400232&r2=400233&view=diff
==============================================================================
--- geronimo/branches/1.1/modules/jetty-builder/src/java/org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java (original)
+++ geronimo/branches/1.1/modules/jetty-builder/src/java/org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java Fri May 5 21:03:29 2006
@@ -840,6 +840,7 @@
String servletName = servletType.getServletName().getStringValue().trim();
AbstractName servletAbstractName = earContext.getNaming().createChildName(webModuleName, servletName, NameFactory.SERVLET);
GBeanData servletData;
+ Map initParams = new HashMap();
if (servletType.isSetServletClass()) {
String servletClassName = servletType.getServletClass().getStringValue().trim();
Class servletClass;
@@ -872,6 +873,7 @@
servletData.setAttribute("jspFile", servletType.getJspFile().getStringValue().trim());
//TODO MAKE THIS CONFIGURABLE!!! Jetty uses the servlet mapping set up from the default-web.xml
servletData.setAttribute("servletClass", "org.apache.jasper.servlet.JspServlet");
+ initParams.put("development", "false");
} else {
throw new DeploymentException("Neither servlet class nor jsp file is set for " + servletName); // TODO identify web app in message
}
@@ -886,7 +888,6 @@
//TODO in init param setter, add classpath if jspFile is not null.
servletData.setReferencePattern("JettyServletRegistration", webModuleName);
servletData.setAttribute("servletName", servletName);
- Map initParams = new HashMap();
ParamValueType[] initParamArray = servletType.getInitParamArray();
for (int j = 0; j < initParamArray.length; j++) {
ParamValueType paramValueType = initParamArray[j];
Modified: geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/geronimo-tomcat/var/catalina/conf/web.xml
URL: http://svn.apache.org/viewcvs/geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/geronimo-tomcat/var/catalina/conf/web.xml?rev=400233&r1=400232&r2=400233&view=diff
==============================================================================
--- geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/geronimo-tomcat/var/catalina/conf/web.xml (original)
+++ geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/geronimo-tomcat/var/catalina/conf/web.xml Fri May 5 21:03:29 2006
@@ -187,6 +187,10 @@
<servlet-name>jsp</servlet-name>
<servlet-class>org.apache.jasper.servlet.JspServlet</servlet-class>
<init-param>
+ <param-name>development</param-name>
+ <param-value>false</param-value>
+ </init-param>
+ <init-param>
<param-name>fork</param-name>
<param-value>false</param-value>
</init-param>
Re: svn commit: r400233 - in /geronimo/branches/1.1: applications/daytrader/core/
applications/daytrader/ear/ applications/daytrader/ejb/ applications/daytrader/streamer/
applications/daytrader/web/ applications/daytrader/wsappclient/ applications/jmxdebug...
Posted by Matt Hogstrom <ma...@hogstrom.org>.
Thanks Kevan and John.
Kevan Miller wrote:
>
> On May 6, 2006, at 11:05 AM, Matt Hogstrom wrote:
>
>> John,
>>
>> Is there an issue in DayTrader that required us to add the directories
>> back into the branch? I'm working to completely remove it. Let me
>> know what the problem is and we can work to remove the monster.
>
> John didn't add them, I did -- not intentionally... The dirs didn't show
> up in a diff I ran before committing. I'm not sure why the dirs were
> still there... Is there some procedure that I'm not following properly?
> I'll consult the svn doc, when I have a few moments...
>
> --kevan
>
>
>>
>> John Sisson wrote:
>>> FYI, I removed the directories added in this commit. See
>>> URL: http://svn.apache.org/viewcvs?rev=400273&view=rev
>>> Log:
>>> GERONIMO-1640 Delete directories that were accidentally added in
>>> previous checkin.
>>> John
>>> kevan@apache.org wrote:
>>>> Author: kevan
>>>> Date: Fri May 5 21:03:29 2006
>>>> New Revision: 400233
>>>>
>>>> URL: http://svn.apache.org/viewcvs?rev=400233&view=rev
>>>> Log:
>>>> GERONIMO-1640 Upgrade Tomcat version to 5.5.15. For compliant jsp
>>>> operation, upgrade required that Jasper config option 'development'
>>>> be configured to false. So, involved a bit more than simple version
>>>> change...
>>>>
>>>> Added:
>>>> geronimo/branches/1.1/applications/daytrader/core/
>>>> geronimo/branches/1.1/applications/daytrader/ear/
>>>> geronimo/branches/1.1/applications/daytrader/ejb/
>>>> geronimo/branches/1.1/applications/daytrader/streamer/
>>>> geronimo/branches/1.1/applications/daytrader/web/
>>>> geronimo/branches/1.1/applications/daytrader/wsappclient/
>>>> geronimo/branches/1.1/applications/jmxdebug/
>>>> geronimo/branches/1.1/modules/interop/
>>>> Modified:
>>>> geronimo/branches/1.1/configs/jetty-deployer/src/plan/plan.xml
>>>> geronimo/branches/1.1/etc/explicit_versions.properties
>>>> geronimo/branches/1.1/etc/project.properties
>>>>
>>>> geronimo/branches/1.1/modules/jetty-builder/src/java/org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java
>>>>
>>>>
>>>> geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/geronimo-tomcat/var/catalina/conf/web.xml
>>>>
>>>>
>>>> Modified:
>>>> geronimo/branches/1.1/configs/jetty-deployer/src/plan/plan.xml
>>>> URL:
>>>> http://svn.apache.org/viewcvs/geronimo/branches/1.1/configs/jetty-deployer/src/plan/plan.xml?rev=400233&r1=400232&r2=400233&view=diff
>>>>
>>>> ==============================================================================
>>>>
>>>> --- geronimo/branches/1.1/configs/jetty-deployer/src/plan/plan.xml
>>>> (original)
>>>> +++ geronimo/branches/1.1/configs/jetty-deployer/src/plan/plan.xml
>>>> Fri May 5 21:03:29 2006
>>>> @@ -93,8 +93,9 @@
>>>> <attribute
>>>> name="servletClass">org.apache.jasper.servlet.JspServlet</attribute>
>>>> <attribute name="loadOnStartup">0</attribute>
>>>> <attribute name="initParams">
>>>> - logVerbosityLevel=DEBUG
>>>> + development=false
>>>> fork=false
>>>> + logVerbosityLevel=DEBUG
>>>> xpoweredBy=false</attribute>
>>>> <attribute
>>>> name="servletMappings">*.jsp,*.jspf,*.jspx,*.xsp</attribute>
>>>> </gbean>
>>>>
>>>> Modified: geronimo/branches/1.1/etc/explicit_versions.properties
>>>> URL:
>>>> http://svn.apache.org/viewcvs/geronimo/branches/1.1/etc/explicit_versions.properties?rev=400233&r1=400232&r2=400233&view=diff
>>>>
>>>> ==============================================================================
>>>>
>>>> --- geronimo/branches/1.1/etc/explicit_versions.properties (original)
>>>> +++ geronimo/branches/1.1/etc/explicit_versions.properties Fri May
>>>> 5 21:03:29 2006
>>>> @@ -45,7 +45,7 @@
>>>> howl///=0.1.11
>>>> #security:
>>>> hsqldb///=1.7.2.2
>>>> -jasper///=5.5.12
>>>> +jasper///=5.5.15
>>>> javacc///=2.1
>>>> jdbm///=0.20-dev
>>>> jdom///=1.0
>>>> @@ -73,8 +73,8 @@
>>>> standard_taglibs///=1.1.1
>>>> stax/stax//jar=1.1.1-dev
>>>> stax/stax_api//jar=1.0
>>>> -tomcat_ajp///=5.5.9
>>>> -tomcat///=5.5.9
>>>> +tomcat_ajp///=5.5.15
>>>> +tomcat///=5.5.15
>>>> tomcat_servlet_examples///=5.5.15
>>>> tomcat_jsp_examples///=5.5.15
>>>> wadi///=2.0M1
>>>>
>>>> Modified: geronimo/branches/1.1/etc/project.properties
>>>> URL:
>>>> http://svn.apache.org/viewcvs/geronimo/branches/1.1/etc/project.properties?rev=400233&r1=400232&r2=400233&view=diff
>>>>
>>>> ==============================================================================
>>>>
>>>> --- geronimo/branches/1.1/etc/project.properties (original)
>>>> +++ geronimo/branches/1.1/etc/project.properties Fri May 5 21:03:29
>>>> 2006
>>>> @@ -155,7 +155,7 @@
>>>> howl_version=0.1.11
>>>> #security:
>>>> hsqldb_version=1.7.2.2
>>>> -jasper_version=5.5.12
>>>> +jasper_version=5.5.15
>>>> javacc_version=2.1
>>>> jdbm_version=0.20-dev
>>>> jdom_version=1.0
>>>> @@ -183,8 +183,8 @@
>>>> standard_taglibs_version=1.1.1
>>>> stax_version=1.1.1-dev
>>>> stax_api_version=1.0
>>>> -tomcat_ajp_version=5.5.9
>>>> -tomcat_version=5.5.9
>>>> +tomcat_ajp_version=5.5.15
>>>> +tomcat_version=5.5.15
>>>> tomcat_servlet_examples_version=5.5.15
>>>> tomcat_jsp_examples_version=5.5.15
>>>> wadi_version=2.0M1
>>>>
>>>> Modified:
>>>> geronimo/branches/1.1/modules/jetty-builder/src/java/org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java
>>>>
>>>> URL:
>>>> http://svn.apache.org/viewcvs/geronimo/branches/1.1/modules/jetty-builder/src/java/org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java?rev=400233&r1=400232&r2=400233&view=diff
>>>>
>>>> ==============================================================================
>>>>
>>>> ---
>>>> geronimo/branches/1.1/modules/jetty-builder/src/java/org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java
>>>> (original)
>>>> +++
>>>> geronimo/branches/1.1/modules/jetty-builder/src/java/org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java
>>>> Fri May 5 21:03:29 2006
>>>> @@ -840,6 +840,7 @@
>>>> String servletName =
>>>> servletType.getServletName().getStringValue().trim();
>>>> AbstractName servletAbstractName =
>>>> earContext.getNaming().createChildName(webModuleName, servletName,
>>>> NameFactory.SERVLET);
>>>> GBeanData servletData;
>>>> + Map initParams = new HashMap();
>>>> if (servletType.isSetServletClass()) {
>>>> String servletClassName =
>>>> servletType.getServletClass().getStringValue().trim();
>>>> Class servletClass;
>>>> @@ -872,6 +873,7 @@
>>>> servletData.setAttribute("jspFile",
>>>> servletType.getJspFile().getStringValue().trim());
>>>> //TODO MAKE THIS CONFIGURABLE!!! Jetty uses the servlet
>>>> mapping set up from the default-web.xml
>>>> servletData.setAttribute("servletClass",
>>>> "org.apache.jasper.servlet.JspServlet");
>>>> + initParams.put("development", "false");
>>>> } else {
>>>> throw new DeploymentException("Neither servlet class
>>>> nor jsp file is set for " + servletName); // TODO identify web app
>>>> in message
>>>> }
>>>> @@ -886,7 +888,6 @@
>>>> //TODO in init param setter, add classpath if jspFile is
>>>> not null.
>>>> servletData.setReferencePattern("JettyServletRegistration",
>>>> webModuleName);
>>>> servletData.setAttribute("servletName", servletName);
>>>> - Map initParams = new HashMap();
>>>> ParamValueType[] initParamArray =
>>>> servletType.getInitParamArray();
>>>> for (int j = 0; j < initParamArray.length; j++) {
>>>> ParamValueType paramValueType = initParamArray[j];
>>>>
>>>> Modified:
>>>> geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/geronimo-tomcat/var/catalina/conf/web.xml
>>>>
>>>> URL:
>>>> http://svn.apache.org/viewcvs/geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/geronimo-tomcat/var/catalina/conf/web.xml?rev=400233&r1=400232&r2=400233&view=diff
>>>>
>>>> ==============================================================================
>>>>
>>>> ---
>>>> geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/geronimo-tomcat/var/catalina/conf/web.xml
>>>> (original)
>>>> +++
>>>> geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/geronimo-tomcat/var/catalina/conf/web.xml
>>>> Fri May 5 21:03:29 2006
>>>> @@ -187,6 +187,10 @@
>>>> <servlet-name>jsp</servlet-name>
>>>>
>>>> <servlet-class>org.apache.jasper.servlet.JspServlet</servlet-class>
>>>> <init-param>
>>>> + <param-name>development</param-name>
>>>> + <param-value>false</param-value>
>>>> + </init-param>
>>>> + <init-param>
>>>> <param-name>fork</param-name>
>>>> <param-value>false</param-value>
>>>> </init-param>
>>>>
>>>>
>>>>
>>>>
>
>
>
>
Re: svn commit: r400233 - in /geronimo/branches/1.1: applications/daytrader/core/ applications/daytrader/ear/ applications/daytrader/ejb/ applications/daytrader/streamer/ applications/daytrader/web/ applications/daytrader/wsappclient/ applications/jmxdebug...
Posted by Kevan Miller <ke...@gmail.com>.
On May 6, 2006, at 11:05 AM, Matt Hogstrom wrote:
> John,
>
> Is there an issue in DayTrader that required us to add the
> directories back into the branch? I'm working to completely remove
> it. Let me know what the problem is and we can work to remove the
> monster.
John didn't add them, I did -- not intentionally... The dirs didn't
show up in a diff I ran before committing. I'm not sure why the dirs
were still there... Is there some procedure that I'm not following
properly? I'll consult the svn doc, when I have a few moments...
--kevan
>
> John Sisson wrote:
>> FYI, I removed the directories added in this commit. See
>> URL: http://svn.apache.org/viewcvs?rev=400273&view=rev
>> Log:
>> GERONIMO-1640 Delete directories that were accidentally added in
>> previous checkin.
>> John
>> kevan@apache.org wrote:
>>> Author: kevan
>>> Date: Fri May 5 21:03:29 2006
>>> New Revision: 400233
>>>
>>> URL: http://svn.apache.org/viewcvs?rev=400233&view=rev
>>> Log:
>>> GERONIMO-1640 Upgrade Tomcat version to 5.5.15. For compliant jsp
>>> operation, upgrade required that Jasper config option
>>> 'development' be configured to false. So, involved a bit more
>>> than simple version change...
>>>
>>> Added:
>>> geronimo/branches/1.1/applications/daytrader/core/
>>> geronimo/branches/1.1/applications/daytrader/ear/
>>> geronimo/branches/1.1/applications/daytrader/ejb/
>>> geronimo/branches/1.1/applications/daytrader/streamer/
>>> geronimo/branches/1.1/applications/daytrader/web/
>>> geronimo/branches/1.1/applications/daytrader/wsappclient/
>>> geronimo/branches/1.1/applications/jmxdebug/
>>> geronimo/branches/1.1/modules/interop/
>>> Modified:
>>> geronimo/branches/1.1/configs/jetty-deployer/src/plan/plan.xml
>>> geronimo/branches/1.1/etc/explicit_versions.properties
>>> geronimo/branches/1.1/etc/project.properties
>>> geronimo/branches/1.1/modules/jetty-builder/src/java/org/
>>> apache/geronimo/jetty/deployment/JettyModuleBuilder.java
>>> geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/
>>> geronimo-tomcat/var/catalina/conf/web.xml
>>>
>>> Modified: geronimo/branches/1.1/configs/jetty-deployer/src/plan/
>>> plan.xml
>>> URL: http://svn.apache.org/viewcvs/geronimo/branches/1.1/configs/
>>> jetty-deployer/src/plan/plan.xml?
>>> rev=400233&r1=400232&r2=400233&view=diff
>>> ====================================================================
>>> ==========
>>> --- geronimo/branches/1.1/configs/jetty-deployer/src/plan/
>>> plan.xml (original)
>>> +++ geronimo/branches/1.1/configs/jetty-deployer/src/plan/
>>> plan.xml Fri May 5 21:03:29 2006
>>> @@ -93,8 +93,9 @@
>>> <attribute
>>> name="servletClass">org.apache.jasper.servlet.JspServlet</attribute>
>>> <attribute name="loadOnStartup">0</attribute>
>>> <attribute name="initParams">
>>> - logVerbosityLevel=DEBUG
>>> + development=false
>>> fork=false
>>> + logVerbosityLevel=DEBUG
>>> xpoweredBy=false</attribute>
>>> <attribute
>>> name="servletMappings">*.jsp,*.jspf,*.jspx,*.xsp</attribute>
>>> </gbean>
>>>
>>> Modified: geronimo/branches/1.1/etc/explicit_versions.properties
>>> URL: http://svn.apache.org/viewcvs/geronimo/branches/1.1/etc/
>>> explicit_versions.properties?
>>> rev=400233&r1=400232&r2=400233&view=diff
>>> ====================================================================
>>> ==========
>>> --- geronimo/branches/1.1/etc/explicit_versions.properties
>>> (original)
>>> +++ geronimo/branches/1.1/etc/explicit_versions.properties Fri
>>> May 5 21:03:29 2006
>>> @@ -45,7 +45,7 @@
>>> howl///=0.1.11
>>> #security:
>>> hsqldb///=1.7.2.2
>>> -jasper///=5.5.12
>>> +jasper///=5.5.15
>>> javacc///=2.1
>>> jdbm///=0.20-dev
>>> jdom///=1.0
>>> @@ -73,8 +73,8 @@
>>> standard_taglibs///=1.1.1
>>> stax/stax//jar=1.1.1-dev
>>> stax/stax_api//jar=1.0
>>> -tomcat_ajp///=5.5.9
>>> -tomcat///=5.5.9
>>> +tomcat_ajp///=5.5.15
>>> +tomcat///=5.5.15
>>> tomcat_servlet_examples///=5.5.15
>>> tomcat_jsp_examples///=5.5.15
>>> wadi///=2.0M1
>>>
>>> Modified: geronimo/branches/1.1/etc/project.properties
>>> URL: http://svn.apache.org/viewcvs/geronimo/branches/1.1/etc/
>>> project.properties?rev=400233&r1=400232&r2=400233&view=diff
>>> ====================================================================
>>> ==========
>>> --- geronimo/branches/1.1/etc/project.properties (original)
>>> +++ geronimo/branches/1.1/etc/project.properties Fri May 5
>>> 21:03:29 2006
>>> @@ -155,7 +155,7 @@
>>> howl_version=0.1.11
>>> #security:
>>> hsqldb_version=1.7.2.2
>>> -jasper_version=5.5.12
>>> +jasper_version=5.5.15
>>> javacc_version=2.1
>>> jdbm_version=0.20-dev
>>> jdom_version=1.0
>>> @@ -183,8 +183,8 @@
>>> standard_taglibs_version=1.1.1
>>> stax_version=1.1.1-dev
>>> stax_api_version=1.0
>>> -tomcat_ajp_version=5.5.9
>>> -tomcat_version=5.5.9
>>> +tomcat_ajp_version=5.5.15
>>> +tomcat_version=5.5.15
>>> tomcat_servlet_examples_version=5.5.15
>>> tomcat_jsp_examples_version=5.5.15
>>> wadi_version=2.0M1
>>>
>>> Modified: geronimo/branches/1.1/modules/jetty-builder/src/java/
>>> org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java
>>> URL: http://svn.apache.org/viewcvs/geronimo/branches/1.1/modules/
>>> jetty-builder/src/java/org/apache/geronimo/jetty/deployment/
>>> JettyModuleBuilder.java?rev=400233&r1=400232&r2=400233&view=diff
>>> ====================================================================
>>> ==========
>>> --- geronimo/branches/1.1/modules/jetty-builder/src/java/org/
>>> apache/geronimo/jetty/deployment/JettyModuleBuilder.java (original)
>>> +++ geronimo/branches/1.1/modules/jetty-builder/src/java/org/
>>> apache/geronimo/jetty/deployment/JettyModuleBuilder.java Fri May
>>> 5 21:03:29 2006
>>> @@ -840,6 +840,7 @@
>>> String servletName = servletType.getServletName
>>> ().getStringValue().trim();
>>> AbstractName servletAbstractName = earContext.getNaming
>>> ().createChildName(webModuleName, servletName, NameFactory.SERVLET);
>>> GBeanData servletData;
>>> + Map initParams = new HashMap();
>>> if (servletType.isSetServletClass()) {
>>> String servletClassName = servletType.getServletClass
>>> ().getStringValue().trim();
>>> Class servletClass;
>>> @@ -872,6 +873,7 @@
>>> servletData.setAttribute("jspFile",
>>> servletType.getJspFile().getStringValue().trim());
>>> //TODO MAKE THIS CONFIGURABLE!!! Jetty uses the
>>> servlet mapping set up from the default-web.xml
>>> servletData.setAttribute("servletClass",
>>> "org.apache.jasper.servlet.JspServlet");
>>> + initParams.put("development", "false");
>>> } else {
>>> throw new DeploymentException("Neither servlet class
>>> nor jsp file is set for " + servletName); // TODO identify web
>>> app in message
>>> }
>>> @@ -886,7 +888,6 @@
>>> //TODO in init param setter, add classpath if jspFile is
>>> not null.
>>> servletData.setReferencePattern
>>> ("JettyServletRegistration", webModuleName);
>>> servletData.setAttribute("servletName", servletName);
>>> - Map initParams = new HashMap();
>>> ParamValueType[] initParamArray =
>>> servletType.getInitParamArray();
>>> for (int j = 0; j < initParamArray.length; j++) {
>>> ParamValueType paramValueType = initParamArray[j];
>>>
>>> Modified: geronimo/branches/1.1/modules/tomcat/src/resources/META-
>>> INF/geronimo-tomcat/var/catalina/conf/web.xml
>>> URL: http://svn.apache.org/viewcvs/geronimo/branches/1.1/modules/
>>> tomcat/src/resources/META-INF/geronimo-tomcat/var/catalina/conf/
>>> web.xml?rev=400233&r1=400232&r2=400233&view=diff
>>> ====================================================================
>>> ==========
>>> --- geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/
>>> geronimo-tomcat/var/catalina/conf/web.xml (original)
>>> +++ geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/
>>> geronimo-tomcat/var/catalina/conf/web.xml Fri May 5 21:03:29 2006
>>> @@ -187,6 +187,10 @@
>>> <servlet-name>jsp</servlet-name>
>>> <servlet-class>org.apache.jasper.servlet.JspServlet</
>>> servlet-class>
>>> <init-param>
>>> + <param-name>development</param-name>
>>> + <param-value>false</param-value>
>>> + </init-param>
>>> + <init-param>
>>> <param-name>fork</param-name>
>>> <param-value>false</param-value>
>>> </init-param>
>>>
>>>
>>>
>>>
Re: svn commit: r400233 - in /geronimo/branches/1.1: applications/daytrader/core/
applications/daytrader/ear/ applications/daytrader/ejb/ applications/daytrader/streamer/
applications/daytrader/web/ applications/daytrader/wsappclient/ applications/jmxdebug...
Posted by Matt Hogstrom <ma...@hogstrom.org>.
John,
Is there an issue in DayTrader that required us to add the directories back into the branch? I'm
working to completely remove it. Let me know what the problem is and we can work to remove the monster.
John Sisson wrote:
> FYI, I removed the directories added in this commit. See
>
> URL: http://svn.apache.org/viewcvs?rev=400273&view=rev
> Log:
> GERONIMO-1640 Delete directories that were accidentally added in
> previous checkin.
>
> John
> kevan@apache.org wrote:
>> Author: kevan
>> Date: Fri May 5 21:03:29 2006
>> New Revision: 400233
>>
>> URL: http://svn.apache.org/viewcvs?rev=400233&view=rev
>> Log:
>> GERONIMO-1640 Upgrade Tomcat version to 5.5.15. For compliant jsp
>> operation, upgrade required that Jasper config option 'development' be
>> configured to false. So, involved a bit more than simple version
>> change...
>>
>> Added:
>> geronimo/branches/1.1/applications/daytrader/core/
>> geronimo/branches/1.1/applications/daytrader/ear/
>> geronimo/branches/1.1/applications/daytrader/ejb/
>> geronimo/branches/1.1/applications/daytrader/streamer/
>> geronimo/branches/1.1/applications/daytrader/web/
>> geronimo/branches/1.1/applications/daytrader/wsappclient/
>> geronimo/branches/1.1/applications/jmxdebug/
>> geronimo/branches/1.1/modules/interop/
>> Modified:
>> geronimo/branches/1.1/configs/jetty-deployer/src/plan/plan.xml
>> geronimo/branches/1.1/etc/explicit_versions.properties
>> geronimo/branches/1.1/etc/project.properties
>>
>> geronimo/branches/1.1/modules/jetty-builder/src/java/org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java
>>
>>
>> geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/geronimo-tomcat/var/catalina/conf/web.xml
>>
>>
>> Modified: geronimo/branches/1.1/configs/jetty-deployer/src/plan/plan.xml
>> URL:
>> http://svn.apache.org/viewcvs/geronimo/branches/1.1/configs/jetty-deployer/src/plan/plan.xml?rev=400233&r1=400232&r2=400233&view=diff
>>
>> ==============================================================================
>>
>> --- geronimo/branches/1.1/configs/jetty-deployer/src/plan/plan.xml
>> (original)
>> +++ geronimo/branches/1.1/configs/jetty-deployer/src/plan/plan.xml Fri
>> May 5 21:03:29 2006
>> @@ -93,8 +93,9 @@
>> <attribute
>> name="servletClass">org.apache.jasper.servlet.JspServlet</attribute>
>> <attribute name="loadOnStartup">0</attribute>
>> <attribute name="initParams">
>> - logVerbosityLevel=DEBUG
>> + development=false
>> fork=false
>> + logVerbosityLevel=DEBUG
>> xpoweredBy=false</attribute>
>> <attribute
>> name="servletMappings">*.jsp,*.jspf,*.jspx,*.xsp</attribute>
>> </gbean>
>>
>> Modified: geronimo/branches/1.1/etc/explicit_versions.properties
>> URL:
>> http://svn.apache.org/viewcvs/geronimo/branches/1.1/etc/explicit_versions.properties?rev=400233&r1=400232&r2=400233&view=diff
>>
>> ==============================================================================
>>
>> --- geronimo/branches/1.1/etc/explicit_versions.properties (original)
>> +++ geronimo/branches/1.1/etc/explicit_versions.properties Fri May 5
>> 21:03:29 2006
>> @@ -45,7 +45,7 @@
>> howl///=0.1.11
>> #security:
>> hsqldb///=1.7.2.2
>> -jasper///=5.5.12
>> +jasper///=5.5.15
>> javacc///=2.1
>> jdbm///=0.20-dev
>> jdom///=1.0
>> @@ -73,8 +73,8 @@
>> standard_taglibs///=1.1.1
>> stax/stax//jar=1.1.1-dev
>> stax/stax_api//jar=1.0
>> -tomcat_ajp///=5.5.9
>> -tomcat///=5.5.9
>> +tomcat_ajp///=5.5.15
>> +tomcat///=5.5.15
>> tomcat_servlet_examples///=5.5.15
>> tomcat_jsp_examples///=5.5.15
>> wadi///=2.0M1
>>
>> Modified: geronimo/branches/1.1/etc/project.properties
>> URL:
>> http://svn.apache.org/viewcvs/geronimo/branches/1.1/etc/project.properties?rev=400233&r1=400232&r2=400233&view=diff
>>
>> ==============================================================================
>>
>> --- geronimo/branches/1.1/etc/project.properties (original)
>> +++ geronimo/branches/1.1/etc/project.properties Fri May 5 21:03:29 2006
>> @@ -155,7 +155,7 @@
>> howl_version=0.1.11
>> #security:
>> hsqldb_version=1.7.2.2
>> -jasper_version=5.5.12
>> +jasper_version=5.5.15
>> javacc_version=2.1
>> jdbm_version=0.20-dev
>> jdom_version=1.0
>> @@ -183,8 +183,8 @@
>> standard_taglibs_version=1.1.1
>> stax_version=1.1.1-dev
>> stax_api_version=1.0
>> -tomcat_ajp_version=5.5.9
>> -tomcat_version=5.5.9
>> +tomcat_ajp_version=5.5.15
>> +tomcat_version=5.5.15
>> tomcat_servlet_examples_version=5.5.15
>> tomcat_jsp_examples_version=5.5.15
>> wadi_version=2.0M1
>>
>> Modified:
>> geronimo/branches/1.1/modules/jetty-builder/src/java/org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java
>>
>> URL:
>> http://svn.apache.org/viewcvs/geronimo/branches/1.1/modules/jetty-builder/src/java/org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java?rev=400233&r1=400232&r2=400233&view=diff
>>
>> ==============================================================================
>>
>> ---
>> geronimo/branches/1.1/modules/jetty-builder/src/java/org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java
>> (original)
>> +++
>> geronimo/branches/1.1/modules/jetty-builder/src/java/org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java
>> Fri May 5 21:03:29 2006
>> @@ -840,6 +840,7 @@
>> String servletName =
>> servletType.getServletName().getStringValue().trim();
>> AbstractName servletAbstractName =
>> earContext.getNaming().createChildName(webModuleName, servletName,
>> NameFactory.SERVLET);
>> GBeanData servletData;
>> + Map initParams = new HashMap();
>> if (servletType.isSetServletClass()) {
>> String servletClassName =
>> servletType.getServletClass().getStringValue().trim();
>> Class servletClass;
>> @@ -872,6 +873,7 @@
>> servletData.setAttribute("jspFile",
>> servletType.getJspFile().getStringValue().trim());
>> //TODO MAKE THIS CONFIGURABLE!!! Jetty uses the servlet
>> mapping set up from the default-web.xml
>> servletData.setAttribute("servletClass",
>> "org.apache.jasper.servlet.JspServlet");
>> + initParams.put("development", "false");
>> } else {
>> throw new DeploymentException("Neither servlet class nor
>> jsp file is set for " + servletName); // TODO identify web app in message
>> }
>> @@ -886,7 +888,6 @@
>> //TODO in init param setter, add classpath if jspFile is not
>> null.
>> servletData.setReferencePattern("JettyServletRegistration",
>> webModuleName);
>> servletData.setAttribute("servletName", servletName);
>> - Map initParams = new HashMap();
>> ParamValueType[] initParamArray =
>> servletType.getInitParamArray();
>> for (int j = 0; j < initParamArray.length; j++) {
>> ParamValueType paramValueType = initParamArray[j];
>>
>> Modified:
>> geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/geronimo-tomcat/var/catalina/conf/web.xml
>>
>> URL:
>> http://svn.apache.org/viewcvs/geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/geronimo-tomcat/var/catalina/conf/web.xml?rev=400233&r1=400232&r2=400233&view=diff
>>
>> ==============================================================================
>>
>> ---
>> geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/geronimo-tomcat/var/catalina/conf/web.xml
>> (original)
>> +++
>> geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/geronimo-tomcat/var/catalina/conf/web.xml
>> Fri May 5 21:03:29 2006
>> @@ -187,6 +187,10 @@
>> <servlet-name>jsp</servlet-name>
>>
>> <servlet-class>org.apache.jasper.servlet.JspServlet</servlet-class>
>> <init-param>
>> + <param-name>development</param-name>
>> + <param-value>false</param-value>
>> + </init-param>
>> + <init-param>
>> <param-name>fork</param-name>
>> <param-value>false</param-value>
>> </init-param>
>>
>>
>>
>>
>
>
>
>
Re: svn commit: r400233 - in /geronimo/branches/1.1: applications/daytrader/core/
applications/daytrader/ear/ applications/daytrader/ejb/ applications/daytrader/streamer/
applications/daytrader/web/ applications/daytrader/wsappclient/ applications/jmxdebug...
Posted by John Sisson <jr...@gmail.com>.
FYI, I removed the directories added in this commit. See
URL: http://svn.apache.org/viewcvs?rev=400273&view=rev
Log:
GERONIMO-1640 Delete directories that were accidentally added in previous checkin.
John
kevan@apache.org wrote:
> Author: kevan
> Date: Fri May 5 21:03:29 2006
> New Revision: 400233
>
> URL: http://svn.apache.org/viewcvs?rev=400233&view=rev
> Log:
> GERONIMO-1640 Upgrade Tomcat version to 5.5.15. For compliant jsp operation, upgrade required that Jasper config option 'development' be configured to false. So, involved a bit more than simple version change...
>
> Added:
> geronimo/branches/1.1/applications/daytrader/core/
> geronimo/branches/1.1/applications/daytrader/ear/
> geronimo/branches/1.1/applications/daytrader/ejb/
> geronimo/branches/1.1/applications/daytrader/streamer/
> geronimo/branches/1.1/applications/daytrader/web/
> geronimo/branches/1.1/applications/daytrader/wsappclient/
> geronimo/branches/1.1/applications/jmxdebug/
> geronimo/branches/1.1/modules/interop/
> Modified:
> geronimo/branches/1.1/configs/jetty-deployer/src/plan/plan.xml
> geronimo/branches/1.1/etc/explicit_versions.properties
> geronimo/branches/1.1/etc/project.properties
> geronimo/branches/1.1/modules/jetty-builder/src/java/org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java
> geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/geronimo-tomcat/var/catalina/conf/web.xml
>
> Modified: geronimo/branches/1.1/configs/jetty-deployer/src/plan/plan.xml
> URL: http://svn.apache.org/viewcvs/geronimo/branches/1.1/configs/jetty-deployer/src/plan/plan.xml?rev=400233&r1=400232&r2=400233&view=diff
> ==============================================================================
> --- geronimo/branches/1.1/configs/jetty-deployer/src/plan/plan.xml (original)
> +++ geronimo/branches/1.1/configs/jetty-deployer/src/plan/plan.xml Fri May 5 21:03:29 2006
> @@ -93,8 +93,9 @@
> <attribute name="servletClass">org.apache.jasper.servlet.JspServlet</attribute>
> <attribute name="loadOnStartup">0</attribute>
> <attribute name="initParams">
> - logVerbosityLevel=DEBUG
> + development=false
> fork=false
> + logVerbosityLevel=DEBUG
> xpoweredBy=false</attribute>
> <attribute name="servletMappings">*.jsp,*.jspf,*.jspx,*.xsp</attribute>
> </gbean>
>
> Modified: geronimo/branches/1.1/etc/explicit_versions.properties
> URL: http://svn.apache.org/viewcvs/geronimo/branches/1.1/etc/explicit_versions.properties?rev=400233&r1=400232&r2=400233&view=diff
> ==============================================================================
> --- geronimo/branches/1.1/etc/explicit_versions.properties (original)
> +++ geronimo/branches/1.1/etc/explicit_versions.properties Fri May 5 21:03:29 2006
> @@ -45,7 +45,7 @@
> howl///=0.1.11
> #security:
> hsqldb///=1.7.2.2
> -jasper///=5.5.12
> +jasper///=5.5.15
> javacc///=2.1
> jdbm///=0.20-dev
> jdom///=1.0
> @@ -73,8 +73,8 @@
> standard_taglibs///=1.1.1
> stax/stax//jar=1.1.1-dev
> stax/stax_api//jar=1.0
> -tomcat_ajp///=5.5.9
> -tomcat///=5.5.9
> +tomcat_ajp///=5.5.15
> +tomcat///=5.5.15
> tomcat_servlet_examples///=5.5.15
> tomcat_jsp_examples///=5.5.15
> wadi///=2.0M1
>
> Modified: geronimo/branches/1.1/etc/project.properties
> URL: http://svn.apache.org/viewcvs/geronimo/branches/1.1/etc/project.properties?rev=400233&r1=400232&r2=400233&view=diff
> ==============================================================================
> --- geronimo/branches/1.1/etc/project.properties (original)
> +++ geronimo/branches/1.1/etc/project.properties Fri May 5 21:03:29 2006
> @@ -155,7 +155,7 @@
> howl_version=0.1.11
> #security:
> hsqldb_version=1.7.2.2
> -jasper_version=5.5.12
> +jasper_version=5.5.15
> javacc_version=2.1
> jdbm_version=0.20-dev
> jdom_version=1.0
> @@ -183,8 +183,8 @@
> standard_taglibs_version=1.1.1
> stax_version=1.1.1-dev
> stax_api_version=1.0
> -tomcat_ajp_version=5.5.9
> -tomcat_version=5.5.9
> +tomcat_ajp_version=5.5.15
> +tomcat_version=5.5.15
> tomcat_servlet_examples_version=5.5.15
> tomcat_jsp_examples_version=5.5.15
> wadi_version=2.0M1
>
> Modified: geronimo/branches/1.1/modules/jetty-builder/src/java/org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java
> URL: http://svn.apache.org/viewcvs/geronimo/branches/1.1/modules/jetty-builder/src/java/org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java?rev=400233&r1=400232&r2=400233&view=diff
> ==============================================================================
> --- geronimo/branches/1.1/modules/jetty-builder/src/java/org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java (original)
> +++ geronimo/branches/1.1/modules/jetty-builder/src/java/org/apache/geronimo/jetty/deployment/JettyModuleBuilder.java Fri May 5 21:03:29 2006
> @@ -840,6 +840,7 @@
> String servletName = servletType.getServletName().getStringValue().trim();
> AbstractName servletAbstractName = earContext.getNaming().createChildName(webModuleName, servletName, NameFactory.SERVLET);
> GBeanData servletData;
> + Map initParams = new HashMap();
> if (servletType.isSetServletClass()) {
> String servletClassName = servletType.getServletClass().getStringValue().trim();
> Class servletClass;
> @@ -872,6 +873,7 @@
> servletData.setAttribute("jspFile", servletType.getJspFile().getStringValue().trim());
> //TODO MAKE THIS CONFIGURABLE!!! Jetty uses the servlet mapping set up from the default-web.xml
> servletData.setAttribute("servletClass", "org.apache.jasper.servlet.JspServlet");
> + initParams.put("development", "false");
> } else {
> throw new DeploymentException("Neither servlet class nor jsp file is set for " + servletName); // TODO identify web app in message
> }
> @@ -886,7 +888,6 @@
> //TODO in init param setter, add classpath if jspFile is not null.
> servletData.setReferencePattern("JettyServletRegistration", webModuleName);
> servletData.setAttribute("servletName", servletName);
> - Map initParams = new HashMap();
> ParamValueType[] initParamArray = servletType.getInitParamArray();
> for (int j = 0; j < initParamArray.length; j++) {
> ParamValueType paramValueType = initParamArray[j];
>
> Modified: geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/geronimo-tomcat/var/catalina/conf/web.xml
> URL: http://svn.apache.org/viewcvs/geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/geronimo-tomcat/var/catalina/conf/web.xml?rev=400233&r1=400232&r2=400233&view=diff
> ==============================================================================
> --- geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/geronimo-tomcat/var/catalina/conf/web.xml (original)
> +++ geronimo/branches/1.1/modules/tomcat/src/resources/META-INF/geronimo-tomcat/var/catalina/conf/web.xml Fri May 5 21:03:29 2006
> @@ -187,6 +187,10 @@
> <servlet-name>jsp</servlet-name>
> <servlet-class>org.apache.jasper.servlet.JspServlet</servlet-class>
> <init-param>
> + <param-name>development</param-name>
> + <param-value>false</param-value>
> + </init-param>
> + <init-param>
> <param-name>fork</param-name>
> <param-value>false</param-value>
> </init-param>
>
>
>
>