You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by ho...@apache.org on 2006/12/14 06:45:40 UTC

svn commit: r486974 - /geronimo/server/branches/2.0-M1/modules/geronimo-jetty6/pom.xml

Author: hogstrom
Date: Wed Dec 13 21:45:39 2006
New Revision: 486974

URL: http://svn.apache.org/viewvc?view=rev&rev=486974
Log:
Fixed POM to address test failures.  Here is the net of the issue:

I had to explicitly add testsupport as a dependency and then exclude selenium-server 
so that the method not found error could be addressed.

          ** Major hack IMHO...testsupport brings in selenium as a transitive dependency.
          ** Selenium apparently has Jetty 5 classes buried in its selenium-server jar
          ** and as such was causing the test org.apache.geronimo.jetty6.GBeanInfoTest to
          ** fail with a method not found error on org.mortbay.jetty.util.IO which has 
          ** changed in Jetty 6.  When Selenium fixes this little problem the following 
          ** dependency, and thus the exclusion, can be removed.
 

Modified:
    geronimo/server/branches/2.0-M1/modules/geronimo-jetty6/pom.xml

Modified: geronimo/server/branches/2.0-M1/modules/geronimo-jetty6/pom.xml
URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.0-M1/modules/geronimo-jetty6/pom.xml?view=diff&rev=486974&r1=486973&r2=486974
==============================================================================
--- geronimo/server/branches/2.0-M1/modules/geronimo-jetty6/pom.xml (original)
+++ geronimo/server/branches/2.0-M1/modules/geronimo-jetty6/pom.xml Wed Dec 13 21:45:39 2006
@@ -33,6 +33,24 @@
     <name>Geronimo :: Jetty 6</name>
     
     <dependencies>
+        <!--
+          ** Major hack IMHO...testsupport brings in selenium as a transitive dependency.
+          ** Selenium apparently has Jetty 5 classes buried in its selenium-server jar
+          ** and as such was causing the test org.apache.geronimo.jetty6.GBeanInfoTest to
+          ** fail with a method not found error on org.mortbay.jetty.util.IO which has 
+          ** changed in Jetty 6.  When Selenium fixes this little problem the following 
+          ** dependency, and thus the exclusion, can be removed.
+          -->
+        <dependency>
+            <groupId>org.apache.geronimo.testsupport</groupId>
+            <artifactId>testsupport-common</artifactId>
+            <exclusions>
+                <exclusion>
+                    <groupId>org.openqa.selenium.server</groupId>
+                    <artifactId>selenium-server</artifactId>
+                </exclusion>
+            </exclusions>
+        </dependency>
     
         <dependency>
             <groupId>org.apache.geronimo.modules</groupId>
@@ -105,8 +123,10 @@
             <artifactId>commons-primitives</artifactId>
         </dependency>
         
+
     </dependencies>
     
+    
     <build>
         <plugins>
             <plugin>
@@ -118,6 +138,7 @@
                             <value>${basedir}/src/test/resources/data/login.config</value>
                         </property>
                     </systemProperties>
+                    <childDelegation>true</childDelegation>
                 </configuration>
             </plugin>
         </plugins>