Thanks. I had thought I'd let you review first before commit, then I figure you'd review it after the commit anyway. :)
Lin On Thu, Oct 2, 2008 at 12:19 PM, David Jencks <[EMAIL PROTECTED]> wrote: > 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> >> >> > >
