You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@servicemix.apache.org by Guillaume Nodet <gn...@gmail.com> on 2007/10/22 10:22:04 UTC

Re: svn commit: r587026 - /incubator/servicemix/trunk/deployables/serviceengines/servicemix-lwcontainer/pom.xml

Any particular reason for this change ? I don't see why the
lwcontainer would depend on servicemix-quartz (though I don't really
see why it would depend on quartz either....)

On 10/22/07, ffang@apache.org <ff...@apache.org> wrote:
> Author: ffang
> Date: Mon Oct 22 01:12:50 2007
> New Revision: 587026
>
> URL: http://svn.apache.org/viewvc?rev=587026&view=rev
> Log:
> correct servicemix-lwcontainer dependency
>
> Modified:
>     incubator/servicemix/trunk/deployables/serviceengines/servicemix-lwcontainer/pom.xml
>
> Modified: incubator/servicemix/trunk/deployables/serviceengines/servicemix-lwcontainer/pom.xml
> URL: http://svn.apache.org/viewvc/incubator/servicemix/trunk/deployables/serviceengines/servicemix-lwcontainer/pom.xml?rev=587026&r1=587025&r2=587026&view=diff
> ==============================================================================
> --- incubator/servicemix/trunk/deployables/serviceengines/servicemix-lwcontainer/pom.xml (original)
> +++ incubator/servicemix/trunk/deployables/serviceengines/servicemix-lwcontainer/pom.xml Mon Oct 22 01:12:50 2007
> @@ -49,8 +49,8 @@
>        <scope>provided</scope>
>      </dependency>
>      <dependency>
> -      <groupId>quartz</groupId>
> -      <artifactId>quartz</artifactId>
> +      <groupId>org.apache.servicemix</groupId>
> +      <artifactId>servicemix-quartz</artifactId>
>        <scope>test</scope>
>      </dependency>
>    </dependencies>
>
>
>


-- 
Cheers,
Guillaume Nodet
------------------------
Blog: http://gnodet.blogspot.com/

Re: svn commit: r587026 - /incubator/servicemix/trunk/deployables/serviceengines/servicemix-lwcontainer/pom.xml

Posted by Guillaume Nodet <gn...@gmail.com>.
Actually, it seems you're right and there is a dependency to
servicemix-quartz at test level (I missed the scope in your commit).
So feel free to keep the change if it works better (and sorry about
that)

On 10/22/07, Freeman Fang <fr...@iona.com> wrote:
> Hi Guillaume,
>
> This is my fault, I already revert it.
>
> The reason I change it is I get test failure for servicemix-lwcontainer,
> testEndpoints() failed when try to deploy servicemix-quartz component,
> I thought the reason is that miss the dependency to servicemix-quartz,
> but now I realize the reason is that this test is based on the
> assumption that my local maven repository should be ~/.m2/repository,
> but I point it to another folder in my settings.xml, I change it back so
> the test pass.
>
> Best Regards
>
> Freeman
> Guillaume Nodet wrote:
> > Any particular reason for this change ? I don't see why the
> > lwcontainer would depend on servicemix-quartz (though I don't really
> > see why it would depend on quartz either....)
> >
> > On 10/22/07, ffang@apache.org <ff...@apache.org> wrote:
> >
> >> Author: ffang
> >> Date: Mon Oct 22 01:12:50 2007
> >> New Revision: 587026
> >>
> >> URL: http://svn.apache.org/viewvc?rev=587026&view=rev
> >> Log:
> >> correct servicemix-lwcontainer dependency
> >>
> >> Modified:
> >>     incubator/servicemix/trunk/deployables/serviceengines/servicemix-lwcontainer/pom.xml
> >>
> >> Modified: incubator/servicemix/trunk/deployables/serviceengines/servicemix-lwcontainer/pom.xml
> >> URL: http://svn.apache.org/viewvc/incubator/servicemix/trunk/deployables/serviceengines/servicemix-lwcontainer/pom.xml?rev=587026&r1=587025&r2=587026&view=diff
> >> ==============================================================================
> >> --- incubator/servicemix/trunk/deployables/serviceengines/servicemix-lwcontainer/pom.xml (original)
> >> +++ incubator/servicemix/trunk/deployables/serviceengines/servicemix-lwcontainer/pom.xml Mon Oct 22 01:12:50 2007
> >> @@ -49,8 +49,8 @@
> >>        <scope>provided</scope>
> >>      </dependency>
> >>      <dependency>
> >> -      <groupId>quartz</groupId>
> >> -      <artifactId>quartz</artifactId>
> >> +      <groupId>org.apache.servicemix</groupId>
> >> +      <artifactId>servicemix-quartz</artifactId>
> >>        <scope>test</scope>
> >>      </dependency>
> >>    </dependencies>
> >>
> >>
> >>
> >>
> >
> >
> >
>


-- 
Cheers,
Guillaume Nodet
------------------------
Blog: http://gnodet.blogspot.com/

Re: svn commit: r587026 - /incubator/servicemix/trunk/deployables/serviceengines/servicemix-lwcontainer/pom.xml

Posted by Freeman Fang <fr...@iona.com>.
Hi Guillaume,

This is my fault, I already revert it.

The reason I change it is I get test failure for servicemix-lwcontainer,
testEndpoints() failed when try to deploy servicemix-quartz component,
I thought the reason is that miss the dependency to servicemix-quartz, 
but now I realize the reason is that this test is based on the 
assumption that my local maven repository should be ~/.m2/repository, 
but I point it to another folder in my settings.xml, I change it back so 
the test pass.

Best Regards

Freeman
Guillaume Nodet wrote:
> Any particular reason for this change ? I don't see why the
> lwcontainer would depend on servicemix-quartz (though I don't really
> see why it would depend on quartz either....)
>
> On 10/22/07, ffang@apache.org <ff...@apache.org> wrote:
>   
>> Author: ffang
>> Date: Mon Oct 22 01:12:50 2007
>> New Revision: 587026
>>
>> URL: http://svn.apache.org/viewvc?rev=587026&view=rev
>> Log:
>> correct servicemix-lwcontainer dependency
>>
>> Modified:
>>     incubator/servicemix/trunk/deployables/serviceengines/servicemix-lwcontainer/pom.xml
>>
>> Modified: incubator/servicemix/trunk/deployables/serviceengines/servicemix-lwcontainer/pom.xml
>> URL: http://svn.apache.org/viewvc/incubator/servicemix/trunk/deployables/serviceengines/servicemix-lwcontainer/pom.xml?rev=587026&r1=587025&r2=587026&view=diff
>> ==============================================================================
>> --- incubator/servicemix/trunk/deployables/serviceengines/servicemix-lwcontainer/pom.xml (original)
>> +++ incubator/servicemix/trunk/deployables/serviceengines/servicemix-lwcontainer/pom.xml Mon Oct 22 01:12:50 2007
>> @@ -49,8 +49,8 @@
>>        <scope>provided</scope>
>>      </dependency>
>>      <dependency>
>> -      <groupId>quartz</groupId>
>> -      <artifactId>quartz</artifactId>
>> +      <groupId>org.apache.servicemix</groupId>
>> +      <artifactId>servicemix-quartz</artifactId>
>>        <scope>test</scope>
>>      </dependency>
>>    </dependencies>
>>
>>
>>
>>     
>
>
>