Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 321692

Summary: Remove dependencies to plugins from features
Product: [Tools] CDT Reporter: Marc Khouzam <marc.khouzam>
Component: cdt-relengAssignee: Marc Khouzam <marc.khouzam>
Status: RESOLVED FIXED QA Contact: Vivian Kong <vivkong>
Severity: normal    
Priority: P3 CC: angvoz.dev
Version: 7.0   
Target Milestone: 8.0   
Hardware: PC   
OS: Windows 7   
Whiteboard:
Attachments:
Description Flags
Removing all plugin dependencies marc.khouzam: iplog-

Description Marc Khouzam CLA 2010-08-04 06:27:16 EDT 
At the last CDT conference call we discussed the fact that the different CDT features have dependencies filled towards plugins but that those have not been kept up-to-date, which can cause installations to fail when plugin versions change.

We believe that with P2, those dependencies are no longer necessary.  So, to make things simpler, we agree to remove all the dependencies to plugins from the different CDT features.
Comment 1 Marc Khouzam CLA 2010-08-04 09:52:08 EDT 
Created attachment 175837 [details]
Removing all plugin dependencies

I tried to find all plugin with 'feature' in them for CDT and I removed the <requires> section from feature.xml of each of them.

I don't really know how to test this change...
Should we do a build and try to install  every feature?
Comment 2 Marc Khouzam CLA 2010-08-13 07:26:22 EDT 
Anyone with more knowledge about releng have any opinions?
Comment 3 Marc Khouzam CLA 2010-08-23 14:27:56 EDT 
Ping?
Comment 4 Andrew Gvozdev CLA 2010-09-17 12:38:23 EDT 
Any progress on this bug? It may be a good occasion to apply the patch now as currently CDT in not-installable state:

Cannot complete the install because of a conflicting dependency.
  Software being installed: C/C++ GDB Hardware Debugging 7.0.0.201009170810 (org.eclipse.cdt.debug.gdbjtag.feature.group 7.0.0.201009170810)
  Software being installed: C/C++ Development Tools 7.1.0.201009170810 (org.eclipse.cdt.feature.group 7.1.0.201009170810)
  Only one of the following can be installed at once: 
    C/C++ Development Tools GDB/MI CDI Debugger Core 7.0.0.201006141710 (org.eclipse.cdt.debug.mi.core 7.0.0.201006141710)
    C/C++ Development Tools GDB/MI CDI Debugger Core 7.1.0.201009170810 (org.eclipse.cdt.debug.mi.core 7.1.0.201009170810)
  Cannot satisfy dependency:
    From: C/C++ GDB Hardware Debugging 7.0.0.201009170810 (org.eclipse.cdt.debug.gdbjtag.feature.group 7.0.0.201009170810)
    To: org.eclipse.cdt.gnu.debug.feature.group [6.0.0,7.0.0)
  Cannot satisfy dependency:
    From: C/C++ Development Tools 7.1.0.201009170810 (org.eclipse.cdt.feature.group 7.1.0.201009170810)
    To: org.eclipse.cdt.gnu.debug.feature.group [7.1.0.201009170810]
  Cannot satisfy dependency:
    From: C/C++ GNU Toolchain Debug Support 6.1.0.201006141710 (org.eclipse.cdt.gnu.debug.feature.group 6.1.0.201006141710)
    To: org.eclipse.cdt.debug.mi.core [7.0.0.201006141710]
  Cannot satisfy dependency:
    From: C/C++ GNU Toolchain Debug Support 7.1.0.201009170810 (org.eclipse.cdt.gnu.debug.feature.group 7.1.0.201009170810)
    To: org.eclipse.cdt.debug.mi.core [7.1.0.201009170810]

I think it can have something to do with in org.eclipse.cdt.debug.gdbjtag-feature/feature.xml we got
>       <import feature="org.eclipse.cdt.gnu.debug" version="6.0.0"/>
while org.eclipse.cdt.gnu.debug-feature has
> <feature
>      id="org.eclipse.cdt.gnu.debug"
>      label="%featureName"
>      version="7.1.0.qualifier"
>      provider-name="%providerName">

We could try test the patch trying another installation after the patch applied.
Comment 5 Marc Khouzam CLA 2010-09-17 14:16:31 EDT 
I was thinking that doing the commit on a Friday night may not be the best idea :-)  If I break something, we need to react fast.

