[jbosstools-dev] Re: [jbosstools-commits] JBoss Tools SVN: r8244 - in branches/jbosstools-2.1.x/as/plugins/org.jboss.ide.eclipse.as.core: META-INF and 1 other directory.
Max:
I just found this commit today (I don't browse commit logs much)...
you just take all updates that comes in too ? :)
Anyhow - it was the minimal fix for GA so ok by me for now.
The code we depend on has zero to do with tomcat.
But yes, going forward isolating it into our project will probably make more sense.
/max
Is it better to depend on a new plugin or to copy the class in
question
to a .xpl package?
I personally would have coppied the class. Snjeza decided to add a
dependency.
Thoughts? I personally find it very silly that the AS adapter would
depend on the tomcat adapter.
jbosstools-commits(a)lists.jboss.org wrote:
> Author: snjeza
> Date: 2008-05-20 17:21:12 -0400 (Tue, 20 May 2008)
> New Revision: 8244
>
> Modified:
>
branches/jbosstools-2.1.x/as/plugins/org.jboss.ide.eclipse.as.core/META-INF/MANIFEST.MF
> branches/jbosstools-2.1.x/as/plugins/org.jboss.ide.eclipse.as.core/plugin.xml
> Log:
> JBIDE-2215 Unable to add User Library as Source Lookup Path to AS/EAP server
>
> Modified:
branches/jbosstools-2.1.x/as/plugins/org.jboss.ide.eclipse.as.core/META-INF/MANIFEST.MF
> ===================================================================
> ---
branches/jbosstools-2.1.x/as/plugins/org.jboss.ide.eclipse.as.core/META-INF/MANIFEST.MF 2008-05-20
21:09:49 UTC (rev 8243)
> +++
branches/jbosstools-2.1.x/as/plugins/org.jboss.ide.eclipse.as.core/META-INF/MANIFEST.MF 2008-05-20
21:21:12 UTC (rev 8244)
> @@ -19,7 +19,8 @@
> org.eclipse.jst.common.project.facet.core,
> org.eclipse.wst.xml.core,
> org.jboss.ide.eclipse.archives.core,
> - org.apache.ant
> + org.apache.ant,
> + org.eclipse.jst.server.tomcat.core
> Eclipse-LazyStart: true
> Export-Package: org.jboss.ide.eclipse.as.core,
> org.jboss.ide.eclipse.as.core.extensions.archives,
>
> Modified:
branches/jbosstools-2.1.x/as/plugins/org.jboss.ide.eclipse.as.core/plugin.xml
> ===================================================================
> ---
branches/jbosstools-2.1.x/as/plugins/org.jboss.ide.eclipse.as.core/plugin.xml 2008-05-20
21:09:49 UTC (rev 8243)
> +++
branches/jbosstools-2.1.x/as/plugins/org.jboss.ide.eclipse.as.core/plugin.xml 2008-05-20
21:21:12 UTC (rev 8244)
> @@ -253,7 +253,7 @@
> name="JBoss Application Server Startup Configuration"
> public="true"
>
sourceLocatorId="org.eclipse.jdt.launching.sourceLocator.JavaSourceLookupDirector"
> -
sourcePathComputerId="org.jboss.ide.eclipse.as.core.serverSourcePathComputer"/>
> +
sourcePathComputerId="org.eclipse.jst.server.tomcat.core.sourcePathComputer"/>
> <launchConfigurationType
>
delegate="org.jboss.ide.eclipse.as.core.server.internal.launch.DeployableLaunchConfiguration"
>
id="org.jboss.ide.eclipse.as.core.server.stripped.launchConfiguration"
> @@ -261,7 +261,7 @@
> name="Stripped Server Launch Configuration"
> public="false"
>
sourceLocatorId="org.eclipse.jdt.launching.sourceLocator.JavaSourceLookupDirector"
> -
sourcePathComputerId="org.jboss.ide.eclipse.as.core.serverSourcePathComputer"/>
> +
sourcePathComputerId="org.eclipse.jst.server.tomcat.core.sourcePathComputer"/>
> <launchConfigurationType
>
delegate="org.jboss.ide.eclipse.as.core.server.internal.launch.TwiddleLaunchConfiguration"
>
id="org.jboss.ide.eclipse.as.core.server.twiddleConfiguration"
>
> _______________________________________________
> jbosstools-commits mailing list
> jbosstools-commits(a)lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/jbosstools-commits
>
_______________________________________________
jbosstools-commits mailing list
jbosstools-commits(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/jbosstools-commits