From: Snjezana Sevo-Zenzerovic <snjezana.sevozenzerovic_at_oracle.com>
Date: Fri, 6 Jul 2012 09:10:22 -0700 (PDT)

Looks fine.

Thanks,

Snjezana

----- Original Message -----
From: tim.quinn_at_oracle.com
To: dev_at_glassfish.java.net
Sent: Friday, July 6, 2012 3:49:30 PM GMT +01:00 Amsterdam / Berlin / Bern / Rome / Stockholm / Vienna
Subject: nucleus/admin/util pom.xml change review request

Hi.

Earlier I added a dependency from admin/util onto the new nucleus/security/services module, but at that time I incorrectly marked it as "provided" which worked because security/services was otherwise included in the nucleus (and other) distributions.

Now that admin/util is using the AuthorizationService from nucleus/security/services the build of the atomic distribution fails. No other module in atomic depends on nucleus/security/services so the module is not there, but the reference from admin/util cannot be resolved.

This change removes the "provided" setting from the dependency from nucleus/admin/util to nucleus/security/services.

Thanks

- Tim

Index: admin/util/pom.xml
===================================================================
--- admin/util/pom.xml (revision 55004)
+++ admin/util/pom.xml (working copy)
@@ -144,7 +144,6 @@
             <groupId>org.glassfish.main.security</groupId>
             <artifactId>security-services</artifactId>
             <version>${project.parent.version}</version>
- <scope>provided</scope>
         </dependency>
         <dependency>
             <groupId>org.glassfish.main.tests</groupId>