I just wanted to confirm that this looks to me like the correct fix.

Many thanks for tracking this down!
david jencks

On Oct 2, 2008, at 8:24 AM, [EMAIL PROTECTED] wrote:

Author: linsun
Date: Thu Oct  2 08:24:53 2008
New Revision: 701153

URL: http://svn.apache.org/viewvc?rev=701153&view=rev
Log:
comment out the provided scope to resolve the daytrader build error NPE

Modified:
geronimo/server/trunk/plugins/client/geronimo-client-builder/ pom.xml

Modified: geronimo/server/trunk/plugins/client/geronimo-client- builder/pom.xml
URL: 
http://svn.apache.org/viewvc/geronimo/server/trunk/plugins/client/geronimo-client-builder/pom.xml?rev=701153&r1=701152&r2=701153&view=diff
= = = = = = = = ====================================================================== --- geronimo/server/trunk/plugins/client/geronimo-client-builder/ pom.xml (original) +++ geronimo/server/trunk/plugins/client/geronimo-client-builder/ pom.xml Thu Oct 2 08:24:53 2008
@@ -43,7 +43,7 @@
            <groupId>org.apache.geronimo.modules</groupId>
            <artifactId>geronimo-connector-builder</artifactId>
            <version>${version}</version>
-            <scope>provided</scope>
+            <!--<scope>provided</scope>-->
        </dependency>

        <dependency>



Reply via email to