Skip to content
This repository has been archived by the owner on Jul 11, 2022. It is now read-only.

Bug/1146167 #5

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Bug/1146167 #5

wants to merge 5 commits into from

Conversation

genman
Copy link

@genman genman commented Oct 17, 2014

Feel free to squash the changes, but I have the commits divided into steps as necessary.

@pilhuhn
Copy link
Member

pilhuhn commented Oct 18, 2014

Did you check that this still works with as4?

Am 17. Oktober 2014 20:10:26 schrieb Elias Ross [email protected]:

Feel free to squash the changes, but I have the commits divided into steps
as necessary.
You can merge this Pull Request by running:

git pull https://github.com/genman/ems bug/1146167

Or you can view, comment on it, or merge it online at:

#5

-- Commit Summary --

  • Allow classsloader to be passed to JMXRemotingConnectionProvider
  • RHQ BZ1146167 - Remove -impl- from the .jar file
  • add maven deployment tool
  • remove ems impl separate jar - no need for it anymore
  • remove support for jdk 1.4; remove associated jar files

-- File Changes --

M README.adoc (5)
M build.xml (69)
D lib/jsr160-includes/mx4j-remote.jar (0)
D lib/jsr160-includes/mx4j.jar (0)
M 
src/ems/org/mc4j/ems/connection/support/classloader/ClassLoaderFactory.java 
(21)
R src/ems/org/mc4j/ems/impl/jmx/connection/DConnection.java (0)
R src/ems/org/mc4j/ems/impl/jmx/connection/PooledConnectionTracker.java (0)
R src/ems/org/mc4j/ems/impl/jmx/connection/bean/DAdvancedBean.java (0)
R src/ems/org/mc4j/ems/impl/jmx/connection/bean/DBeanName.java (0)
R src/ems/org/mc4j/ems/impl/jmx/connection/bean/DMBean.java (0)
R src/ems/org/mc4j/ems/impl/jmx/connection/bean/attribute/DAttribute.java (0)
R 
src/ems/org/mc4j/ems/impl/jmx/connection/bean/attribute/DUnkownAttribute.java 
(0)
R 
src/ems/org/mc4j/ems/impl/jmx/connection/bean/notification/DNotification.java 
(0)
R src/ems/org/mc4j/ems/impl/jmx/connection/bean/operation/DOperation.java (0)
R src/ems/org/mc4j/ems/impl/jmx/connection/bean/parameter/DParameter.java (0)
R 
src/ems/org/mc4j/ems/impl/jmx/connection/support/providers/AbstractConnectionProvider.java 
(0)
R 
src/ems/org/mc4j/ems/impl/jmx/connection/support/providers/InternalVMProvider.java 
(0)
R 
src/ems/org/mc4j/ems/impl/jmx/connection/support/providers/JBossConnectionProvider.java 
(0)
R 
src/ems/org/mc4j/ems/impl/jmx/connection/support/providers/JMXRemotingConnectionProvider.java 
(5)
R 
src/ems/org/mc4j/ems/impl/jmx/connection/support/providers/LocalVMProvider.java 
(0)
R 
src/ems/org/mc4j/ems/impl/jmx/connection/support/providers/Oc4jConnectionProvider.java 
(0)
R 
src/ems/org/mc4j/ems/impl/jmx/connection/support/providers/PramatiConnectionProvider.java 
(0)
R 
src/ems/org/mc4j/ems/impl/jmx/connection/support/providers/WeblogicConnectionProvider.java 
(0)
R 
src/ems/org/mc4j/ems/impl/jmx/connection/support/providers/WebsphereConnectionProvider.java 
(0)
R 
src/ems/org/mc4j/ems/impl/jmx/connection/support/providers/jaas/JBossCallbackHandler.java 
(0)
R 
src/ems/org/mc4j/ems/impl/jmx/connection/support/providers/jaas/JBossConfiguration.java 
(0)
R 
src/ems/org/mc4j/ems/impl/jmx/connection/support/providers/local/LocalVMConnector.java 
(0)
R 
src/ems/org/mc4j/ems/impl/jmx/connection/support/providers/proxy/GenericMBeanServerProxy.java 
(0)
R 
src/ems/org/mc4j/ems/impl/jmx/connection/support/providers/proxy/JMXRemotingMBeanServerProxy.java 
(0)
R 
src/ems/org/mc4j/ems/impl/jmx/connection/support/providers/proxy/JSR77ManagementMBeanServerProxy.java 
(0)
R 
src/ems/org/mc4j/ems/impl/jmx/connection/support/providers/proxy/StatsProxy.java 
(0)

-- Patch Links --

https://github.com/rhq-project/ems/pull/5.patch
https://github.com/rhq-project/ems/pull/5.diff


Reply to this email directly or view it on GitHub:
#5

@genman
Copy link
Author

genman commented Oct 18, 2014

I haven't. Only Tomcat and JBoss EAP (the one that comes with RHQ).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants