[jbosstools-issues] [JBoss JIRA] (JBDS-3071) JBDS Installer shouldn't contain source features/plugins

[ https://issues.jboss.org/browse/JBDS-3071?page=com.atlassian.jira.plugin.... ] Nick Boldt edited comment on JBDS-3071 at 6/30/14 1:11 PM: ----------------------------------------------------------- Actually, Central/EA sites do NOT contain jst.jsdt, but EA does include: * Arquillian (and Sapphire) * AngularJS (and Tern) * VJET (and Jetty) jst.jsdt is currently only in JBT afaict. So this too is an improvement. New PR to remove things that were in the JBDS update site but which are not included in the installer: https://github.com/jbdevstudio/jbdevstudio-product/pull/201 Removed items include: * org.jboss.tools.vpe.cordovasim.feature (and source) * org.jboss.tools.maven.jbosspackaging.feature (and source) * org.jboss.tools.common.mylyn.feature (and source) Problem w/ removing these is that the JBDS Central plugin.xml requirs all three of them in these connectors: * JBoss Hybrid Mobile Tools + CordovaSim * JBoss Tools Maven Packaging Configurator * Mylyn support for reporting JBoss JIRAs and Red Hat Bugzillas So if we remove them, then those connectors can't be installed in JBDS. Possible workarounds: a) do nothing and accept that for offline installs, you need the update site (not the installer) + central zip; also accept that the installer != update site b) put these 3 features in the installer so that the installer contents == the update site contents (except for sources) c) move these 3 features into the JBDS TP; this is bad because it creates an ugly dependency chain: JBDS -> JBDS TP -> JBT -> JBT TP... every time JBT rebuilds, we need to update the version of org.jboss.tools.* in the JBDS TP d) move these 3 features into the Central TP; this is bad in that as in (c) we would have to keep updating the Central TP each time there's a new JBT release... but we already have this problem because of Arquillian being included in the EA TP. So... I'd vote for (b) [easiest] or (d) [cleanest but requires more maintenance when we push out milestones]. was (Author: nickboldt): Actually, Central/EA sites do NOT contain jst.jsdt, but EA does include: * Arquillian (and Sapphire) * AngularJS (and Tern) * VJET (and Jetty) jst.jsdt is currently only in JBT afaict. So this too is an improvement. New PR to remove things that were in the JBDS update site but which are not included in the installer: https://github.com/jbdevstudio/jbdevstudio-product/pull/201 Removed items include: * org.jboss.tools.vpe.cordovasim.feature (and source) * org.jboss.tools.maven.jbosspackaging.feature (and source) * org.jboss.tools.common.mylyn.feature (and source) -- This message was sent by Atlassian JIRA (v6.2.6#6264)