> That's the plugin that provides Central discovery for SOA-P and BRMS connectors. > > Not sure if the code's needed. AFAIK we just need the plugin.xml file. Afaik there should be *zero* java code in the discovery plugin, both for JBDS and SOA connectors. /max > > N > > On 05/10/2012 08:02 AM, Max Rydahl Andersen wrote: >> ..and why does it have java code in it copied from central ?! >> >> ./src/org/jboss/tools/central/configurators/DefaultJBossCentralDiscoveryConfigurator.java >> ./src/org/jboss/tools/central/configurators/IJBossCentralDiscoveryConfigurator.java >> ./src/org/jboss/tools/central/discovery/JBossCentralDiscoveryActivator.java >> >> /max

remove duplicate code from discovery plugins -------------------------------------------- Key: JBDS-2131 URL: https://issues.jboss.org/browse/JBDS-2131 Project: Developer Studio (JBoss Developer Studio) Issue Type: Bug Security Level: Public(Everyone can see) Components: central Affects Versions: 5.0.0.Beta3 Reporter: Nick Boldt Assignee: Snjezana Peco Fix For: 5.0.0.CR1 Snjezana, can you take a look at this? Discovery plugins are here, for JBT/JBDS Core: https://svn.jboss.org/repos/jbosstools/trunk/central/plugins/org.jboss.to... https://svn.jboss.org/repos/devstudio/trunk/product/plugins/com.jboss.jbd... and JBT/JBDS SOA Tooling: https://svn.jboss.org/repos/jbosstools/trunk/build/aggregate/soa-site/plu... https://svn.jboss.org/repos/devstudio/trunk/product-soa/plugins/com.jboss... On 05/10/2012 12:22 PM, Max Rydahl Andersen wrote: > >> That's the plugin that provides Central discovery for SOA-P and BRMS connectors. >> >> Not sure if the code's needed. AFAIK we just need the plugin.xml file. > > Afaik there should be *zero* java code in the discovery plugin, both for JBDS and SOA connectors. > > /max > >> >> N >> >> On 05/10/2012 08:02 AM, Max Rydahl Andersen wrote: >>> ..and why does it have java code in it copied from central ?! >>> >>> ./src/org/jboss/tools/central/configurators/DefaultJBossCentralDiscoveryConfigurator.java >>> ./src/org/jboss/tools/central/configurators/IJBossCentralDiscoveryConfigurator.java >>> ./src/org/jboss/tools/central/discovery/JBossCentralDiscoveryActivator.java >>> >>> /max

remove duplicate code from discovery plugins -------------------------------------------- Key: JBDS-2131 URL: https://issues.jboss.org/browse/JBDS-2131 Project: Developer Studio (JBoss Developer Studio) Issue Type: Bug Security Level: Public(Everyone can see) Components: central Affects Versions: 5.0.0.Beta3 Reporter: Nick Boldt Assignee: Snjezana Peco Fix For: 5.0.0.CR1 Snjezana, can you take a look at this? Discovery plugins are here, for JBT/JBDS Core: https://svn.jboss.org/repos/jbosstools/trunk/central/plugins/org.jboss.to... https://svn.jboss.org/repos/devstudio/trunk/product/plugins/com.jboss.jbd... and JBT/JBDS SOA Tooling: https://svn.jboss.org/repos/jbosstools/trunk/build/aggregate/soa-site/plu... https://svn.jboss.org/repos/devstudio/trunk/product-soa/plugins/com.jboss... On 05/10/2012 12:22 PM, Max Rydahl Andersen wrote: > >> That's the plugin that provides Central discovery for SOA-P and BRMS connectors. >> >> Not sure if the code's needed. AFAIK we just need the plugin.xml file. > > Afaik there should be *zero* java code in the discovery plugin, both for JBDS and SOA connectors. > > /max > >> >> N >> >> On 05/10/2012 08:02 AM, Max Rydahl Andersen wrote: >>> ..and why does it have java code in it copied from central ?! >>> >>> ./src/org/jboss/tools/central/configurators/DefaultJBossCentralDiscoveryConfigurator.java >>> ./src/org/jboss/tools/central/configurators/IJBossCentralDiscoveryConfigurator.java >>> ./src/org/jboss/tools/central/discovery/JBossCentralDiscoveryActivator.java >>> >>> /max