Re: svn commit: r610624 - in /geronimo/server/trunk: framework/modules/geronimo-deployment/src/main/java/org/apache/geronimo/deployment/xmlbeans/ plugins/j2ee/geronimo-naming-builder/src/main/xsd/ plu

2008-01-10 Thread Jarek Gawor
David,

The change of the
http://geronimo.apache.org/xml/ns/web/jetty/config-1.0 namespace is
now causing the test-jetty failure in trunk builds. It looks like the
old ../jetty/config-1.0 namespace is not being automatically upgraded
to .../jetty/config-1.0.1 namespace.

Here's the geronimo-web.xml file for this test:

http://svn.apache.org/viewvc/geronimo/server/trunk/testsuite/web-testsuite/test-jetty/src/main/webapp/WEB-INF/geronimo-web.xml?content-type=text%2Fplainview=co

Thanks,
Jarek

On Jan 9, 2008 8:04 PM,  [EMAIL PROTECTED] wrote:
 Author: djencks
 Date: Wed Jan  9 17:03:50 2008
 New Revision: 610624

 URL: http://svn.apache.org/viewvc?rev=610624view=rev
 Log:
 GERONIMO-3738 Expose new compactPath (or, expose security vulns) jetty 
 parameter

 Added:
 
 geronimo/server/trunk/plugins/jetty/geronimo-jetty6-builder/src/main/xsd/geronimo-jetty-2.0.2.xsd
(contents, props changed)
   - copied, changed from r610611, 
 geronimo/server/trunk/plugins/jetty/geronimo-jetty6-builder/src/main/xsd/geronimo-jetty-2.0.1.xsd
 
 geronimo/server/trunk/plugins/jetty/geronimo-jetty6-builder/src/main/xsd/geronimo-jetty-config-1.0.1.xsd
