You are viewing a plain text version of this content. The canonical link for it is here.
Posted to pluto-dev@portals.apache.org by "David H. DeWolf" <dd...@apache.org> on 2007/02/22 03:59:05 UTC
Re: svn commit: r510284 - in /portals/pluto/trunk/pluto-container:
pom.xml src/main/java/org/apache/pluto/core/ContainerInvocation.java src/test/java/org/apache/pluto/core/ContainerInvocationTest.java
I'm assuming this change was a mistake? It creates a cyclical
dependency. The container should NOT depend on the driver.
esm@apache.org wrote:
> -
> + <dependency>
> + <groupId>org.apache.pluto</groupId>
> + <artifactId>pluto-portal-driver</artifactId>
> + <version>${pom.version}</version>
> + <scope>test</scope>
> + </dependency>
> <!-- Other Support Libraries =========================================== -->
> <dependency>
> <groupId>commons-logging</groupId>
>
Re: svn commit: r510284 - in /portals/pluto/trunk/pluto-container:
pom.xml src/main/java/org/apache/pluto/core/ContainerInvocation.java src/test/java/org/apache/pluto/core/ContainerInvocationTest.java
Posted by "David H. DeWolf" <dd...@apache.org>.
I've reimplemented the test so it doesn't require this. I'd rather
leave the dependencies clear since the container's not really dependent.
check out what I'm about to commit and let me know what you think.
D
Elliot Metsger wrote:
> I didn't see that it causes a problem - it is scoped to test.
>
> It was to support the ContainerInvoker test. we can take it out if you
> want.
>
> David H. DeWolf wrote:
>> I'm assuming this change was a mistake? It creates a cyclical
>> dependency. The container should NOT depend on the driver.
>>
>> esm@apache.org wrote:
>>
>>> -
>>> + <dependency>
>>> + <groupId>org.apache.pluto</groupId>
>>> + <artifactId>pluto-portal-driver</artifactId>
>>> + <version>${pom.version}</version>
>>> + <scope>test</scope>
>>> + </dependency>
>>> <!-- Other Support Libraries
>>> =========================================== -->
>>> <dependency>
>>> <groupId>commons-logging</groupId>
>>>
>
Re: svn commit: r510284 - in /portals/pluto/trunk/pluto-container:
pom.xml src/main/java/org/apache/pluto/core/ContainerInvocation.java src/test/java/org/apache/pluto/core/ContainerInvocationTest.java
Posted by Elliot Metsger <em...@jhu.edu>.
I didn't see that it causes a problem - it is scoped to test.
It was to support the ContainerInvoker test. we can take it out if you
want.
David H. DeWolf wrote:
> I'm assuming this change was a mistake? It creates a cyclical
> dependency. The container should NOT depend on the driver.
>
> esm@apache.org wrote:
>
>> -
>> + <dependency>
>> + <groupId>org.apache.pluto</groupId>
>> + <artifactId>pluto-portal-driver</artifactId>
>> + <version>${pom.version}</version>
>> + <scope>test</scope>
>> + </dependency>
>> <!-- Other Support Libraries
>> =========================================== -->
>> <dependency>
>> <groupId>commons-logging</groupId>
>>