I'll commit the patch on Monday morning.
Comment 6 Andrew Gvozdev CLA 2010-09-17 14:19:54 EDT 
You are afraid to break what is already broken? Anyway, I am fine with Monday, I know workaround :-).
I think it is a good idea to attach the final patch (if it changes) so you could revert it easily.
Comment 7 Marc Khouzam CLA 2010-09-20 05:06:35 EDT 
I committed to HEAD the patch that is attached.
Comment 8 CDT Genie CLA 2010-09-20 05:23:05 EDT 
*** cdt cvs genie on behalf of mkhouzam ***
Bug 321692: Remove dependencies to plugins from features

[*] feature.xml 1.7 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/xlc/org.eclipse.cdt.xlc.sdk-feature/feature.xml?root=Tools_Project&r1=1.6&r2=1.7

[*] feature.xml 1.5 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/lrparser/org.eclipse.cdt.core.lrparser.sdk.feature/feature.xml?root=Tools_Project&r1=1.4&r2=1.5

[*] feature.xml 1.9 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/edc/org.eclipse.cdt.debug.edc-feature/feature.xml?root=Tools_Project&r1=1.8&r2=1.9

[*] feature.xml 1.8 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/util/org.eclipse.cdt.util-feature/feature.xml?root=Tools_Project&r1=1.7&r2=1.8

[*] feature.xml 1.12 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/upc/org.eclipse.cdt.core.parser.upc.feature/feature.xml?root=Tools_Project&r1=1.11&r2=1.12

[*] feature.xml 1.4 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/cppunit/org.eclipse.cdt.cppunit-feature/feature.xml?root=Tools_Project&r1=1.3&r2=1.4

[*] feature.xml 1.3 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/upc/org.eclipse.cdt.bupc-feature/feature.xml?root=Tools_Project&r1=1.2&r2=1.3

[*] feature.xml 1.13 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/dsf-gdb/org.eclipse.cdt.gnu.dsf-feature/feature.xml?root=Tools_Project&r1=1.12&r2=1.13

[*] feature.xml 1.38 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/all/org.eclipse.cdt.testing-feature/feature.xml?root=Tools_Project&r1=1.37&r2=1.38

[*] feature.xml 1.5 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/upc/org.eclipse.cdt.core.parser.upc.sdk.feature/feature.xml?root=Tools_Project&r1=1.4&r2=1.5

[*] feature.xml 1.16 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/jtag/org.eclipse.cdt.debug.gdbjtag-feature/feature.xml?root=Tools_Project&r1=1.15&r2=1.16

[*] feature.xml 1.6 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/cross/org.eclipse.cdt.launch.remote-feature/feature.xml?root=Tools_Project&r1=1.5&r2=1.6

[*] feature.xml 1.44 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/all/org.eclipse.cdt.sdk-feature/feature.xml?root=Tools_Project&r1=1.43&r2=1.44

[*] feature.xml 1.8 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/dsf/org.eclipse.cdt.examples.dsf-feature/feature.xml?root=Tools_Project&r1=1.7&r2=1.8

[*] feature.xml 1.17 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/all/org.eclipse.cdt.gnu.build-feature/feature.xml?root=Tools_Project&r1=1.16&r2=1.17

[*] feature.xml 1.8 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/lrparser/org.eclipse.cdt.core.lrparser.feature/feature.xml?root=Tools_Project&r1=1.7&r2=1.8

[*] feature.xml 1.7 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/codan/org.eclipse.cdt.codan-feature/feature.xml?root=Tools_Project&r1=1.6&r2=1.7

[*] feature.xml 1.29 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/all/org.eclipse.cdt.platform-feature/feature.xml?root=Tools_Project&r1=1.28&r2=1.29

[*] feature.xml 1.7 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/c99/org.eclipse.cdt.core.parser.c99.feature/feature.xml?root=Tools_Project&r1=1.6&r2=1.7

[*] feature.xml 1.22 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/all/org.eclipse.cdt.gnu.debug-feature/feature.xml?root=Tools_Project&r1=1.21&r2=1.22

[*] feature.xml 1.10 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/memory/org.eclipse.cdt.debug.ui.memory-feature/feature.xml?root=Tools_Project&r1=1.9&r2=1.10

[*] feature.xml 1.23 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/xlc/org.eclipse.cdt.xlc.feature/feature.xml?root=Tools_Project&r1=1.22&r2=1.23

[*] feature.xml 1.3 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/all/org.eclipse.cdt.gdb-feature/feature.xml?root=Tools_Project&r1=1.2&r2=1.3