(contents, props changed)
   - copied, changed from r610611, 
 geronimo/server/trunk/plugins/jetty/geronimo-jetty6-builder/src/main/xsd/geronimo-jetty-config-1.0.xsd
 Removed:
 
 geronimo/server/trunk/plugins/jetty/geronimo-jetty6-builder/src/main/xsd/geronimo-jetty-2.0.1.xsd
 
 geronimo/server/trunk/plugins/jetty/geronimo-jetty6-builder/src/main/xsd/geronimo-jetty-config-1.0.xsd
 Modified:
 
 geronimo/server/trunk/framework/modules/geronimo-deployment/src/main/java/org/apache/geronimo/deployment/xmlbeans/XmlBeansUtil.java
 
 geronimo/server/trunk/plugins/j2ee/geronimo-naming-builder/src/main/xsd/geronimo-naming-1.2.xsd
 
 geronimo/server/trunk/plugins/j2ee/geronimo-security-builder/src/main/xsd/geronimo-security-1.2.xsd
 
 geronimo/server/trunk/plugins/jetty/geronimo-jetty6-builder/src/main/java/org/apache/geronimo/jetty6/deployment/JettyModuleBuilder.java
 
 geronimo/server/trunk/plugins/jetty/geronimo-jetty6-builder/src/main/xsdconfig/xmlconfig.xml
 
 geronimo/server/trunk/plugins/jetty/geronimo-jetty6-builder/src/test/resources/plans/plan4-converted.xml
 
 geronimo/server/trunk/plugins/jetty/geronimo-jetty6/src/main/java/org/apache/geronimo/jetty6/JettyWebAppContext.java
 
 geronimo/server/trunk/plugins/jetty/geronimo-jetty6/src/test/java/org/apache/geronimo/jetty6/AbstractWebModuleTest.java

 Modified: 
 geronimo/server/trunk/framework/modules/geronimo-deployment/src/main/java/org/apache/geronimo/deployment/xmlbeans/XmlBeansUtil.java
 URL: 
 http://svn.apache.org/viewvc/geronimo/server/trunk/framework/modules/geronimo-deployment/src/main/java/org/apache/geronimo/deployment/xmlbeans/XmlBeansUtil.java?rev=610624r1=610623r2=610624view=diff
 ==
 --- 
 geronimo/server/trunk/framework/modules/geronimo-deployment/src/main/java/org/apache/geronimo/deployment/xmlbeans/XmlBeansUtil.java
  (original)
 +++ 
 geronimo/server/trunk/framework/modules/geronimo-deployment/src/main/java/org/apache/geronimo/deployment/xmlbeans/XmlBeansUtil.java
  Wed Jan  9 17:03:50 2008
 @@ -71,10 +71,12 @@
  
 NAMESPACE_UPDATES.put(http://geronimo.apache.org/xml/ns/j2ee/web-1.2;, 
 http://geronimo.apache.org/xml/ns/j2ee/web-2.0.1;);
  
 NAMESPACE_UPDATES.put(http://geronimo.apache.org/xml/ns/j2ee/web-2.0;, 
 http://geronimo.apache.org/xml/ns/j2ee/web-2.0.1;);
  NAMESPACE_UPDATES.put(http://geronimo.apache.org/xml/ns/web/jetty;, 
 http://geronimo.apache.org/xml/ns/j2ee/web/jetty-2.0.1;);
 -
 NAMESPACE_UPDATES.put(http://geronimo.apache.org/xml/ns/j2ee/web/jetty-1.1;, 
 http://geronimo.apache.org/xml/ns/j2ee/web/jetty-2.0.1;);
 -
 NAMESPACE_UPDATES.put(http://geronimo.apache.org/xml/ns/j2ee/web/jetty-1.2;, 
 http://geronimo.apache.org/xml/ns/j2ee/web/jetty-2.0.1;);
 -
 NAMESPACE_UPDATES.put(http://geronimo.apache.org/xml/ns/j2ee/web/jetty-2.0;, 
 http://geronimo.apache.org/xml/ns/j2ee/web/jetty-2.0.1;);
 -
 NAMESPACE_UPDATES.put(http://geronimo.apache.org/xml/ns/web/jetty/config;, 
 http://geronimo.apache.org/xml/ns/j2ee/web/jetty/config-1.0;);
 +
 NAMESPACE_UPDATES.put(http://geronimo.apache.org/xml/ns/j2ee/web/jetty-1.1;, 
 http://geronimo.apache.org/xml/ns/j2ee/web/jetty-2.0.2;);
 +
 NAMESPACE_UPDATES.put(http://geronimo.apache.org/xml/ns/j2ee/web/jetty-1.2;, 
 http://geronimo.apache.org/xml/ns/j2ee/web/jetty-2.0.2;);
 +
 NAMESPACE_UPDATES.put(http://geronimo.apache.org/xml/ns/j2ee/web/jetty-2.0;, 
 http://geronimo.apache.org/xml/ns/j2ee/web/jetty-2.0.2;);
 +
 NAMESPACE_UPDATES.put(http://geronimo.apache.org/xml/ns/j2ee/web/jetty-2.0.1;,
  http://geronimo.apache.org/xml/ns/j2ee/web/jetty-2.0.2;);
 +
 NAMESPACE_UPDATES.put(http://geronimo.apache.org/xml/ns/web/jetty/config;, 
 

Re: svn commit: r610624 - in /geronimo/server/trunk: framework/modules/geronimo-deployment/src/main/java/org/apache/geronimo/deployment/xmlbeans/ plugins/j2ee/geronimo-naming-builder/src/main/xsd/ plu

2008-01-10 Thread David Jencks
Thanks for telling me about this.  I ran the testsuite before  
committing and assumed that since it reported success at the end  
there were no errors.  This is wrong, it reports success no matter  
what happens.


Is there some way the testsuite could be modified to fail on errors  
or at least clearly report all the errors at the end of a full run?


Anyway this should now be fixed.

thanks
david jencks

On Jan 10, 2008, at 2:27 PM, Jarek Gawor wrote:


David,

The change of the
http://geronimo.apache.org/xml/ns/web/jetty/config-1.0 namespace is
now causing the test-jetty failure in trunk builds. It looks like the
old ../jetty/config-1.0 namespace is not being automatically upgraded
to .../jetty/config-1.0.1 namespace.

Here's the geronimo-web.xml file for this test:

http://svn.apache.org/viewvc/geronimo/server/trunk/testsuite/web- 
testsuite/test-jetty/src/main/webapp/WEB-INF/geronimo-web.xml? 
content-type=text%2Fplainview=co


Thanks,
Jarek

On Jan 9, 2008 8:04 PM,  [EMAIL PROTECTED] wrote:

Author: djencks
Date: Wed Jan  9 17:03:50 2008
New Revision: 610624