dev@glassfish.java.net

Re: please approve pom.xml changes (removal of amx-api and amx-impl modules)

From: Lloyd Chambers <Lloyd.Chambers_at_Sun.COM>
Date: Wed, 01 Jul 2009 19:30:58 -0700

I've hacked out the dependency as follows. It should not have been
there, I don't know why it was there. We have multiple copies of
statistics classes around, that may have contributed.

Sreeni should know what's going on.

Lloyd


llcMP:web llc$ svn diff
Index: admin/src/main/java/org/glassfish/web/admin/monitor/statistics/
WebModuleVirtualServerStatsImpl.java
===================================================================
--- admin/src/main/java/org/glassfish/web/admin/monitor/statistics/
WebModuleVirtualServerStatsImpl.java (revision 28413)
+++ admin/src/main/java/org/glassfish/web/admin/monitor/statistics/
WebModuleVirtualServerStatsImpl.java (working copy)
@@ -45,8 +45,8 @@
  import org.glassfish.j2ee.statistics.CountStatistic;
  import org.glassfish.j2ee.statistics.TimeStatistic;
  import org.glassfish.j2ee.statistics.Stats;
-import com.sun.appserv.management.j2ee.statistics.StringStatistic;
-import
com
.sun.appserv.management.monitor.statistics.WebModuleVirtualServerStats;
+import org.glassfish.admin.amx.j2ee.statistics.StringStatistic;
+//import
com
.sun.appserv.management.monitor.statistics.WebModuleVirtualServerStats;
  import org.glassfish.admin.monitor.cli.MonitorContract;
  import org.glassfish.flashlight.datatree.TreeNode;
  import org.glassfish.flashlight.MonitoringRuntimeDataRegistry;
@@ -77,7 +77,7 @@
  @Service
  @Scoped(PerLookup.class)
  public class WebModuleVirtualServerStatsImpl
