Terminals: Bug 435150 - Support CDT 8.3 minimal dependency via p2.inf

This change makes sure that the Terminal can depend on a minimal
subset of CDT functionality, imported by import-package in MANIFEST.MF
plus fragment dependencies in p2.inf.

The org.eclipse.cdt.core.native feature is pulled in if it's available,
but we can also work on top of CDT 8.3 which didn't have the "native"
feature yet.

Change-Id: Ibd2dbe1b971f52195aa28fdcdb7e1048ad1ed3cf
Signed-off-by: Martin Oberhuber <martin.oberhuber@windriver.com>

3 files changed, 30 insertions, 3 deletions

diff --git a/target_explorer/features/org.eclipse.tcf.te.terminals.feature/build.properties b/target_explorer/features/org.eclipse.tcf.te.terminals.feature/build.properties
index d9e56c50b..c8eb60109 100644
--- a/target_explorer/features/org.eclipse.tcf.te.terminals.feature/build.properties
+++ b/target_explorer/features/org.eclipse.tcf.te.terminals.feature/build.properties
@@ -8,5 +8,6 @@
 # Wind River Systems - initial API and implementation
 ###############################################################################
 bin.includes = feature.xml,\
-               feature.properties
+               feature.properties,\
+               p2.inf
 src.includes = feature.properties
diff --git a/target_explorer/features/org.eclipse.tcf.te.terminals.feature/feature.xml b/target_explorer/features/org.eclipse.tcf.te.terminals.feature/feature.xml
index 7ae137c1f..a98f9ba68 100644
--- a/target_explorer/features/org.eclipse.tcf.te.terminals.feature/feature.xml
+++ b/target_explorer/features/org.eclipse.tcf.te.terminals.feature/feature.xml
@@ -24,6 +24,12 @@
       <update label="%updateSiteName" url="http://download.eclipse.org/releases/kepler"/>
    </url>
 
+   <!-- For PTY.Modes.TERMINAL , we need minimum o.e.cdt.core_5.6 which is in CDT 8.3      -->
+   <!-- But we express this in bundle dependencies only in order to get a smaller install  -->
+   <!-- The side-effect of depending on feature o.e.cdt.native is that it requires CDT 8.4 -->
+   <!-- This is OK for this "stand-alone" feature. Product builders who want CDT 8.3 can   -->
+   <!-- Still get it by not using this feature but direct cdt.platform / bundle dependency -->
+   <!-- See https://bugs.eclipse.org/bugs/show_bug.cgi?id=435150                           -->
    <requires>
       <import plugin="org.eclipse.core.expressions" version="3.4.400" match="greaterOrEqual"/>
       <import plugin="org.eclipse.core.runtime" version="3.8.0" match="greaterOrEqual"/>
@@ -36,7 +42,6 @@
       <import plugin="org.eclipse.tm.terminal" version="3.2.300" match="greaterOrEqual"/>
       <import plugin="org.eclipse.ui" version="3.8.0" match="greaterOrEqual"/>
       <import plugin="org.eclipse.ui.forms" version="3.5.200" match="greaterOrEqual"/>
-      <import plugin="org.eclipse.cdt.core" version="5.6.0" match="greaterOrEqual"/>
       <import plugin="org.eclipse.tcf.te.ui.terminals" version="1.3.0" match="greaterOrEqual"/>
       <import plugin="org.eclipse.tcf.te.ui.terminals.process" version="1.3.0" match="greaterOrEqual"/>
       <import plugin="org.eclipse.tcf.te.runtime.statushandler" version="1.3.0" match="greaterOrEqual"/>
@@ -48,7 +53,6 @@
       <import feature="org.eclipse.tm.terminal.serial" version="2.1.300" match="greaterOrEqual"/>
       <import feature="org.eclipse.tm.terminal.ssh" version="2.1.300" match="greaterOrEqual"/>
       <import feature="org.eclipse.tm.terminal.telnet" version="2.1.300" match="greaterOrEqual"/>
-      <import feature="org.eclipse.cdt.platform" version="8.3.0" match="greaterOrEqual"/>
    </requires>
 
    <plugin
diff --git a/target_explorer/features/org.eclipse.tcf.te.terminals.feature/p2.inf b/target_explorer/features/org.eclipse.tcf.te.terminals.feature/p2.inf
new file mode 100644
index 000000000..86a0e388b
--- /dev/null
+++ b/target_explorer/features/org.eclipse.tcf.te.terminals.feature/p2.inf
@@ -0,0 +1,22 @@
+###############################################################################
+# Copyright (c) 2014 Wind River Systems, Inc. and others. All rights reserved.
+# This program and the accompanying materials are made available under the terms
+# of the Eclipse Public License v1.0 which accompanies this distribution, and is
+# available at http://www.eclipse.org/legal/epl-v10.html
+#
+# Contributors:
+# Wind River Systems - initial API and implementation
+###############################################################################
+
+# Make the org.eclipse.cdt.native feature dependency "optional" since it didn't
+# exist in CDT 8.3 and we can also work on top of CDT 8.3.
+
+# The real dependencies are all properly managed via p2.inf in the
+# org.eclipse.tcf.te.ui.terminals.process/META-INF/p2.inf file.
+# The only benefit of requesting the feature here is that it allow pulling in
+# additional fragments that we are not aware of in the bundle's p2.inf.
+
+requires.0.namespace = org.eclipse.equinox.p2.iu
+requires.0.name = org.eclipse.cdt.native.feature.group
+requires.0.greedy = true
+requires.0.optional = true