You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geronimo.apache.org by David Jencks <da...@yahoo.com> on 2008/10/02 18:19:25 UTC
Re: svn commit: r701153 - /geronimo/server/trunk/plugins/client/geronimo-client-builder/pom.xml
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, linsun@apache.org 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>
>
>
Re: svn commit: r701153 - /geronimo/server/trunk/plugins/client/geronimo-client-builder/pom.xml
Posted by Lin Sun <li...@gmail.com>.
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 <da...@yahoo.com> 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, linsun@apache.org 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>
>>
>>
>
>