dev@glassfish.java.net

Re: deployment/common pom.xml change review

From: Sahoo <sanjeeb.sahoo_at_oracle.com>
Date: Thu, 10 Mar 2011 00:05:58 +0530

Looks alright to me.

On Wednesday 09 March 2011 11:46 PM, Tim Quinn wrote:
> Hi.
>
> I'd like to add a dependency with scope "test" on the tests/utils
> module to deployment/common. Unit tests of some bug fix code needs to
> use the habitat for injection and tests/utils provides that.
>
> Diffs below, entire file attached.
>
> Thanks.
>
> - Tim
>
> Index: deployment/common/pom.xml
> ===================================================================
> --- deployment/common/pom.xml (revision 45435)
> +++ deployment/common/pom.xml (working copy)
> @@ -134,5 +134,11 @@
> <artifactId>asm-all-repackaged</artifactId>
> <version>${hk2.version}</version>
> </dependency>
> + <dependency>
> + <groupId>org.glassfish.tests</groupId>
> + <artifactId>utils</artifactId>
> + <version>${project.version}</version>
> + <scope>test</scope>
> + </dependency>
> </dependencies>
> </project>
>
>