- implements WebModuleVirtualServerStats, MonitorContract {
+ implements /*WebModuleVirtualServerStats,*/ MonitorContract {

      // app name otherwise web
      @Param (optional=true)
Index: admin/src/main/java/org/glassfish/web/admin/monitor/statistics/
HTTPListenerStatsImpl.java
===================================================================
--- admin/src/main/java/org/glassfish/web/admin/monitor/statistics/
HTTPListenerStatsImpl.java (revision 28413)
+++ admin/src/main/java/org/glassfish/web/admin/monitor/statistics/
HTTPListenerStatsImpl.java (working copy)
@@ -45,7 +45,7 @@
  import org.glassfish.j2ee.statistics.CountStatistic;
  import org.glassfish.j2ee.statistics.TimeStatistic;
  import org.glassfish.j2ee.statistics.Stats;
-import com.sun.appserv.management.monitor.statistics.HTTPListenerStats;
+import com.sun.enterprise.admin.monitor.stats.HTTPListenerStats;
  import org.glassfish.admin.monitor.cli.MonitorContract;
  import org.glassfish.flashlight.datatree.TreeNode;
  import org.glassfish.flashlight.MonitoringRuntimeDataRegistry;
Index: admin/src/main/java/org/glassfish/web/admin/monitor/statistics/
AltServletStatsImpl.java
===================================================================
--- admin/src/main/java/org/glassfish/web/admin/monitor/statistics/
AltServletStatsImpl.java (revision 28413)
+++ admin/src/main/java/org/glassfish/web/admin/monitor/statistics/
AltServletStatsImpl.java (working copy)
@@ -49,7 +49,7 @@

  import org.glassfish.admin.monitor.cli.MonitorContract;

-import com.sun.appserv.management.monitor.statistics.AltServletStats;
+//import com.sun.appserv.management.monitor.statistics.AltServletStats;

  /**
          Defines additional Application Server specific statistics
@@ -60,7 +60,7 @@

  @Service
  @Scoped(PerLookup.class)
-public class AltServletStatsImpl implements AltServletStats,
MonitorContract {
+public class AltServletStatsImpl implements /*AltServletStats,*/
MonitorContract {

      private final String name = "servlet";

Index: admin/pom.xml
===================================================================
--- admin/pom.xml (revision 28413)
+++ admin/pom.xml (working copy)
@@ -141,11 +141,6 @@
              <version>${project.version}</version>
          </dependency>
          <dependency>
- <groupId>org.glassfish.common</groupId>
- <artifactId>amx-api</artifactId>
- <version>${project.version}</version>
- </dependency>
- <dependency>
              <groupId>org.glassfish.provider</groupId>
              <artifactId>gfprobe-provider-client</artifactId>
          </dependency>
@@ -153,6 +148,11 @@
              <groupId>org.glassfish.gmbal</groupId>
              <artifactId>gmbal</artifactId>
          </dependency>
+ <dependency>
+ <groupId>org.glassfish.common</groupId>
+ <artifactId>amx-j2ee</artifactId>
+ <version>${project.parent.version}</version>
+ </dependency>
      </dependencies>
  </project>


Lloyd

On Jul 1, 2009, at 5:09 PM, Jane Young wrote:

> Hi Lloyd,
>
> There's a dependency on amx-api in web/admin/pom.xml. You need to
> remove that or change it to amx-core.
> The weekly build is breaking because it's trying to resolve the
> artifact amx-api, 3.0-SNAPSHOT.
> See: http://gf-hudson.sfbay/hudson/job/gf-trunk-build-weekly/122/console
>
> Please fix this ASAP.
>
> Thanks,
> Jane
>
>
> Lloyd Chambers wrote:
>>
>> I am removing the defunct amx-api and amx-impl modules in their
>> entirety.
>>
>> The following pom.xml changes are required:
>>
>> llcMP:code llc$ svn diff tests/quicklook/adminconsole/pom.xml tests/
>> amx/pom.xml distributions packager/glassfish-amx/pom.xml admin/cli/
>> pom.xml common/pom.xml common/mbeanserver/pom.xml
>> Index: tests/quicklook/adminconsole/pom.xml
>> ===================================================================
>> --- tests/quicklook/adminconsole/pom.xml (revision 28402)
>> +++ tests/quicklook/adminconsole/pom.xml (working copy)
>> @@ -117,8 +117,13 @@
>> </dependency>
>> <dependency>
>> <groupId>org.glassfish.common</groupId>
>> - <artifactId>amx-api</artifactId>
>> + <artifactId>amx-core</artifactId>
>> <version>3.0-SNAPSHOT</version>
>> </dependency>
>> + <dependency>
>> + <groupId>org.glassfish.common</groupId>
>> + <artifactId>amx-config</artifactId>
>> + <version>3.0-SNAPSHOT</version>
>> + </dependency>
>> </dependencies>
>> </project>
>> Index: tests/amx/pom.xml
>> ===================================================================
>> --- tests/amx/pom.xml (revision 28402)
>> +++ tests/amx/pom.xml (working copy)
>> @@ -122,17 +122,24 @@
>>
>> <dependency>
>> <groupId>org.glassfish.common</groupId>
>> - <artifactId>amx-api</artifactId>
>> + <artifactId>amx-core</artifactId>
>> <version>${project.version}</version>
>> <scope>compile</scope>
>> </dependency>
>>
>> <dependency>
>> <groupId>org.glassfish.common</groupId>
>> - <artifactId>amx-impl</artifactId>
>> + <artifactId>amx-config</artifactId>
>> <version>${project.version}</version>
>> <scope>provided</scope>
>> </dependency>
>> +
>> + <dependency>
>> + <groupId>org.glassfish.common</groupId>
>> + <artifactId>amx-j2ee</artifactId>
>> + <version>${project.version}</version>
>> + <scope>provided</scope>
>> + </dependency>
>>
>> <!--
>> <dependency>
>> Index: distributions/distributions.xml
>> ===================================================================
>> --- distributions/distributions.xml (revision 28402)
>> +++ distributions/distributions.xml (working copy)
>> @@ -303,19 +303,16 @@
>> <resolveArtifact artifactId="server-mgmt"
>> groupId="org.glassfish.admin" property="server-mgmt.jar"/>
>> <resolveArtifact artifactId="admin-util"
>> groupId="org.glassfish.admin" property="admin-util.jar"/>
>> <resolveArtifact artifactId="security"
>> groupId="org.glassfish.security" property="security.jar"/>
>> - <resolveArtifact artifactId="amx-api"
>> groupId="org.glassfish.common" property="amx-api.jar"/>
>>
>> <echo message="cli-optional jar: ${cli-optional.jar}"/>
>> <echo message="server-mgmt jar: ${server-mgmt.jar}"/>
>> <echo message="admin-util jar: ${admin-util.jar}"/>
>> <echo message="security jar: ${security.jar}"/>
>> - <echo message="amx-api jar: ${amx-api.jar}"/>
>>
>> <copy file="${cli-optional.jar}" tofile="${modules.dir}/cli-
>> optional.jar"/>
>> <copy file="${server-mgmt.jar}" tofile="${modules.dir}/server-
>> mgmt.jar"/>
>> <copy file="${admin-util.jar}" tofile="${modules.dir}/admin-
>> util.jar"/>
>> <copy file="${security.jar}" tofile="${modules.dir}/
>> security.jar"/>
>> - <copy file="${amx-api.jar}" tofile="${modules.dir}/amx-
>> api.jar"/>
>> </target>
>>
>> <!-- delete artifacts required to create default domain; these are
>> not bundled as part of nucleus -->
>> @@ -327,7 +324,6 @@
>> <include name="admin-util.jar"/>
>> <include name="security.jar"/>
>> <include name="backup.jar"/>
>> - <include name="amx-api.jar"/>
>> </fileset>
>> </delete>
>> </target>
>> Index: distributions/nucleus/pom.xml
>> ===================================================================
>> --- distributions/nucleus/pom.xml (revision 28402)
>> +++ distributions/nucleus/pom.xml (working copy)
>> @@ -200,12 +200,6 @@
>> <version>${project.version}</version>
>> <type>jar</type>
>> </dependency>
>> - <dependency>
>> - <groupId>org.glassfish.common</groupId>
>> - <artifactId>amx-api</artifactId>
>> - <version>${project.version}</version>
>> - <type>jar</type>
>> - </dependency>
>> <!-- additional servicetag registry dependency -->
>> <dependency>
>> <groupId>org.glassfish.registration</groupId>
>> Index: packager/glassfish-amx/pom.xml
>> ===================================================================
>> --- packager/glassfish-amx/pom.xml (revision 28402)
>> +++ packager/glassfish-amx/pom.xml (working copy)
>> @@ -72,11 +72,6 @@
>> -->
>> <dependency>
>> <groupId>org.glassfish.common</groupId>
>> - <artifactId>amx-impl</artifactId>
>> - <version>${project.version}</version>
>> - </dependency>
>> - <dependency>
>> - <groupId>org.glassfish.common</groupId>
>> <artifactId>amx-core-impl</artifactId>
>> <version>${project.version}</version>
>> </dependency>
>> Index: admin/cli/pom.xml
>> ===================================================================
>> --- admin/cli/pom.xml (revision 28402)
>> +++ admin/cli/pom.xml (working copy)
>> @@ -135,7 +135,7 @@
>>
>> <mainClass>com.sun.enterprise.admin.cli.AsadminMain</mainClass>
>> </manifest>
>> <manifestEntries>
>> - <Class-Path>cli-framework.jar cli-optional.jar glassfish.jar
>> server-mgmt.jar common-util.jar admin-util.jar launcher.jar
>> backup.jar amx-api.jar stax-osgi.jar admin-cli-l10n.jar
>> security.jar glassfish-api.jar</Class-Path>
>> + <Class-Path>cli-framework.jar cli-optional.jar glassfish.jar
>> server-mgmt.jar common-util.jar admin-util.jar launcher.jar
>> backup.jar stax-osgi.jar admin-cli-l10n.jar security.jar glassfish-
>> api.jar</Class-Path>
>> <!-- 5776: we do not need backup jar
>> entry, but I retain it, Kedar -->
>> </manifestEntries>
>> </archive>
>> Index: common/pom.xml
>> ===================================================================
>> --- common/pom.xml (revision 28402)
>> +++ common/pom.xml (working copy)
>> @@ -62,8 +62,6 @@
>> <module>amx-ext-impl</module>
>> <module>mejb</module>
>>
>> - <module>amx-api</module>
>> - <module>amx-impl</module>
>> <module>glassfish-ee-api</module>
>> <module>internal-api</module>
>> <module>annotation-framework</module>
>> Index: common/mbeanserver/pom.xml
>> ===================================================================
>> --- common/mbeanserver/pom.xml (revision 28402)
>> +++ common/mbeanserver/pom.xml (working copy)
>> @@ -84,12 +84,5 @@
>> <artifactId>jmxremote_optional-repackaged</artifactId>
>> <version>${project.version}</version>
>> </dependency>
>> - <!--
>> - <dependency>
>> - <groupId>org.jvnet</groupId>
>> - <artifactId>tiger-types</artifactId>
>> - <version>RELEASE</version>
>> - </dependency>
>> - -->
>> </dependencies>
>> </project>
>> llcMP:code llc$
>>
>>
>>
>> Lloyd Chambers
>> lloyd.chambers_at_sun.com
>> GlassFish Team
>>
>>
>>
>

Lloyd Chambers
lloyd.chambers_at_sun.com
GlassFish Team